Update needless_continue stderr

Rewrite suggestion generation of needless_continue

Update block_in_if_condition test files

Make block_in_if_condition auto applicable

Optionally indent snippet_block relative to an Expr

Auto merge of #5139 - lzutao:linecount, r=llogiq

dev: Use bytecount for faster line count

changelog: none

Auto merge of #5125 - Areredify:too_many_bools, r=flip1995

Port mitsuhiko's excessive bools lints

Closes #4 .
changelog: add `struct_excessive_bools` and `fn_params_excessive_bools` lints.
I moved is_trait_impl_item check because at first I implemented it as a late pass for some reason but then I realized it's actually an early lint. But it's a useful function to have, should I move it into a separate pr?

dev: Use bytecount for faster line count

Auto merge of #5132 - JohnTitor:fix-fp-in-unwrap-lint, r=flip1995

Do not lint `unnecessary_unwrap` in external macros

Fixes #5131

I think we shouldn't lint `{panicking, unnecessary}_unwrap` in macros, not just `assert!`.

changelog: Fix false positive in `unnecessary_unwrap`

Auto merge of #5121 - lzutao:comptest, r=flip1995

Improve compile_test and dogfood tests

* Hopefully this finally resolves the "multiple matching crates" error, e.g. #4015
* This handle some convenient CARGO env like CARGO_TARGET_DIR and CARGO_BUILD_TARGET

changelog: none

add excessive bools lints

Use lazy_static

move is_trait_impl_item check from functions.rs to utils

Add `serde_derive` to the need-to-be-disambiguated-crates list

Fix dogfood to use cargo mod too

compile-test: Handle CARGO_TARGET_DIR and transitive deps

Auto merge of #5137 - lzutao:race, r=flip1995

Prevent failing to restart setup-toolchain

If `rustup-toolchain-install-master` fails to install master
toolchain (in case `rustup update` executes at the same remove the
tmp directory), `cargo search` (which use master toolchain after
`rustup override` command) will fail. This leads to setup-toolchain
fails too.

changelog: none

Prevent failing to restart setup-toolchain

If `rustup-toolchain-install-master` fails to install master
toolchain (in case `rustup update` executes at the same remove the
tmp directory), `cargo search` (which use master toolchain after
`rustup override` command) will fail. This leads to setup-toolchain
fails too.

Auto merge of #5135 - lzutao:config, r=flip1995

Few improvement to `utils::conf` module

* Fix a few typos
* Handle Option<&Path> early
* Use `env::var_os` when possible

changelog: none

Few improvement to `utils::conf` module

* Fix a few typos
* Handle Option<&Path> early
* Use `env::var_os` when possible

Auto merge of #5061 - ThibsG:UselessMatch3664, r=flip1995

Add new lint: match with a single binding statement

This lint catches `match` statements that binds to only one block and could be rewritten using a simple `let` statement.

Lint name: MATCH_SINGLE_BINDING

fixes: #3664

changelog: add lint for match with single binding statement

Manage macros case + move to MaybeIncorrect when binding values

Merge fixes

Add wild and struct handling

Use span_lint_and_sugg + move infaillible lint

 - moving infaillible lint to prevent collisions

Re-cover use of unnecessary unwraps in macros

Auto merge of #5124 - flip1995:backport_back_merge, r=flip1995

Merge backport branch back into master

Merge back backport branch rust-1.42.0 into master.

Waiting on rust-lang/rust#68753

changelog: none

Add new lint: match with a single binding statement

 - Lint name: MATCH_SINGLE_BINDING

Do not lint `unnecessary_unwrap` in macros

Auto merge of #5130 - JohnTitor:split-up-index-slice, r=flip1995

Split up `indexing_slicing` ui test

Closes #2038

Now all the stderrs are less than 200 lines :tada:

changelog: none

Auto merge of #5129 - JohnTitor:use-checked-sub, r=flip1995

Use `checked_sub` to avoid index out of bounds

(Fixes) #4681 (possibly)

The issue likely occurs due to `lit_snip.len() < suffix.len() + 1`. You can see similar backtrace to change it to `lit_snip.len() - suffix.len() - 1000` or something then run `cargo test --release`.
But I couldn't come up with the test so I'd leave the issue open if we want.

changelog: Fix potential ICE in `misc_early`

Remove unnecessary comment

Auto merge of #5127 - Areredify:issue-5115, r=flip1995

improve 'iter_nth_zero' documentation

closes #5115

changelog: Improve the documentation of `iter_nth_zero`

