check last statement

cargo dev fmt

check macro in eq_block

check only first statement

check macro_backtrace only

check only the end

use get_diagnostic_name for checking macro_call

feat(fix): ignore todo! and unimplemented! in if_same_then_else

Ignore bodies containing `todo!()` in `clippy::if_same_then_else`

Auto merge of #8964 - tamaroning:read_zero_byte_vec, r=dswij

Warn about read into zero-length `Vec`

Closes #8886

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

changelog: none

Auto merge of #8999 - Alexendoo:error-pattern, r=xFrednet

Remove error-pattern comments

The `clippy_lints` one [is unused](https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/.60error-pattern.60), the others in `ui-toml` also appear not to have an effect

changelog: none

Remove error-pattern comments

add lint

Auto merge of #8947 - Serial-ATA:lint-produces-output, r=xFrednet

Add lint output to lint list

changelog: Add the ability to show the lint output in the lint list

This just adds the logic to produce the output, it hasn't been added to any lints yet. It did help find some mistakes in some docs though :smile:.

### Screenshots

<details>
<summary>A single code block</summary>

</details>

<details>
<summary>A single code block with a "Use instead" section</summary>

</details>

<details>
<summary>Multiple code blocks</summary>

</details>

This is the last task in #7172 :tada:.
r? `@xFrednet` (?)

Auto merge of #8901 - Jarcho:sharing_code, r=dswij

Rework `branches_sharing_code`

fixes #7378

This changes the lint from checking pairs of blocks, to checking all the blocks at the same time. As such there's almost none of the original code left.

changelog: Don't lint `branches_sharing_code` when using different binding names

Auto merge of #8997 - Jarcho:clap_deprecate, r=flip1995

Fix `clap` deprecation warnings

Clap `3.2.0` deprecated a few functions used by lintcheck.

changelog: None

Fix `clap` deprecation warnings

Auto merge of #8988 - xFrednet:8947-test-monster-in-bors-ci, r=flip1995

Test metadata collection in Bors CI workflow

This PR adds a new check to bors CI workflows, which ensures that the metadata collection success, when it's run as part of the `deploy` script. I've only added it to bors workflows, as the runtime will be high while it'll also succeed most of the time. This is a preparation for rust-lang/rust-clippy#8947.

---

changelog: none

r? `@ghost`

Auto merge of #8983 - Alexendoo:redundant_closure_tests, r=giraffate

Add tests for some fixed `redundant_closure` issues

closes #7861
closes #5939

changelog: none

Test metadata collection in Bors CI workflow

Add tests for some fixed redundant_closure issues

Auto merge of #8976 - xFrednet:rust-97660-catch-emissions-with-expect, r=Jarcho

Fix some `#[expect]` lint interaction

Fixing the first few lints that aren't caught by `#[expect]`. The root cause of these examples was, that the lint was emitted at the wrong location.

---

changelog: none

r? `@Jarcho`

cc: rust-lang/rust#97660

Auto merge of #8979 - Alexendoo:version-numbers, r=flip1995

Correct some `#[clippy::version]`s

Some were a bit off

changelog: none

Correct some `#[clippy::version]`s

Auto merge of #8954 - Serial-ATA:doc-comment-issues, r=xFrednet

Improve lint doc consistency

changelog: none

This is a continuation of #8908.

Notable changes:
- Removed empty `Known Problems` sections
- Removed "Good"/"Bad" language (replaced with "Use instead")
- Removed (and added some :smile:) duplication
- Ignored the [`create_dir`] example so it doesn't create `clippy_lints/foo` :smile:

Fix [`needless_bool`] example

Fix `#[expect]` for `default_numeric_fallback`

Auto merge of #8907 - kyoto7250:fix_8898, r=giraffate

fix(lint): check const context

close: https://github.com/rust-lang/rust-clippy/issues/8898

This PR fixes a bug in checked_conversions.

Thank you in advance.

changelog: check const context in checked_conversions.

Auto merge of #8950 - Jarcho:derive_non_pub, r=dswij

Fixes for `derive_partial_eq_without_eq`

fixes  #8875

changelog: Don't lint `derive_partial_eq_without_eq` on non-public types
changelog: Better handle generics in `derive_partial_eq_without_eq`

Auto merge of #8960 - Jarcho:iter_cloned, r=giraffate

Changes to `iter_overeager_cloned`

fixes: #8494

