Fixes for `branches_sharing_code`
* Don't suggest moving modifications to locals used in any of the condition expressions
* Don't suggest moving anything after a local with a significant drop

Add `for_each_expr`

Auto merge of #8958 - Alexendoo:simple_filter_map, r=giraffate

Lint simple expressions in `manual_filter_map`, `manual_find_map`

changelog: Lint simple expressions in [`manual_filter_map`], [`manual_find_map`]

The current comparison rules out `.find(|a| a.is_some()).map(|b| b.unwrap())` because `a` being a reference can effect more complicated expressions, this adds a simple check for that case and adds the necessary derefs

There's some overlap with `option_filter_map` so `lint_filter_some_map_unwrap` now returns a `bool` to indicate it linted

Auto merge of #9120 - anall:bugfix/ice9041, r=Jarcho

Fix ICE in sugg::DerefDelegate with (named) closures

rustc comiler internals helpfully tell us how to fix the issue:
```
  to get the signature of a closure, use `substs.as_closure().sig()` not `fn_sig()`
```
Fixes ICE in #9041

This also makes this code in `sugg::DerefDelegate` match a different use `typ.fn_sig(…)` I found: in `mixed_read_write_in_expression` -- being strict on the value of `typ.kind()` will hopefully reduce any future possibility of ICE crashes in this area.

---

changelog: none

Fix ICE in sugg::DerefDelegate with (named) closures

rustc comiler internals helpfully tell us how to fix the issue:

  to get the signature of a closure, use `substs.as_closure().sig()` not `fn_sig()`

Fixes ICE in #9041

Auto merge of #9132 - hellow554:maybe_trait_bound_on_type_repetition, r=Manishearth

Maybe trait bound on type repetition

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: fix maybe trait on [`type_repetition_in_bounds`] lint

I simplified the two for loops, which did exactly the same. Only downside is, that I need a `copied`, but that's to convert from `&&` to `&`, to that should be a noop?