improve 'iter_nth_zero' documentation

Use `checked_sub` to avoid index out of bounds

Decrease line length limit for stderrs

Split up `indexing_slicing` ui test

Auto merge of #5120 - JohnTitor:split-up-drop-forget-ref, r=flip1995

Split up `drop_forget_ref` ui test

Part of #2038

changelog: none

Auto merge of #5119 - JohnTitor:tweak-doc, r=flip1995

Tweak documentation in `multiple_crate_versions`

This example isn't reproducible now since `ctrlc` upgrades `winapi` to `0.3.x` in `3.1.1`. We should pin their versions to trigger lint correctly.

changelog: none

Merge remote-tracking branch 'upstream/rust-1.42.0' into backport_back_merge

Auto merge of #5123 - JohnTitor:rustup-0202, r=matthiaskrgr

Rustup to rust-lang/rust#68133

changelog: none

Rustup to rust-lang/rust#68133

Move debug_assertions_with_mut_call to nursery

Add test for `await` in `debug_assert!(..)`

Don't trigger `debug_assert_with_mut_call` on `.await`

Small refactor of mutable_debug_assertions

Split up `drop_forget_ref` ui test

Pin versions to trigger lint correctly

Auto merge of #5116 - matthiaskrgr:rustup_29, r=flip1995

update test stderr

changelog: none

update test stderr

Auto merge of #5106 - flip1995:dbg_assert_mut_async, r=oli-obk

Don't trigger [debug_assert_with_mut_call] on debug_assert!(_.await)

Fixes #5105

cc #5112

changelog: Don't trigger [`debug_assert_with_mut_call`] on `debug_assert!(_.await)` and move it to nursery.

Move debug_assertions_with_mut_call to nursery

Add test for `await` in `debug_assert!(..)`

Don't trigger `debug_assert_with_mut_call` on `.await`

Small refactor of mutable_debug_assertions

Auto merge of #5111 - flip1995:changelog, r=phansch

Update changelog

