Auto merge of #4512 - mati865:rustup, r=phansch

Rustup to rust-lang/rust#64111

changelog: none

Rustup to rust-lang/rust#64111

Auto merge of #4500 - jeremystucki:refactoring, r=flip1995

Small refactoring

changelog: none

Auto merge of #4506 - derivmug:fix-add-lint-example, r=phansch

Fix two occurences of TESTNAME=ui/foo_functions

changelog: Fix two occurrences of the wrong path to the foo_functions test in doc/adding_lints.md

This PR fixes the other two wrong paths to the example test. I already created a PR yesterday but only changed one. The old command was `TESTNAME=ui/foo_functions cargo uitest` and is now `TESTNAME=foo_functions cargo uitest`.

Fix two occurences of TESTNAME=ui/foo_functions

This commit fixes two occurences of the wrong path to the foo_functions example test in doc/adding-lints.md

Auto merge of #4489 - JohnTitor:fix-redundant-pattern-false-positive, r=flip1995

Fix `redundant_pattern` false positive

Fix #4428

changelog: Fix `redundant_pattern` false positive

Run rustfix

Auto merge of #4502 - lzutao:rustup-64141, r=phansch

Rustup "Minimize uses of `LocalInternedString`"

Rustup https://github.com/rust-lang/rust/pull/64141

changelog: none

Rustup "Minimize uses of `LocalInternedString`"

Auto merge of #4501 - derivmug:update-lint-doc, r=phansch

Remove ui/ from TESTNAME for foo lint example

changelog: Remove ui/ from TESTNAME in doc/adding-lints.md

This PR remove the `ui/` from the TESTNAME of the example foo lint in the 'Adding lints' documentation.

Remove ui/ from TESTNAME for foo lint example

Auto merge of #4495 - JohnTitor:fix-map-entry-false-positive, r=phansch

Fix `map_entry` false positive

Fixes #4219

changelog: Fix `map_entry` false positive

Make it auto fixable lint

Fix `redundant_pattern` false positive

Improve tests

Auto merge of #4487 - JohnTitor:deref-addrof-external-macro, r=flip1995

Don't check across macro boundary in `deref_addrof`

Fixes #4289

changelog: Allow `deref_addrof` in macros

Refactor option handling

Auto merge of #4418 - euclio:byte-lit-suggestion, r=flip1995

use a structured suggestion for char-lit-as-u8

changelog: use a structured suggestion for char-lit-as-u8

Run update_lints

use a structured suggestion for char-lit-as-u8

Auto merge of #4498 - sinkuu:checked_arithmetic_unwrap, r=flip1995

Add manual_saturating_arithmetic lint

changelog: add `manual_saturating_arithmetic` lint

Fixes #1557. This lint detects manual saturating arithmetics like `x.checked_add(10u32).unwrap_or(u32::max_value())` and suggests replacing with `x.saturating_add(10u32)`.

Hide variables in doc-test

Fix doctest and renaming src

Auto merge of #4490 - mikerite:fix-4364, r=flip1995

Fix `too_many_lines` false positive

changelog: Fix `too_many_lines` false positive

Add manual_saturating_arithmetic lint

Auto merge of #4454 - BO41:search_is_some, r=flip1995

Dereference one less on search_is_some and make it auto-fixable

Fixes #4453

changelog: none

Try to fix .fixed

Auto merge of #4479 - rust-lang:uninit_assume_init, r=flip1995

lint against `MaybeUninit::uninit().assume_init()`

changelog: add `uninit_assumed_init` lint

This fixes #4272

lint against `MaybeUninit::uninit().assume_init()`

Auto merge of #4486 - lzutao:fix-panic-unseparate-literals, r=flip1995

Fix index out of bound in case of empty snippet

cc #4480

changelog: none

Fix `map_entry` false positive

Auto merge of #4493 - JohnTitor:fix-build-bytes, r=flip1995

Rustup to rust-lang/rust#63561

changelog: none

Remove Allocation::bytes

Fix occurrences of `too_many_lines` violations

Fix `too_many_lines` false positive

Fixes #4364

Allow deref_addrof in macros

Fix index out of bound in case of empty snippet

Auto merge of #4482 - awoimbee:doc_explicit_counter_loop, r=flip1995

