Camille GILLOT
5b68fc16ed
Use ItemId as a strongly typed index.
2021-02-15 19:24:58 +01:00
Yoshitomo Nakanishi
fb91c76586
Add more tests for default_numeric_fallback
2021-02-15 23:33:27 +09:00
Yoshitomo Nakanishi
0198ac7bdd
Add fn_sig_opt to get fn signature from HirId
2021-02-15 23:29:18 +09:00
Yoshitomo Nakanishi
93796b2346
Add some restrictions to default_numeric_fallback to avoid FNs
2021-02-15 14:43:37 +09:00
bors
2f19f5f0d7
Auto merge of #6741 - ThibsG:BlockInIfConditions1141, r=flip1995
...
Do not lint when the closure is called using an iterator
Fix FP when the closure is used in an iterator for `blocks_in_if_conditions` lint
FIxes : #1141
changelog: none
2021-02-14 15:55:12 +00:00
ThibsG
12025506d6
Do not lint when the closure is called using an iterator
2021-02-14 16:21:12 +01:00
bors
9c3b43efdd
Auto merge of #6701 - camsteffen:collapsible-if, r=flip1995
...
Fix collapsible_if with attributes
changelog: Fix collapsible_if FP with attributes
Fixes #6593
2021-02-14 14:43:35 +00:00
bors
eb80ac4e72
Auto merge of #6697 - camsteffen:vec-init-push-fp, r=flip1995
...
Fix vec_init_then_push false positives
changelog: Fix vec_init_then_push false positives
Fixes #6615
2021-02-14 14:27:13 +00:00
bors
ce0a47ab8c
Auto merge of #81238 - RalfJung:copy-intrinsics, r=m-ou-se
...
directly expose copy and copy_nonoverlapping intrinsics
This effectively un-does https://github.com/rust-lang/rust/pull/57997 . That should help with `ptr::read` codegen in debug builds (and any other of these low-level functions that bottoms out at `copy`/`copy_nonoverlapping`), where the wrapper function will not get inlined. See the discussion in https://github.com/rust-lang/rust/pull/80290 and https://github.com/rust-lang/rust/issues/81163 .
Cc `@bjorn3` `@therealprof`
2021-02-13 20:30:07 +00:00
bors
5f611ceef7
Auto merge of #6736 - Y-Nak:reproducer-for-use_self-ice, r=flip1995
...
Add a minimal reproducer for the ICE in #6179
This PR is an auxiliary PR for #6179 , just add a minimal reproducer for the ICE discussed in #6179 .
See #6179 for more details.
changelog: none
2021-02-13 18:59:59 +00:00
Mara Bos
dd91987533
Fix clippy's path to the copy intrinsics.
2021-02-13 18:58:54 +01:00
Yoshitomo Nakanishi
87109bb4f5
Add minimal reproducer for ICE in #6179
2021-02-14 01:20:30 +09:00
bors
51aefbdd91
Auto merge of #6735 - matthiaskrgr:lintcheck, r=flip1995
...
lintcheck: accept env var to set crates.toml file
*Please write a short comment explaining your change (or "none" for internal only changes)*
changelog: lintcheck: accept LINTCHECK_TOML env var to set list of crates to be checked.
2021-02-13 16:03:07 +00:00
Philipp Krones
0256103c10
Use title case in clippy_dev README
2021-02-13 16:55:01 +01:00
bors
0ce5eccd66
Auto merge of #6738 - xFrednet:0000-update-labels-in-template-files, r=flip1995
...
Update labels in template files
It seems like we forgot to update the GitHub templates when we decided to update the label names. This PR just adjusts the templates to use the new label names :)
Context: [Zulip discussion about renaming the labels](https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/Label.20cleanup/near/224083870 )
---
changelog: none
2021-02-13 15:51:14 +00:00
xFrednet
8c5d301e1b
Updated labels in template files
...
For context: https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/Label.20cleanup/near/224083870
2021-02-13 16:45:34 +01:00
Matthias Krüger
fedfbb9011
lintcheck: explain sources.toml configuration
2021-02-13 01:28:00 +01:00
Matthias Krüger
5bbb1bc20a
lintcheck: env var LINTCHECK_TOML can be used to override toml file location (has precedence over --crates-toml flag)
2021-02-13 00:39:19 +01:00
boolean_coercion
d36fe85569
Made parens addition smarter and added tests with bless
2021-02-13 00:33:08 +02:00
bors
047f3e16bf
Auto merge of #6700 - daxpedda:panics-doc-unreachable, r=llogiq
...
Fix missing_panics_doc warning on `unreachable!`.
Fixes #6699 .
Are there any other test-cases I should cover?
changelog: [`missing_panics_doc`](https://rust-lang.github.io/rust-clippy/master/index.html#missing_panics_doc ): No longer lints on [`unreachable!`](https://doc.rust-lang.org/std/macro.unreachable.html )
2021-02-12 20:48:05 +00:00
boolean_coercion
642efabfbb
Fixed typos and updated to matches! where applicable
2021-02-12 11:54:22 +02:00
boolean_coercion
9194c11d69
Fixed doctests that shouldn't have been compiled
2021-02-12 11:54:22 +02:00
boolean_coercion
d1a627ab3b
Ran bless and rustfmt
2021-02-12 11:54:22 +02:00
boolean_coercion
0b31b470ad
Changed applicability to MaybeIncorrect because of surrounding braces
2021-02-12 11:54:22 +02:00
boolean_coercion
a389c02461
from_str_radix_10 should be done
2021-02-12 11:54:22 +02:00
boolean_coercion
64729390a1
Implemented majority of from_str_radix_10
2021-02-12 11:54:22 +02:00
boolean_coercion
b80ac2af9c
Added boilerplate
2021-02-12 11:54:22 +02:00
bors
605e9ba3d7
Auto merge of #6179 - flip1995:rewrite_use_self, r=phansch
...
Rework use_self impl based on ty::Ty comparison #3410 | Take 2
This builds on top of #5531
I already reviewed and approved the commits by `@montrivo.` So only the review of my commits should be necessary.
I would also appreciate your review `@montrivo,` since you are familiar with the challenges here.
Fixes #3410 and Fixes #4143 (same problem)
Fixes #2843
Fixes #3859
Fixes #4734 and fixes #6221
Fixes #4305
Fixes #5078 (even at expression level now 🎉 )
Fixes #3881 and Fixes #4887 (same problem)
Fixes #3909
Not yet: #4140 (test added)
All the credit for the fixes goes to `@montrivo.` I only refactored and copy and pasted his code.
changelog: rewrite [`use_self`] lint and fix multiple (8) FPs. One to go.
2021-02-12 06:09:07 +00:00
bors
fa2c1d63a2
Auto merge of #6719 - flip1995:changelog, r=xFrednet,flip1995
...
Update changelog for 1.51
~~Don't merge yet. This is the last step of the release process.~~ Release process for 1.50 is done, this can now be merged.
[Rendered](https://github.com/flip1995/rust-clippy/blob/changelog/CHANGELOG.md )
changelog: none
2021-02-11 16:06:07 +00:00
flip1995
cc82e559f6
Update changelog for 1.51
2021-02-11 16:49:44 +01:00
bors
8117573bc1
Auto merge of #6720 - flip1995:backport_remerge, r=flip1995
...
Backport remerge
changelog: none
r? `@ghost`
2021-02-11 15:10:42 +00:00
flip1995
feeed17bba
Merge remote-tracking branch 'upstream/beta' into backport_remerge
2021-02-11 16:09:12 +01:00
flip1995
8b9f4a0d34
Merge commit '70c0f90453701e7d6d9b99aaa1fc6a765937b736' into clippyup
2021-02-11 15:04:38 +01:00
bors
70c0f90453
Auto merge of #6718 - flip1995:rustup, r=flip1995
...
Rustup
r? `@ghost`
changelog: none
2021-02-11 13:38:47 +00:00
flip1995
4efc4541d2
Bump nightly version -> 2021-02-11
2021-02-11 14:37:13 +01:00
flip1995
00f9981f5c
Merge remote-tracking branch 'upstream/master' into rustup
2021-02-11 14:36:52 +01:00
bors
8dbcffed44
Auto merge of #6708 - matthiaskrgr:lintcheck, r=flip1995
...
some more lintcheck changes
* Explain why tokei is commented out in the lintcheck sources.
* If we specify a custom sources.toml, don't override the preexisting lintcheck logs, but rather start a new log with the filename depending on the sources.toml name.
* Start adding a readme.md to clippy_dev and add some information on how to use the lintcheck subcommand.
* Add support for path/local sources (I needed this for the next item which is: )
* Collect ICEs that happen while clippy checks crates
changelog: more lintcheck changes
2021-02-11 09:46:42 +00:00
Matthias Krüger
a6d493d52a
lintcheck: collect ICEs
2021-02-11 10:43:06 +01:00
Matthias Krüger
5e29aa6fdf
lintcheck: add support for path sources
2021-02-11 10:43:04 +01:00
Matthias Krüger
cfe154be8c
start a clippy-dev readme and some rough info on how to use lintcheck
2021-02-11 10:41:31 +01:00
Matthias Krüger
c7241b6e5e
lintcheck: make the log file be ${source-file}-logs.txt
...
this allows us to check multiple source.tomls and not worry about overriding our logfiles accidentally
2021-02-11 10:41:31 +01:00
Matthias Krüger
1025cd349d
lintcheck toml: explain why tokei is commented out
2021-02-11 10:41:31 +01:00
bors
beb49bad26
Auto merge of #6650 - daxpedda:cargo-common-metadata-publish, r=flip1995
...
Fix cargo_common_metadata warning on `publish = false`.
I believe `cargo_common_metadata` shouldn't trigger when `publish = false`, not sure if everybody agrees.
Made some tests to handle all edge-cases.
Fixes #6649 .
changelog: [`cargo_common_metadata`](https://rust-lang.github.io/rust-clippy/master/index.html#cargo_common_metadata ): No longer lints if [`publish = false`](https://doc.rust-lang.org/cargo/reference/manifest.html#the-publish-field ) is defined in the manifest
2021-02-11 09:34:44 +00:00
daxpedda
37f978299e
Add test for checking a combination of unreachable and panic.
2021-02-10 20:55:31 +01:00
bors
3784cdf98e
Auto merge of #6657 - ThibsG:FromIterParens, r=llogiq
...
Fix suggestions that need parens in `from_iter_instead_of_collect` lint
Fixes broken suggestions that need parens (i.e.: range)
Fixes : #6648
changelog: none
2021-02-10 19:37:27 +00:00
bors
a6d6b1b2ee
Auto merge of #6714 - alpaca-tc:fixed_tiny_typo, r=flip1995
...
Fix typo
changelog: none
2021-02-10 15:56:05 +00:00
alpaca-tc
94b8f23baf
Fix typo
2021-02-11 00:45:28 +09:00
bors
b5e4389f53
Auto merge of #6695 - TaKO8Ki:add-bytes-nth, r=phansch
...
New lint: `bytes_nth`
This pull request adds a new lint named `bytes_nth`.
---
closes: https://github.com/rust-lang/rust-clippy/issues/6391
changelog: Added a new lint: `bytes_nth`
2021-02-10 13:54:33 +00:00
bors
9c0ae2a5ab
Auto merge of #6687 - ehuss:stabilize-wrapper, r=flip1995
...
Stabilize workspace wrapper.
This fixes it so that `cargo clippy` doesn't share the same cache artifacts as `cargo check`. The Cargo side was stabilized a while ago (https://github.com/rust-lang/cargo/pull/8976 ), so this should be ready to go. I'm not aware of any blockers or concerns.
Closes #4612
---
changelog: `cargo clippy` no longer shares the same build cache as `cargo check`.
2021-02-10 13:27:53 +00:00
Cameron Steffen
52f98d832d
Use TyS::same_type
2021-02-10 11:58:21 +01:00