changelog: Don't lint `iter_overeager_cloned` on `.cloned().flatten()` when `T::Item` doesn't implement `IntoIterator`

Hide unnecessary code in [`implicit_saturating_sub`]

Changes to `iter_overeager_cloned`
* Don't lint on `.cloned().flatten()` when `T::Item` doesn't implement `IntoIterator`
* Reduce verbosity of lint message
* Narrow down the scope of the replacement range

Suppress `unused` in doc examples

Add lint output to lint list

Auto merge of #7359 - joshrotenberg:clippy_guide, r=xFrednet

Clippy book

A work in progress Clippy Book using mdbook. See #6011.

This is currently just a moving around of things:

1. The current README.md split up a bit and put into sections.
1. A rough outline of Clippy lint categories (currently no content, potentially add a basic introduction for each and some example, see questions below.
1. The `docs` content repurposed into a top level `Development` section.
1. The current Roadmap.

Some big questions:

1. is `guide/` the right place? I'm modeling after mdbook itself.
1. What is the relationship between ALL the Clippy Lints and this guide? It seems like they can coexist. Does that mean the guide should just point to the current side with regard to actual lints, and maybe just include some examples to keep it interesting? Keeping both up to date seems like a maintenance nightmare unless its automated somehow. Or should the current ALL the Clippy lints somehow be incorporated into the book?
1. Related to the above, where should this guide be published since the `gh-pages` branch is already in use?
1. This PR doesn't currently change any existing content. Obviously that would make sense assuming the general structure and relocation is an acceptable approach.

---

Open Tasks for follow up PR:
- Set up CI/CD
- Split up Installation and Usage
- Add more content to Usage (and Installation) chapters
- Enhance CI chapter with more examples for different CIs

---

changelog: The first version of the *Clippy Book*
staring, *The Clippy Team*, *Bors* and *rustc*

Auto merge of #8944 - xFrednet:8877-append-doc-idents, r=Manishearth

List configuration values can now be extended instead of replaced

I've seen some `clippy.toml` files, that have a few additions to the default list of a configuration and then a copy of our default. The list will therefore not be updated, when we add new names. This change should make it simple for new users to append values instead of replacing them.

I'm uncertain if the documentation of the `".."` is apparent. Any suggestions are welcome. I've also check that the lint list displays the examples correctly.

<details>
<summary>Lint list screenshots</summary>



</details>

---

changelog: enhancement: [`doc_markdown`]: Users can now indicate, that the `doc-valid-idents` should extend the default and not replace it
changelog: enhancement: [`blacklisted-name`]: Users can now indicate, that the `blacklisted-names` should extend the default and not replace it

Closes: #8877
That's it. Have a fantastic weekend to everyone reading this. Here is a cookie :cookie:

Book: Improve release documentation

Make it clear for all code blocks in which repository they should be
run. Also make sure that the correct beta commit is fetched.

Book: Improve sync documentation

- Move doc about defining remotes to the front and use the defined
  remotes in the further documentation
- Don't recommend pushing to the remote repo directly
- Add some clarifying comments

Book: Improve chapter on CI

Recommend the -Dwarnings and --all-targets/--all-features more strongly.

Remove bit-rotty list of Clippy team members from CONTRIBUTING.md

Auto update lint count in Clippy book

Book: Add infrastructure description

Move parts of CONTRIBUTING.md to the book

Book: Add continuous integration description

Book: Add Proposals description

Book: Restructure Dev chapter

Group everything that has something to do with Clippy development under
the "Development" chapter, so that Clippy users can't get confused.

Book: Write lint group descriptions

This removes the empty separate files for the different groups and adds
a single file giving short explanations for each group and what to
expect from those groups.

Book: Split up and rewrite installation and usage

Book: Update GHA doc and remove GitLab placeholder

Remove Edition 2018 tests section from adding lints doc

The UI tests now use the latest edition by default. Testing on older
editions should almost never be necessary, so I don't see a need to
document this.

Reformat internal docs

This reformats all the internal docs, so that the md files use at most
80 characters per line. This is the usual formatting of md files. We
allow 120 chars per line in CI though.

Move syncing doc to book

Move internal documentation to book

Book: add a ci chapter

Initial commit for the Clippy Book

Fix `#[expect]` for `async_yields_async`

Auto merge of #8933 - DennisOSRM:needless_braces_range_literal, r=dswij

Add new lint [`needless_parens_on_range_literals`]

changelog: Adds a new lint [`needless_parens_on_range_literals`] to warn on needless braces on literals in a range statement

For example, the lint would catch

```log
error: needless parenthesis on range literals can be removed
  --> $DIR/needless_parens_on_range_literals.rs:8:13
   |
LL |     let _ = ('a')..=('z');
   |             ^^^^^ help: try: `'a'`
   |
   = note: `-D clippy::needless-parens-on-range-literals` implied by `-D warnings`
```

Fix `#[expect]` for `same_name_method`

Fix `#[expect]` for `needless_borrow`, `ref_binding_to_ref`

Auto merge of #8951 - mikerite:needless-late-init-20220605, r=giraffate

Some refactoring in `needless_late_init`

changelog: none

Update CHANGELOG.md

Fix names to use plural

Apply suggestions from code review

Co-authored-by: dswij <dharmasw@outlook.com>

Rerun cargo dev update_lints

Implement suggestion generation with snippet_with_applicability(.)

Implement support for implicit start and end

Move description into lint macro

Rewrite check to account for floating point literals

Fix misnomer braces -> parenthesis

Update clippy_lints/src/needless_braces_on_range_literal.rs

Co-authored-by: Alex <69764315+Serial-ATA@users.noreply.github.com>

Update clippy_lints/src/needless_braces_on_range_literal.rs

Co-authored-by: Alex <69764315+Serial-ATA@users.noreply.github.com>

Add new lint [`needless_braces_on_range_literal`]

Make [`create_dir`] example ignored

Improve lint doc consistency

Auto merge of #8941 - DevAccentor:for_loops_over_fallibles, r=llogiq

improve [`for_loops_over_fallibles`] to detect the usage of iter, iter_mut and into_iterator

fix #6762

detects code like
```rust
for _ in option.iter() {
    //..
}
```

changelog: Improve [`for_loops_over_fallibles`] to detect `for _ in option.iter() {}` or using `iter_mut()` or `into_iterator()`.

change based on review

needless_late_init refactoring

Remove the unneeded wrapping and unwrapping in suggestion creation.
Collecting to Option<Vec<_>> only returns None if one of the elements is
None and that is never the case here.

needless_late_init refactoring

Simplify the creation of suggestions by using `flat_map` instead of
`chain`. Note that the order of the suggestions is not important.

needless_late_init refactoring

Remove duplication in creating suggestions by first mapping assignments
to spans and then suggestions.

Actually get the correct `ParamEnv` in `derive_partial_eq_without_eq`

Don't lint `derive_partial_eq_without_eq` on private types

List configuration values can now be extended instead of replaced

Auto merge of #8930 - kyoto7250:issue_8920, r=Alexendoo

fix(manual_find_map and manual_filter_map): check clone method

close #8920

Added conditional branching when the clone method is used.

Thank you in advance.

---

changelog: check `clone()` and other variant preserving methods in [`manual_find_map`] and [`manual_filter_map`]

refactor: check copied and cloned

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

Rustup

r? `@ghost`

changelog: none

Bump nightly version -> 2022-06-04

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

Auto merge of #8843 - Serial-ATA:collect-renamed, r=xFrednet

Collect renamed lints

changelog: Display past names of renamed lints on Clippy's lint list

cc #7172

r? `@xFrednet`

improve for_loops_over_fallibles to detect the usage of iter, iter_mut and into_iterator

refactor: Add some methods

Auto merge of #8937 - Jarcho:merge_match_passes, r=llogiq

Merge various passes into `Matches`

changelog: None

Move `ManualMap` into `Matches` lint pass

Move `TryErr` into `Matches` lint pass

Merge `SignificantDropInScrutinee` into `Matches` lint pass

Move `MatchStrCaseMismatch` into `Matches` lint pass

Move `MatchOnVecItems` into `Matches` lint pass

Merge `ManualUnwrapOr` into `Matches` lint pass

Auto merge of #8934 - DevAccentor:as_underscore, r=Manishearth

add [`as_underscore`] lint

closes #8847

detect usage of `as _` and enforce the usage of explicit type like
```rust
fn foo(n: usize) {}
let n: u16 = 256;
foo(n as _);
```
will suggest to change to
```rust
fn foo(n: usize) {}
let n: u16 = 256;
foo(n as usize);
```

changelog: add [`as_underscore`] lint

Merge `CollapsibleMatch` into `Matches` lint pass

add as_underscore lint