bors
97d1cfa2b4
Auto merge of #11286 - Centri3:#11283, r=Alexendoo
...
Suppress `question_mark` warning if `question_mark_used` is not allowed
Closes #11283
changelog: [`question_mark`]: Don't lint if `question_mark_used` is not allowed
2023-08-02 22:20:30 +00:00
Catherine Flores
4d49065a6c
Suppress question_mark
if question_mark_used
is not allowed
2023-08-02 14:13:16 -05:00
bors
7af5ea14fc
Auto merge of #11252 - Centri3:#11245, r=xFrednet
...
[`unwrap_used`]: Do not lint unwrapping on `!` or never-like enums
Fixes #11245
changelog: [`unwrap_used`]: Do not lint unwrapping on `!` or never-like enums
changelog: [`expect_used`]: Do not lint unwrapping on `!` or never-like enums
2023-08-02 19:02:47 +00:00
Catherine Flores
71c54137ea
Extract never-like into clippy_utils
2023-08-02 14:00:26 -05:00
Catherine Flores
779e0f4021
Do not lint unwrapping on !
or never-like enums
2023-08-02 14:00:12 -05:00
Deadbeef
b07de24a58
Remove constness from TraitPredicate
2023-08-02 15:38:00 +00:00
bors
78f5e0d3ec
Auto merge of #11284 - lengyijun:reorder, r=blyxyas
...
Alphabetically order arms in `methods/mod.rs` match
changelog: none
just order the arms in the match block
2023-08-02 11:45:32 +00:00
Your Name
72074a0f00
Alphabetically order arms in methods/mod.rs match
2023-08-02 16:44:26 +08:00
yukang
0ff6579eac
fix RedundantLocals clippy caused by async and await
2023-08-02 16:32:49 +08:00
Urgau
7ef1a54ffe
Rename incorrect_fn_null_checks to useless_ptr_null_checks (clippy side)
2023-08-01 20:04:01 +02:00
bors
588c1abb76
Auto merge of #11269 - y21:issue11268, r=Centri3
...
[`unnecessary_mut_passed`]: don't lint in macro expansions
Fixes #11268
changelog: [`unnecessary_mut_passed`]: don't lint in macro expansions
2023-08-01 05:15:09 +00:00
Philipp Krones
b0e64a9c09
Merge commit '5436dba826191964ac1d0dab534b7eb6d4c878f6' into clippyup
2023-07-31 23:53:53 +02:00
Samuel "Sam" Tardieu
f9a6dfa60d
New lint ignored_unit_patterns
2023-07-31 22:00:53 +02:00
y21
dc1e8b0dd9
[unnecessary_mut_passed
]: don't lint in macro expansions
2023-07-31 21:09:52 +02:00
Oli Scherer
084c90a305
Remove a bool
for color in favor of the WriteColor
trait wrapping colored and uncolored printing
2023-07-31 09:34:36 +00:00
Oli Scherer
9a0af82781
Use builder pattern instead of lots of arguments for EmitterWriter::new
2023-07-31 09:34:30 +00:00
bors
5436dba826
Auto merge of #11263 - c410-f3r:let-chain, r=Centri3
...
[`arithmetic_side_effects`] Fix #11262
Fix #11262
Rustc already handles paths that refer literals -> https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=d795058a2e1634c867288c20ff9432c8
```
changelog: [`arithmetic_side_effects`]: Ignore paths that refer literals
```
2023-07-30 20:16:57 +00:00
bors
2ab124126d
Auto merge of #11261 - y21:issue11260, r=blyxyas
...
[`unnecessary_find_map`]: look for then_some
Closes #11260
changelog: [`unnecessary_find_map`]: lint `.then_some()` in closure
2023-07-30 18:26:45 +00:00
Caio
35d434d08e
[arithmetic_side_effects] Fix #11262
2023-07-30 14:33:38 -03:00
y21
be6a103c8c
add more tests to unnecessary_find_map and unnecessary_filter_map
2023-07-30 16:52:53 +02:00
Matthias Krüger
f54263af58
Rollup merge of #112655 - WaffleLapkin:must_use_map_or, r=workingjubilee
...
Mark `map_or` as `#[must_use]`
I don't know what else to say.
r? libs
2023-07-30 14:25:08 +02:00
y21
008746cae4
[unnecessary_find_map
]: look for then_some
2023-07-30 13:51:35 +02:00
Maybe Waffle
0c93e30956
Mark map_or
as #[must_use]
2023-07-30 10:22:23 +00:00
bors
c0bdb3d337
Auto merge of #11258 - Jarcho:ice_11256, r=Centri3
...
Resolve type aliases in `type_certainty`
Fixes #11256
changelog: `unwrap_or_default`: Fix ICE when local types are declared using alias types
2023-07-30 07:57:47 +00:00
Jason Newcomb
caf601434b
Resolve type aliases in type_certainty
2023-07-30 03:26:32 -04:00
bors
4c2f460dcc
Auto merge of #8685 - Jarcho:redundant_closure_fixes, r=llogiq
...
`redundant_closure` fixes
fixes #8548
A good chunk of the code is fixing false negatives. The old code banned any non late-bound regions from appearing in the callee's signature. The new version checks when the late-bound region is actually required.
changelog: Better track when a early-bound region appears when a late-bound region is required in `redundant_closure`.
changelog: Don't lint `redundant_closure` when the closure gives explicit types.
2023-07-30 05:41:24 +00:00
Jason Newcomb
7423c2760b
Don't lint redundant closure for any function call inserted by the compiler.
2023-07-30 01:19:36 -04:00
Jason Newcomb
4d80a2ed2e
Rework redundant_closure
...
* Better track when a early-bound region appears when a late-bound region is required
* Don't lint when the closure gives explicit types.
2023-07-30 01:19:29 -04:00
Jason Newcomb
71cc39e1f2
Add debug assertions to implements_trait
...
Improve debug assertions for `make_projection`
2023-07-30 00:38:04 -04:00
bors
436060f637
Auto merge of #113422 - Urgau:cast_ref_to_mut-pre-beta, r=Nilstrieb
...
Rename and allow `cast_ref_to_mut` lint
This PR is a small subset of https://github.com/rust-lang/rust/pull/112431 , that is the renaming of the lint (`cast_ref_to_mut` -> `invalid_reference_casting`).
BUT also temporarily change the default level of the lint from deny-by-default to allow-by-default until https://github.com/rust-lang/rust/pull/112431 is merged.
r? `@Nilstrieb`
2023-07-29 07:48:44 +00:00
bors
0c0026e380
Auto merge of #111916 - fee1-dead-contrib:noop-method-call-warn, r=compiler-errors
...
make `noop_method_call` warn by default
r? `@compiler-errors`
2023-07-29 01:40:50 +00:00
bors
29730969b1
Auto merge of #11250 - flip1995:rustup, r=flip1995
...
Rustup
r? `@ghost`
changelog: none
2023-07-28 21:55:50 +00:00
Philipp Krones
65c5afd196
Bump nightly version -> 2023-07-28
2023-07-28 23:45:09 +02:00
Philipp Krones
3d60241841
Merge remote-tracking branch 'upstream/master' into rustup
2023-07-28 23:44:28 +02:00
León Orell Valerian Liehr
4c9d68e9b8
Make Clippy understand generic const items
2023-07-28 22:21:41 +02:00
bors
d3c5b488db
Auto merge of #11210 - y21:readonly_write_lock, r=giraffate
...
new lint: [`readonly_write_lock`]
Closes #8555
A new lint that catches `RwLock::write` calls to acquire a write lock only to read from it and not actually do any writes (mutations).
changelog: new lint: [`readonly_write_lock`]
2023-07-28 13:08:02 +00:00
y21
5e88003dda
pattern match on local usage slice to avoid ICE
2023-07-28 12:52:06 +02:00
bors
e97a770bf1
Auto merge of #114134 - fee1-dead-contrib:rm-constness-from-param-env, r=oli-obk
...
Remove `constness` from `ParamEnv`
This should be replaced by keyword generics/effects. cc #110395
r? `@oli-obk`
2023-07-28 08:53:12 +00:00
bors
295bdc028f
Auto merge of #10759 - blyxyas:unset_opt_env_unwrap, r=flip1995
...
Now `option_env_unwrap` warns even if a variable isn't set at compiletime
Fixes #10742
changelog: Fix false negative where `option_env_unwrap` wouldn't warn if the env variable isn't set at compile-time.
2023-07-27 19:52:46 +00:00
Timo
205a3741c1
make lint description more consistent (writer/reader -> write/read)
...
Co-authored-by: Takayuki Nakata <f.seasons017@gmail.com>
2023-07-27 21:19:35 +02:00
y21
136339f2d3
new lint: [readonly_write_lock
]
2023-07-27 21:19:35 +02:00
Deadbeef
39fb315396
bless clippy
2023-07-27 17:56:25 +00:00
bors
42d3370d84
Auto merge of #113298 - tgross35:update-bless-envs, r=oli-obk
...
Unite bless environment variables under `RUST_BLESS`
Currently, Clippy and Miri both use an environment variable to indicate that output should be blessed, but they use different variable names. In order to improve consistency, this patch applies the following changes:
- Rename the variable `MIRI_BLESS` (as used in the Miri subtree) to `RUST_BLESS`
- Rename the variable `BLESS` (as used in the Clippy subtree) to `RUST_BLESS`
- Move emitting `RUST_BLESS` into `prepare_cargo_test` so it is always available (I need this for a WIP PR)
---
I prefer something like `RUST_BLESS` to `BLESS` just for a lower chance of conflict (not super common but other tools [do use `BLESS`](https://grep.app/search?q=%22BLESS%22&case=true&words=true&filter[lang][0]=Text&filter[lang][1]=Rust&filter[lang][2]=Python&filter[lang][3]=C%2B%2B&filter[lang][4]=Markdown&filter[lang][5]=C&filter[lang][6]=JSON )), but I can change it to whatever is preferred.
Original discussion: https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/BLESS.20env.20var.3A.20rename.20to.20CLIPPY_BLESS
r? `@oli-obk`
cc `@flip1995`
2023-07-27 17:45:11 +00:00
Deadbeef
be0b4d5a9b
Remove constness
from ParamEnv
2023-07-27 15:50:42 +00:00
blyxyas
3bfccacca9
Add comments + Very minor Refactor
2023-07-26 23:16:24 +02:00
Trevor Gross
6ca6be6f2b
Unite bless environment variables under RUSTC_BLESS
...
Currently, Clippy, Miri, Rustfmt, and rustc all use an environment variable to
indicate that output should be blessed, but they use different variable names.
In order to improve consistency, this patch applies the following changes:
- Emit `RUSTC_BLESS` within `prepare_cargo_test` so it is always
available
- Change usage of `MIRI_BLESS` in the Miri subtree to use `RUSTC_BLESS`
- Change usage of `BLESS` in the Clippy subtree to `RUSTC_BLESS`
- Change usage of `BLESS` in the Rustfmt subtree to `RUSTC_BLESS`
- Adjust the blessable test in `rustc_errors` to use this same
convention
- Update documentation where applicable
Any tools that uses `RUSTC_BLESS` should check that it is set to any value
other than `"0"`.
2023-07-26 16:54:02 -04:00
Matthias Krüger
8615595021
Rollup merge of #114070 - blyxyas:iter_mut_symbol, r=oli-obk
...
Add `sym::iter_mut` + `sym::as_mut_ptr` for Clippy
We currently have `sym::iter` and `sym::iter_repeat`, this PR adds `sym::iter_mut` as it's useful for https://github.com/rust-lang/rust-clippy/pull/11038 and another Clippy lint, it also adds `sym::as_mut_ptr` as it's useful for https://github.com/rust-lang/rust-clippy/pull/10962 .
2023-07-26 20:49:13 +02:00
blyxyas
4e1db44404
Now option_env_unwrap
warns even if a variable isn't set at compile time.
2023-07-26 18:57:57 +02:00
bors
d446378143
Auto merge of #11228 - matthiaskrgr:fix_integration_tests2__with_test, r=flip1995
...
test that we correctly detect panics in intergation tests
should be merged after https://github.com/rust-lang/rust-clippy/pull/11225
changelog: add test for integration tests
2023-07-26 15:21:41 +00:00
Matthias Krüger
01b9f9d9da
ci: test that we fail CI if we don't find clippy panicking
2023-07-26 17:05:20 +02:00