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

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

Rustup to rust-lang/rust#68133

changelog: none

Rustup to rust-lang/rust#68133

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

Auto merge of #5085 - JohnTitor:split-up-test, r=phansch

Split up `needless_range_loop` ui test

Part of #2038

changelog: none

don't fire empty_loop in no_std crates

improve empty_enum documentation

Split up `needless_range_loop` ui test

Update stderrs

Clean up `methods/mod.rs`

Auto merge of #5027 - sinkuu:vectored_io, r=phansch

Lint vectored IO in unused_io_amount lint

`read_vectored` & `write_vectored` require handling returned value likewise non-vectored methods. https://github.com/rust-lang/rust/issues/68041

---

changelog: lint vectored IO in `unused_io_amount` lint

Auto merge of #5082 - Areredify:issue-4980, r=flip1995

disable let_underscore_must_use in external macros

changelog: disable let_underscore_must_use in external macros
Closes #4980

disable let_underscore_must_use in external macros

Auto merge of #4945 - Areredify:as_deref, r=flip1995

add `option_as_ref_deref` lint

changelog: add a new lint that lints `option.as_ref().map(Deref::deref)` (and similar calls), which could be expressed more succinctly as `option.as_deref[_mut]()`. Closes #4918.

add `option_as_ref_deref` lint

Auto merge of #5079 - JohnTitor:fix-eq-op, r=flip1995

Ignore macros with `!` operators in `eq_op`

`SpanlessEq::eq_expr` doesn't ignore macros with `!` operators and I'm not sure we should ignore there, so I ignore in `eq_op` (and `op_ref`).

Fixes #5077

changelog: Fix false positive in `eq_op`

Ignore macros with `!` operators in `eq_op`

Auto merge of #5075 - JohnTitor:rustup-0122, r=flip1995

Rustup to rust-lang/rust#68140

changelog: none

Rustup to rust-lang/rust#68140

Auto merge of #5068 - JohnTitor:split-up-transmute, r=phansch

Split up `transmute` ui test

Part of #2038

changelog: none

Auto merge of #5028 - krishna-veerareddy:issue-5026-mem-ordering-fences, r=phansch

Detect usage of invalid atomic ordering in memory fences

Detect usage of `core::sync::atomic::{fence, compiler_fence}` with `Ordering::Relaxed` and suggest valid alternatives.

changelog: Extend `invalid_atomic_ordering` to lint memory fences

Fixes #5026

Decrease line length limit for stderr files

Auto merge of #5067 - JohnTitor:lint-skip-while-next, r=flip1995

Add `skip_while_next` lint

Fixes #4036

changelog: Add `skip_while_next` lint

Auto merge of #5070 - JohnTitor:suspicious-map-doc, r=flip1995

Improve `suspicious_map`documentation

Fixes #4793

changelog: none

Apply review comment

Auto merge of #5069 - JohnTitor:tweak-wording, r=flip1995

Tweak wording in `assertions_on_constants`

Displays actual macro names

changelog: none

Improve `suspicious_map`documentation

Tweak wording in `assertions_on_constants`

Split up `transmute` ui test

Remove trailing whitespaces

Add `skip_while_next` lint

Auto merge of #5056 - rust-lang:dissasociate-mut-key, r=flip1995

Avoid mut_key on types of unknown layout

This fixes #5020 by requiring a known layout for the key type before linting. Edit: This fixes #5043, too.

changelog: none

Avoid mut_key on types of unknown layout

Auto merge of #5032 - JohnTitor:add-sugg-some-result, r=flip1995

Add suggestions for `if_let_some_result`

Fixes #4991

This approach may be fragile though...

changelog: Add suggestions for `if_let_some_result`

Treat more strange pattern

Auto merge of #5066 - JohnTitor:split-up-1447, r=llogiq

Split up `if_same_then_else` ui test

Part of #2038

changelog: none

Split up `if_same_then_else` ui test

Apply review comments

Run `update_lints`

Reduce span range

Rename `ok_if_let` to `if_let_some_result`

Apply review comments

Add suggestion in `if_let_some_result`

Auto merge of #5065 - matthiaskrgr:rustup_28, r=matthiaskrgr

rustup https://github.com/rust-lang/rust/pull/67712

slice_patterns have been stabilized.

changelog: none