One more thing: I only handle [`TraitBoundModifier::Maybe`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc_hir/enum.TraitBoundModifier.html#variant.Maybe). Can anyone give me an example (and testcase) for [`TraitBoundModifier::MaybeConst`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc_hir/enum.TraitBoundModifier.html#variant.MaybeConst)?

closes #8770

Lint simple expressions in manual_filter_map, manual_find_map

fix incorrect suggestion for maybe trait bounds

trait_bounds: rework two loops into one

the two loops did practically the same, only the type were different (&&
vs &), so I used `copied` to convert `&&` and chained them together.
Instead of parsing the trait info manually, I use the already provided
method `get_trait_info_from_bound`.
Also, instead of using manual string writing, I used `join` by
`itertools`.

Auto merge of #9096 - Jarcho:needless_borrow_subs, r=Manishearth

Fix `needless_borrow` 9095

fixes #9095
changelog: Don't lint `needless_borrow` on method receivers when it would change which trait impl is called

Auto merge of #9099 - joshtriplett:unnecessary-lazy-eval-then-some, r=flip1995

Extend unnecessary_lazy_eval to cover `bool::then` -> `bool::then_some`

fixes #9097

changelog: Extend `unnecessary_lazy_eval` to convert `bool::then` to `bool::then_some`

Dogfood fixes to use `bool::then_some`

Add MSRV check for `bool::then_some`

Extend unnecessary_lazy_eval to cover `bool::then` -> `bool::then_some`

Auto merge of #9124 - Jarcho:shadow_ice, r=Alexendoo

Lint `shadow_*` lints in anon const blocks

changelog: Lint `shadow_*` lints in anon const blocks

Lint `shadow_*` lints in anon const blocks

Auto merge of #9121 - alex-semenyuk:fix_ui_test_cast_32_bits, r=flip1995

Fix stderr for cast_size_32bit

changelog: fix stderr for cast_size_32bit

Fix stderr for cast_size_32bit

Auto merge of #9117 - Jarcho:unsafe_block_closure, r=giraffate

Fix `undocumented_unsafe_blocks` in closures

fixes #9114
changelog: Fix `undocumented_unsafe_blocks` not checking for comments before the start of a closure

Auto merge of #9115 - Alexendoo:new-with-const-generics, r=llogiq

`new_without_default`: ignore const generics/lifetime params on `fn new`

Fixes #9113

No longer lints if `fn new` has any params

changelog: [`new_without_default`]: no longer lints const generics and lifetime params on `fn new`

Fix `undocumented_unsafe_blocks` in closures

new_without_default: ignore const generics/lifetime params on fn new

Auto merge of #9093 - Jarcho:deref_ice, r=giraffate

Fix ICE in `dereference.rs`

fixes #9089
changelog: Fix ICE when dereferencing or borrowing on explicit returns from closures

Auto merge of #9105 - Serial-ATA:lint-invalid-utf8, r=Jarcho

Add `invalid_utf8_in_unchecked`

changelog: Add [`invalid_utf8_in_unchecked`]
closes: #629

Don't know how useful of a lint this is, just saw this was a really old issue :smile:.

Add `invalid_utf8_in_unchecked`

Auto merge of #9110 - alex-semenyuk:fix_trailling_commas, r=xFrednet

Remove trailing spaces

Closes #9108
changelog: remove trailing spaces,  which are not allowed by the JSON standard.

Fixed comment

Auto merge of #9112 - xFrednet:9088-fix-lint-versions, r=Jarcho

Correct `[clippy::version]` for 1.62 lints and add note to docs

closes: https://github.com/rust-lang/rust-clippy/issues/9088

changelog: none

---

It feels weird to have such a short PR description. So, what should I add to this one... Yes, art:

```txt
 ,-----.,--.,--.
'  .--./|  |`--' ,---.  ,---.,--. ,--.
|  |    |  |,--.| .-. || .-. |\  '  /
'  '--'\|  ||  || '-' '| '-' ' \   '
 `-----'`--'`--'|  |-' |  |-'.-'  /
                `--'   `--'  `---'
```

```txt
 ,-----.,--.   ,--.,------. ,------.,--.   ,--.
'  .--./|  |   |  ||  .--. '|  .--. '\  `.'  /
|  |    |  |   |  ||  '--' ||  '--' | '.    /
'  '--'\|  '--.|  ||  | --' |  | --'    |  |
 `-----'`-----'`--'`--'     `--'        `--'
```

```txt
   _____ _      _____ _____  _______     __
  / ____| |    |_   _|  __ \|  __ \ \   / /
 | |    | |      | | | |__) | |__) \ \_/ /
 | |    | |      | | |  ___/|  ___/ \   /
 | |____| |____ _| |_| |    | |      | |
  \_____|______|_____|_|    |_|      |_|
```

Correct `[clippy::version]` for 1.62 lints and add note to docs

Remove trailing spaces

Auto merge of #9106 - usefulmove:master, r=flip1995

corrected README.md book link

This change corrects the book link on the Clippy Book README.md.

changelog: none

corrected README.md book link

Use correct substitutions when checking if `needless_borrow` can apply to a method receiver

Fix ICE in `dereference.rs`

Auto merge of #9087 - alex-semenyuk:fix_link, r=llogiq

Fix broken link

changelog: none

Fix link

Auto merge of #9085 - xFrednet:9081-update-format-push-version, r=Manishearth,flip1995

Correct lint version for `format_push_string`

Closes #9081

changelog: none

IDK what else to say. Look I can draw an ascii penguin =D:

```
 (^v^)
<(   )>
  w w
```

Correct lint version for `format_push_string`

Auto merge of #9082 - Alexendoo:let_unit_allow, r=xFrednet

Fix direct `#[allow]` attributes in `let_unit_value`

Fixes part of #9080

Not sure why it doesn't work when the lint is emitted at the statement, but switching it to the local works fine