fix misleading doc for explicit_counter_loop lint

changelog: replace misleading examples for explicit_counter_loop & more concise `Why is it bad?` section

This fixes #4472

Auto merge of #4477 - mikerite:fix-4291, r=flip1995

Fix `extra_unused_lifetimes` false positive

Fixes #4291

changelog: Fix `extra_unused_lifetimes` false positive

fix misleading doc for explicit_counter_loop lint

Fix `needless_lifetimes` false positive

Simplify issue-4291 test

Fix `extra_unused_lifetimes` false positive

Fixes #4291

Auto merge of #4473 - phansch:fix_cast_lossless_fp, r=flip1995

Fix cast_lossless false positive in impl const fn

Fixes https://github.com/rust-lang/rust-clippy/issues/3656#issuecomment-526387382

changelog: Fix false positive in `cast_lossless`

Auto merge of #4475 - flip1995:rustup, r=phansch

Rustup to rust-lang/rust#60966

changelog: none

Rustup to rust-lang/rust#60966

Auto merge of #4446 - mikerite:fix-4437, r=phansch

Fix `match_as_ref` bad suggestion

Fixes #4437

changelog: Fix `match_as_ref` bad suggestion

Fix cast_lossless false positive in impl const fn

Fixes https://github.com/rust-lang/rust-clippy/issues/3656#issuecomment-526387382

Auto merge of #4450 - phansch:fix_const_fn_fp, r=flip1995

Fix missing_const_for_fn false positive

We don't want to lint if the type of the method implements drop.
(constant functions cannot evaluate destructors)

changelog: Fix `missing_const_for_fn` false positive

Fixes #4449

Add missing UI test change

Fix tests

Use the spans returned by utils::method_calls

Also return the method spans in utils::method_calls

Dereference one less on search_is_some and make it auto-fixable

Auto merge of #4408 - phansch:more_rustfix_tests, r=flip1995

More rustfix tests

<!--
Thank you for making Clippy better!

We're collecting our changelog from pull request descriptions.
If your PR only updates to the latest nightly, you can leave the
`changelog` entry as `none`. Otherwise, please write a short comment
explaining your change.

If your PR fixes an issue, you can add "fixes #issue_number" into this
PR description. This way the issue will be automatically closed when
your PR is merged.

If you added a new lint, here's a checklist for things that will be
checked during review or continuous integration.

- [ ] Followed [lint naming conventions][lint_naming]
- [ ] Added passing UI tests (including committed `.stderr` file)
- [ ] `cargo test` passes locally
- [ ] Executed `./util/dev update_lints`
- [ ] Added lint documentation
- [ ] Run `./util/dev fmt`

[lint_naming]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints

Note that you can skip the above if you are just opening a WIP PR in
order to get feedback.

Delete this line and everything above before opening your PR -->

cc #3630

This is probably easier reviewed per-commit.

changelog: none

Auto merge of #4469 - vityafx:master, r=flip1995

Improve the cognitive_complexity lint's warning.

Adds the value of complexity limit set for the lint
to the warning.

Fixes #4466

Signed-off-by: Victor Polevoy <fx@thefx.co>
changelog: improve the `cognitive_complexity` lint's warning message.

Improve cognitive_complexity lint's warning.

Adds the value of complexity limit set for the lint
to the warning.

Fixes #4466

Signed-off-by: Victor Polevoy <fx@thefx.co>

Merge branch 'master' into fix-4437

Fix missing_const_for_fn false positive

We don't want to lint if any of the input parameters implement drop.

(constant functions cannot evaluate destructors)

Make useless_attribute suggestion MaybeIncorrect

Auto merge of #4465 - JohnTitor:add-note-to-fix-toolstate, r=flip1995

Add note to fix toolstate

The idea from https://github.com/rust-lang/rust-clippy/pull/4462#issuecomment-525725587
It's nice that we have the documentation to guide fix toolstate.

changelog: none

Add note to fix toolstate

Auto merge of #4464 - flip1995:fetch_prs_between, r=phansch

Also fetch rollup merges in fetch_prs_between.sh script

Otherwise rolled up PRs won't be included in the changelog, e.g. 236666138fa15fdd48b1c12180b2c9dc936441b4

changelog: none

Also fetch rollup merges in fetch_prs_between.sh script