[Rendered](https://github.com/flip1995/rust-clippy/blob/changelog/CHANGELOG.md)

r? @phansch

changelog: none

Auto merge of #5109 - phansch:ciao_util_dev, r=flip1995

Deprecate util/dev in favor of cargo alias

This means one less shell script and a bit more cross-platform support
for contributors.

If you've been using `./util/dev` before, this now becomes `cargo dev`.

The key part of this change is found in `.cargo/config` where an alias for calling the `clippy_dev` binary is defined.

changelog: none

Mark clippy_project_root as `must_use`

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

Replace one more copy-pasted clippy_project_root fn

Auto merge of #5110 - Aloso:patch-1, r=flip1995

Fix syntax highlighting of code fences

The documentation for RESULT_EXPECT_USED includes this code:

    let res: Result<usize, ()> = Ok(1);
    res?;
    # Ok::<(), ()>(())

Because the code fence didn't start with `rust`, the code wasn't highlighted and the line starting with `#` was displayed on the website. This is now fixed.

EDIT: I noticed that highlighting for some other lints is broken as well. It only works if the code fence looks like this:

````markdown
```rust
// ..
```
````

However, many code blocks were ignored. I un-ignored most code blocks and made them compile by adding hidden code with `#`. While doing so, I found two mistakes:

```rust
opt.map_or(None, |a| a + 1)
// instead of
opt.map_or(None, |a| Some(a + 1))
```
and

```rust
fn as_str(self) -> &str
// instead of
fn as_str(self) -> &'static str
```

changelog: none

Un-ignore most code blocks to render correctly on website, correct mistakes

Update changelog

Fix code formatting for more lints

Auto merge of #5108 - JohnTitor:split-up-0130, r=flip1995

Split up `match` ui test

Part of #2038

Also, this decreases the line length limit to 220.

changelog: none

Fix syntax highlighting of code fence

The documentation for RESULT_EXPECT_USED includes this code:

    let res: Result<usize, ()> = Ok(1);
    res?;
    # Ok::<(), ()>(())

Because the code fence didn't start with `rust`, the code wasn't highlighted and the line starting with `#` was displayed on the website. This is now fixed.

Move project_root function to clippy_dev/src/lib.rs

This allows us to use the method in both `fmt.rs` and `lib.rs` in
multiple places. The downside is that we panic inside the method now,
instead of using the error handling in `fmt.rs`. We may want to
centralize the error handling for clippy_dev at some point, though.

Deprecate util/dev in favor of cargo alias

If you've been using `./util/dev` before, this now becomes `cargo dev`.

The key part of this change is found in `.cargo/config`.

This means one less shell script and a bit more cross-platform support
for contributors.

Auto merge of #5101 - Areredify:let_underscore_lock, r=flip1995

add `let_underscore_lock` lint

closes #1574
changelog: add `let_underscore_lock` lint

I am not entirely sure about my docs/messages wording here, improvements are welcome

Auto merge of #5107 - JohnTitor:revive-test, r=flip1995

Revive test in `enum_clike_unportable_variant`

Revive one test since mentioned ICE has been resolved.

changelog: none

lint all guard types, not just lock functions

decouple 'let_underscore' tests

add  lint

Decrease line length limit for stderr files

Split up `match` ui test

Revive test in `enum_clike_unportable_variant`

Auto merge of #5058 - xiongmao86:issue4903, r=flip1995

Closes Issue4903

fixes #4903.

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

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

changelog: implement lint that warn about single component path imports(`use ident;`).

Reformat code.

Add test and update reference.

Declare lint and implement lint logic.

Auto merge of #5104 - JohnTitor:rustup-0129, r=Manishearth

Rustup to rust-lang/rust#68512

changelog: none

Rustup to rust-lang/rust#68512

Auto merge of #5099 - JohnTitor:split-up-non-copy-const, r=flip1995

Split up `non_copy_const` ui test

Part of #2038
Maybe there is a better way to avoid duplications of constants.

changelog: none

Auto merge of #5098 - JohnTitor:rename-span-lints, r=flip1995

Rename `span_{help, note}_and_lint` to `span_lint_and_*`

`span_*_and_lint` and `span_lint_and_*` make us confused. The order should be the same.

[context](https://github.com/rust-lang/rust-clippy/pull/5084#issuecomment-578411309)

changelog: none

Split up `non_copy_const` ui test

Rename `span_note_and_lint` to `span_lint_and_note`

Rename `span_help_and_lint` to `span_lint_and_help`

Auto merge of #5084 - JohnTitor:clean-up-span-lint, r=flip1995

Clean up `span_lint` in `methods/mod.rs`

Uses `span_help_and_lint` instead of `span_lint` and `span_lint_and_sugg` instead of `span_lint_and_then`.

changelog: none

Auto merge of #5097 - phansch:remove_fixme, r=llogiq

Resolve FIXME: String::new is now a const fn

`const_string_new` is stable since Rust 1.39

changelog: none

Resolve FIXME: String::new is now a const fn

`const_string_new` is stable since Rust 1.39

Auto merge of #5096 - eddyb:mac-name, r=oli-obk

Don't use ExpnKind::descr to get the name of a bang macro.

This is the same change as the first commit in rust-lang/rust#68407, but applied to clippy.
The new code should work both before and after the changes in rust-lang/rust#68407.

changelog: none

Don't use ExpnKind::descr to get the name of a bang macro.

Apply review comments

Auto merge of #5083 - Areredify:issue-4399, r=flip1995

dont fire `possible_missing_comma` if intendation is present

Closes #4399
changelog: dont fire `possible_missing_comma` if intendation is present

I suspect there is already a utils function for indentation (but searching indent didn't yield a function for that), and my solution is certainly not universal, but it's probably the best we can do.

dont fire possible_missing_comma if intendation is present

Auto merge of #5081 - Areredify:vec_box_threshold, r=flip1995

add size parameter for `vec_box`  lint

changelog: add size threshold for the `vec_box` lint, currently 4096 bytes (one page) (subject to change). Closes #3547.

diff is a little bit confusing due to some refactoring (moving free functions to lint struct functions), relevant portion is [this](https://github.com/rust-lang/rust-clippy/compare/master...Areredify:vec_box_threshold?expand=1#diff-1096120ca9143af89dcc9175ea92b54aR294-R298). In hindsight should've been different commits, but oh well.

add size parameter for  lint

Auto merge of #5090 - JohnTitor:split-up-match-same-arms, r=phansch

Split up `match_same_arms` ui test

Part of #2038

changelog: none

Split up `match_same_arms` ui test

Auto merge of #5087 - Areredify:issue-4905, r=phansch

improve `empty_enum` documentation

closes #4905
changelog: improve `empty_enum` help message and documentation.

Auto merge of #5086 - Areredify:issue-3746, r=phansch

don't fire `empty_loop` in `no_std` crates

closes #3746.
changelog: move no_std detection to utils, don't fire empty_loop in no_std crates