changelog: Fix direct `#[allow]` attributes in [`let_unit_value`]

Auto merge of #9083 - giraffate:fix_link_in_changelog, r=flip1995

Fix some links

changelog: none

Fix some links

Fix allow attributes in let_unit_value

Auto merge of #9039 - Serial-ATA:dev-dogfood, r=giraffate

Add `cargo dev dogfood`

changelog: Add `cargo dev dogfood`

Part of #5394

Auto merge of #9075 - Serial-ATA:update-docs, r=flip1995

Add `dev deprecate` to the development basics

changelog: none

Auto merge of #9078 - hamza1311:patch-1, r=xFrednet

Fix broken hyperlink in documentation

changelog: none

The hyperlinks in [`is_digit_ascii_radix`](https://rust-lang.github.io/rust-clippy/master/index.html#is_digit_ascii_radix)'s docs are broken. This PR fixes those.

Fix broken hyperlink

Auto merge of #9058 - xFrednet:changelog-1-62, r=flip1995

Changelog for Rust 1.62 :t-rex:

This special artifact was just discovered. The artifact details changes to something called Clippy. Presently and from the context, we were unable to determine what this is supposed to be. All we know, is that it seems to have an active community around it which supports it. The members sometimes use paper clips as a secret symbol for Clippy.

I want to donate this artifact to the rust-lang organization, to have it displayed to the public.

---

changelog: none

Add `dev deprecate` to the development basics

Auto merge of #9074 - daxpedda:equatable-if-let-external-macro, r=Manishearth

Fix false-positive in `equatable_if_let`

Was linting in external macros. I guess now that I know about https://github.com/rust-lang/rust-clippy/pull/8694 it seems all kinda pointless until we resolve that.

Nevertheless, it's an improvement.

Fixes #9066.

changelog:`equatable_if_let` No longer lint on macros

Fix false-positive in `equatable_if_let`

Auto merge of #9071 - Alexendoo:8734, r=dswij

Uncomment test for #8734

I believe the issue was an interaction between rustfix and `span_lint_and_sugg_for_edges`, so this would've been fixed by https://github.com/rust-lang/rust/pull/98261 (Thanks, `@WaffleLapkin!)`

Closes #8734

changelog: none

Auto merge of #9070 - flip1995:ci-fix, r=xFrednet

Make sure bors success depends on metadata_collection

r? `@xFrednet`

Currently bors runs the `metadata_collection` but merges before the run is finished, because the bors success dummy step didn't depend on it. This also makes sure that the `metadata_collection` test is run at the same time as the other base runs to not produce overhead.

changelog: none

Uncomment test for #8734

Make sure bors success depends on metadata_collection

Auto merge of #9069 - flip1995:rustup, r=flip1995

Rustup

r? `@ghost`

changelog: none

Bump Clippy version -> 0.1.64

Bump nightly version -> 2022-06-30

Fix dogfood

Merge remote-tracking branch 'upstream/master' into rustup

Auto merge of #9068 - tamaroning:fix-typo-2022-6-30, r=giraffate,xFrednet

fix typo in Clippy Lints page

https://rust-lang.github.io/rust-clippy/master/#significant_drop_in_scrutinee

changelog: none

Update large_const_arrays.rs

Auto merge of #8666 - Jarcho:while_let_loop_7913, r=dswij

Don't lint `while_let_loop` when significant drop order would change

fixes #7226
fixes #7913
fixes #5717

For #5717 it may not stay fully fixed. This is only completely fixed right now due to all the allowed drop impls have `#[may_dangle]` on their drop impls. This may get changed in the future based on how significant drops are determined, but the example listed with `RefCell` shouldn't break.

changelog: Don't lint `while_let_loop` when the order of significant drops would change

fix

Add `dev dogfood` to the book

Don't lint `while_let_loop` when drop order would change

Extract util functions from `redundant_pattern_match`

Auto merge of #9043 - dswij:9037-test, r=Manishearth

`extra_unused_lifetimes` add FP test case emitting from derived attributes.

Add test to cover for #9014 which is fixed in #9037.

changelog: [`extra_unused_lifetimes`] Add FP test case emitting from derived attributes.

---

Seeing the FP from the test:

```sh
$ git revert -m 1 1d1ae10876d3eaa5c982dd3daa083b7c2fc363b9
$ TESTNAME=extra_unused_lifetime cargo uitest
```

Auto merge of #8981 - PrestonFrom:more_details_for_significant_drop_lint, r=flip1995

Add details about how significant drop in match scrutinees can cause deadlocks

Adds more details about how a significant drop in a match scrutinee can cause a deadlock and include link to documentation.

changelog: Add more details to significant drop lint to explicitly show how temporaries in match scrutinees can cause deadlocks.

Auto merge of #9046 - xFrednet:rust-97660-expection-something-something, r=Jarcho

Fix `#[expect]` for most clippy lints

This PR fixes most `#[expect]` - lint interactions listed in rust-lang/rust#97660. [My comment in the issue](https://github.com/rust-lang/rust/issues/97660#issuecomment-1147269504) shows the current progress (Once this is merged). I plan to work on `duplicate_mod` and `multiple_inherent_impl` and leave the rest for later. I feel like stabilizing the feature is more important than fixing the last few nits, which currently also don't work with `#[allow]`.

---

changelog: none

r? `@Jarcho`

cc: rust-lang/rust#97660

Auto merge of #8355 - Jarcho:explicit_auto_deref_2, r=flip1995

Add lint `explicit_auto_deref` take 2

fixes: #234
fixes: #8367
fixes: #8380

Still things to do:

* ~~This currently only lints `&*<expr>` when it doesn't trigger `needless_borrow`.~~
* ~~This requires a borrow after a deref to trigger. So `*<expr>` changing `&&T` to `&T` won't be caught.~~
* The `deref` and `deref_mut` trait methods aren't linted.
* Neither ~~field accesses~~, nor method receivers are linted.
* ~~This probably shouldn't lint reborrowing.~~
* Full slicing to deref should probably be handled here as well. e.g. `&vec[..]` when just `&vec` would do

changelog: new lint `explicit_auto_deref`

Auto merge of #8921 - Jarcho:merge_passes, r=llogiq

Add `Operators` lint pass

changelog: None

Move `SelfAssignment` into `Operators` lint pass

Fix dogfood

Move `PtrEq` into `Operators` lin pass

Move `NeedlessBitwiseBool` into `Operators` lint pass

Move `ModuloArithmetic` into `Operators` lint pass

Move some lints from `Misc` to `Operators`

Move `IntegerDivision` into `Operators` lint pass

Move `IdentityOp` into `Operators` lint pass

Move `FloatEqualityWithoutAbs` into `Operators` lint pass

Move `ErasingOp` into `Operators` lint pass

Move `EqOp` into `Operators` lint pass

Move `DurationSubsec` into `Operators` lint pass

Move `DoubleComparison` into `Operators` lint pass

Move `BitMask` into `Operators` lint pass

Move `AssignOps` into `Operators` lint pass

Move `Arithmetic` into `Operators` lint pass

Add `Operators` lint pass

Add `explicit_auto_deref` test for variadic function

Code cleanup

Handle future precedence issues in `explicit_auto_deref` + cleanup

Fix `needless_borrow` suggestion when calling a trait method taking `self`

Refactor `dereference.rs`

Merge `Position` and `AutoDerefStability`

Lint `explicit_auto_deref` immediately after `needless_borrow`

Lint `explicit_auto_deref` without a leading borrow

Only check parent node once in `dereference.rs`

Lint field accesses in `explicit_auto_deref`

Merge different parent walking loops in `dereference.rs`

`needless_borrow` will now walk further to find the target type.

Don't lint `explicit_auto_deref` on reborrows

Add `explicit_auto_deref` lint