Auto merge of #4463 - flip1995:rollup-240kr2c, r=flip1995

Rollup of 2 pull requests

Successful merges:

 - #4459 (Add note to avoid confusing)
 - #4460 (Fix `inherent_to_string` false positive)

Failed merges:

r? @ghost

changelog: none

Rollup merge of #4460 - JohnTitor:fix-inherent-to-string, r=flip1995

Fix `inherent_to_string` false positive

Fixes #4457

changelog: fixes `inherent_to_string` false positive

Rollup merge of #4459 - JohnTitor:update-testing-notes, r=flip1995

Add note to avoid confusing

Fixes #4456

changelog: none

Auto merge of #4462 - JohnTitor:fix-build-arg, r=flip1995

Replace `Arg` with `Param`

Fix build issue.
Rustup to https://github.com/rust-lang/rust/pull/63127

changelog: none

Replace `Arg` with `Param`

Update int_plus_one suggestion text

This is now in line with the recommendations from here:
https://doc.rust-lang.org/nightly/nightly-rustc/rustc_errors/struct.Diagnostic.html#method.span_suggestion

Change float_cmp Applicability to `HasPlaceholders`

The suggestion includes an 'errors' placeholder currently, so it's not
`MachineApplicable`

Add run-rustfix for rename test

I removed the `Bar` static and `Foo` impl because they are not needed
for what's tested here as far as I can tell.

Add run-rustfix for outer_expn_data

Add run-rustfix for useless_attribute test

Add run-rustfix for int_plus_one test

Add run-rustfix for assign_ops test

Fix `inherent_to_string` false positive

Add note to avoid confusing

Auto merge of #4444 - phansch:split_up_cmp_owned2, r=flip1995

Split up cmp_owned tests, add run-rustfix

Some of the cmp_owned tests emitted non-machine-applicable suggestions,
so I moved them to `tests/ui/cmp_owned/without_suggestion.rs` and added
`// run-rustfix` to the other half.

changelog: none

cc #3630

Auto merge of #4445 - phansch:fix_unused_unit_sugg, r=flip1995

Fix unused_unit false positive

changelog: Fix `unused_unit` false positive

For some reason the `expr` of `stmt.node` didn't contain the expansion information, but the `stmt.span` does.

Fixes #4076

Use correct path in comment

Co-Authored-By: Philipp Krones <hello@philkrones.com>

Auto merge of #4430 - lzutao:defid_trait_alias, r=flip1995

Account for trait alias when looking for defid

I hit the crash on the `expect` call when running clippy on rustc libcore.
Hopefully this will fix it.
changelog: none

Add regression test for looking for trait defid in nocore

Auto merge of #4436 - BO41:written_as, r=phansch

Add some "could be written as" examples

fixes #4405
changelog: none

Auto merge of #4421 - lzutao:unsep_literal, r=flip1995

Cleaner code for unsep literals

Continuing discussion in https://github.com/rust-lang/rust-clippy/pull/4401#issuecomment-522498754
changelog: none
r? @flip1995

Auto merge of #4451 - hvenev:non-exec, r=flip1995

Mark rust source file as non-executable.

It would be nice if you can backport this to beta as well.

changelog: none

Auto merge of #4431 - phansch:more_rustfix, r=flip1995

Add 3 more run-rustfix headers

changelog: none

cc #3630

Cleaner code for unsep literals

Remove empty *.std{err,out} files

Mark rust source file as non-executable.

Account for trait alias when looking for defid

Auto merge of #4447 - phansch:fix_build, r=matthiaskrgr

Rustup to https://github.com/rust-lang/rust/pull/63854

changelog: none

Rustup to https://github.com/rust-lang/rust/pull/63854

Fix `match_as_ref` bad suggestion

Fixes #4437

Fix unused_unit false positive

Fixes #4076

Fix lint doc

Split up cmp_owned tests, add run-rustfix

Some of the cmp_owned tests emitted non-machine-applicable suggestions,
so I moved them to `tests/ui/cmp_owned/without_suggestion.rs` and added
`// run-rustfix` to the other half.

cc #3630

Auto merge of #4443 - jeremystucki:methods-refactoring, r=phansch

Small refactoring of methods/mod.rs

changelog: none

Refactor some minor things

Use more if-chains