Rollup merge of #5409 - dtolnay:letunit, r=flip1995

Downgrade let_unit_value to pedantic

Given that the false positive in #1502 is marked E-hard and I don't have much hope of it getting fixed, I think it would be wise to disable this lint by default. I have had to suppress this lint in every substantial codebase (\>100k line) I have worked in. Any time this lint is being triggered, it's always the false positive case.

The motivation for this lint is documented as:

> A unit value cannot usefully be used anywhere. So binding one is kind of pointless.

with this example:

> ```rust
> let x = {
>     1;
> };
> ```

Sure, but the author would find this out via an unused_variable warning or from `x` not being the type that they need further down. If there ends up being a type error on `x`, clippy's advice isn't going to help get the code compiling because it can only run if the code already compiles.

changelog: Remove let_unit_value from default set of enabled lints

Rollup merge of #5406 - flip1995:update_lints_fix, r=flip1995

Fix update_lints

This fixes a bug in update_lints, where `internal` lints were not registered properly. This also cleans up some code. For example: The code generation functions no longer filter the lints the are given. This is now the task of the caller. This way, it is more obvious in the `replace_in_file` calls which lints will be included in which part of a file.

This also turns the lint modules private. There is no need for them to be public, since shared code should be in the utils module anyway.

And last but not least, this fixes the `register_lints` code generation, so also internal lints get registered.

changelog: none

Auto merge of #5429 - faern:use-assoc-int-float-consts, r=flip1995

Use assoc int and float consts instead of module level ones

changelog: Recommend primitive type associated constants instead of module level constants

In Rust 1.43 integer and float primitive types will have a number of new associated constants. For example `MAX`, `MIN` and a number of constants related to the machine representation of floats. https://github.com/rust-lang/rust/pull/68952

These new constants are preferred over the module level constants in `{core,std}::{f*, u*, i*}`. I have in the last few days made sure that the documentation in the main rust repository uses the new constants in every place I could find (https://github.com/rust-lang/rust/pull/69860, https://github.com/rust-lang/rust/pull/70782). So the next step is naturally to make the linter recommend the new constants as well.

This PR only changes two lints. There are more. But I did not want the PR to be too big. And since I have not contributed to clippy before it felt saner to start with a small PR so I see if there are any quirks. More will come later.

Use int assoc consts in MANUAL_SATURATING_ARITHMETIC

Use int assoc consts in checked_conversions lint

Use primitive type assoc consts in more tests

Use integer assoc consts in more lint example code

Don't import primitive type modules

Use assoc const NAN for zero_div_zero lint

Fix float cmp to use assoc fxx::EPSILON

Fix NAN comparison lint to use assoc NAN

Merge pull request #5434 from eddyb/rustup

rustup: update for the new Ty::walk interface.

The first commit fixes a portability bug in `setup-toolchain.sh`, while the second rewrites the handling of "trait impl methods" in `use_self` - even if `Ty::walk` could've still been used, it was IMO a misuse.

This could also serve as a PSA: *please* use `hir_ty_to_ty` instead of trying to compare `hir::Ty`s between themselves or against semantic `Ty`s. Its "quasi-deprecation" is 3 years old and doesn't really mean anything, just that it's currently uncached and that we should eventually querify it (either for a single HIR node, or for all of the nodes in an entire definition).

---

changelog: none

Format clippy_lints/src/let_underscore.rs

rustup: update for the new Ty::walk interface.

use_self: switch to hir_ty_to_ty.

Only /usr/bin/env is portable in shebangs.

Move matches test in matches module

Run update_lints

Make lint modules private

Don't filter lints in code generation functions

Build lint lists once and the reuse them to update files

Get rid of Lint::is_internal method

Clean up update_lints

Merge pull request #5407 from flip1995/rustup

Rustup to rust-lang/rust#70634

Downgrade let_unit_value to pedantic

Rustup to rust-lang/rust#70634

Auto merge of #5403 - farnz:patch-1, r=flip1995

Improve docs for option_option

Hint about using tri-state enums to replace legitimate uses of `Option<Option<_>>`

changelog: The docs for `option_option` now suggest using a tri-state enum

Update clippy_lints/src/types.rs

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

Update types.rs

Update types.rs

Improve docs for option_option

Hint about using tri-state enums to replace legitimate uses of `Option<Option<_>>`

Auto merge of #5349 - jpospychala:useless_rc, r=Manishearth

useless Rc<Rc<T>>, Rc<Box<T>>, Rc<&T>, Box<&T>

refers to  #2394

changelog: Add lints for Rc<Rc<T>> and Rc<Box<T>> and Rc<&T>, Box<&T>

this is based on top of another change #5310 so probably should go after that one.

useless Rc<Rc<T>>, Rc<Box<T>>, Rc<&T>, Box<&T>

Auto merge of #5402 - pmk21:allow-let-underscore-must-use, r=flip1995

Allow let_underscore_must_use to be ignored

changelog: none
Fixes #5366

Auto merge of #5401 - dtolnay:option, r=Manishearth

Downgrade option_option to pedantic

Based on a search of my work codebase (\>500k lines) for `Option<Option<`, it looks like a bunch of reasonable uses to me. The documented motivation for this lint is:

> an optional optional value is logically the same thing as an optional value but has an unneeded extra level of wrapping

which seems a bit bogus in practice. For example a typical usage would look like:

```rust
let mut host: Option<String> = None;
let mut port: Option<i32> = None;
let mut payload: Option<Option<String>> = None;

for each field {
    match field.name {
        "host" => host = Some(...),
        "port" => port = Some(...),
        "payload" => payload = Some(...), // can be null or string
        _ => return error,
    }
}

let host = host.ok_or(...)?;
let port = port.ok_or(...)?;
let payload = payload.ok_or(...)?;
do_thing(host, port, payload)
```

This lint seems to fit right in with the pedantic group; I don't think linting on occurrences of `Option<Option<T>>` by default is justified.

---

changelog: Remove option_option from default set of enabled lints

Allow let_underscore

Update option_option ui test

Test for ignoring let_underscore_must_use

Downgrade option_option to pedantic

Auto merge of #5400 - flip1995:rustup, r=Manishearth

Rustups

r? @Manishearth

changelog: none

Fix dogfood fallout

Rustup to rust-lang/rust#70632

Rustup to rust-lang/rust#70081

Rustup to rust-lang/rust#70627

Auto merge of #5398 - flip1995:deescalate, r=Manishearth

Stop updating the lint counter with every new lint

r? @Manishearth

This PR does two things:

1. Clean up the clippy_dev module a bit (first 3 commits; cc #5394 )
2. Make the counter in the README count in steps of 50 lints. Also use a `lazy_static` `Vec` for the lint list, so no counter is required there anymore.

changelog: none

Run update_lints

Don't use an exact lint counter anymore

Move update_lints logic to its own module

Make limit_stderr_length a subcommand

Define modules in lib.rs instead of main.rs

Auto merge of #5397 - pmk21:macro-single-match, r=flip1995

Avoid single_match lint in macro rules

changelog: none
fixes #5359

Small formatting change

Skip single_match lints in macro rules

Added test for single_match_else in macro

Added test for single_match in macro

Auto merge of #5293 - matthiaskrgr:macro_skip_lifetime, r=phansch

don't emit lifetime lint warnings for code inside macros.

Fixes #5283

changelog: Don't emit lifetime lint warnings for code inside macros.

don't emit lifetime lints for code inside macros.

Fixes #5283

Auto merge of #5294 - tmiasko:trait-ptr-cmp, r=Manishearth

Lint unnamed address comparisons

Functions and vtables have an insignificant address. Attempts to compare such addresses will lead to very surprising behaviour. For example: addresses of different functions could compare equal; two trait object pointers representing the same object and the same type could be unequal.

Lint against unnamed address comparisons to avoid issues like those in rust-lang/rust#69757 and rust-lang/rust#54685.

changelog: New lints: [`fn_address_comparisons`] [#5294](https://github.com/rust-lang/rust-clippy/pull/5294), [`vtable_address_comparisons`] [#5294](https://github.com/rust-lang/rust-clippy/pull/5294)

Lint unnamed address comparisons

Auto merge of #5365 - mgr-inz-rafal:issue4983_bool_updates, r=yaahc

Issue4983 bool updates

changelog: Check for bool inequality comparison that might be written more concisely

Fixes #4983

Update clippy_lints/src/needless_bool.rs

Auto merge of #5373 - flip1995:doc_release_backport, r=Manishearth

Document how to create releases and backports

This PR adds documentation on how to create Clippy releases and backports.

This PR also introduces a new backport/release process for Clippy:

- A beta branch is introduced: https://github.com/rust-lang/rust-clippy/tree/beta
- Backports are now done by PRs to the `beta` branch
- also commits to the beta branch will be deployed, so that the documentation page has a tab for the Clippy beta release.

This has two advantages:

1. The Clippy release process is closer to the Rust release process: stable releases are tagged, the beta release has its own branch
2. Backports to beta are easier, since they don't need as much coordination as before. No new branch has to be created for a backport. Just a PR to the beta branch, like in the Rust repo.¹

I tested the deployment here: https://github.com/flip1995/rust-clippy/runs/534465081 and it created this commit: https://github.com/flip1995/rust-clippy/commit/734503377e5ade55864ee674c010227a780cbf34, so it works.

r? @Manishearth your thoughts?

[Rendered (release.md)](https://github.com/flip1995/rust-clippy/blob/doc_release_backport/doc/release.md)
[Rendered (backport.md)](https://github.com/flip1995/rust-clippy/blob/doc_release_backport/doc/backport.md)

changelog: none

---

¹: For this, we may have to modify the CI, so that beta rustc is used to check PRs to beta (or we test it locally, and merge it without bors)

Auto merge of #5387 - jpospychala:useless_self_fp, r=yaahc

`unused_self` false positive

fixes #5351

Remove the for loop in `unused_self` so that lint enabled for one method doesn't trigger on another method.

changelog: Fix false positive in `unused_self` around lint gates on impl items

Fix typo in `release.md`

Co-Authored-By: Phil Hansch <dev@phansch.net>

Auto merge of #5392 - matthiaskrgr:rustup_40, r=phansch

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

changelog: none

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

Auto merge of #5380 - lzutao:deprecate-REPLACE_CONSTS, r=phansch

Deprecate REPLACE_CONSTS lint

Closes #5346
changelog: Deprecate `replace_consts` lint

Auto merge of #5390 - phansch:rustupp, r=matthiaskrgr

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

cc https://github.com/rust-lang/rust/pull/70449

changelog: none

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

`unused_self` false positive

Auto merge of #5382 - richkadel:gitattributes-macro, r=phansch

git attribute macros not allowed in submodules

This change simply moves the `rust` macro definition directly into the
attributes for `*.rs` files.

git commands that recurse from the rust toplevel tree into submodules
produce errors in clippy due to the fact that:

  "Custom macro attributes can be defined only in top-level
  gitattributes files"

For example, from the toplevel `rust` directory in a rustc development
build, try:

  $ git grep "search string" --recurse-submodules

Embedded within the actual results is the error message:

  [attr]rust text eol=lf whitespace=tab-in-indent,trailing-space,tabwidth=4
   not allowed: src/tools/clippy/.gitattributes:1

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 `cargo dev update_lints`
- [ ] Added lint documentation
- [ ] Run `cargo 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.

---

changelog: none

Auto merge of #5384 - matthiaskrgr:readme_4, r=matthiaskrgr

readme: move "how to run single lint" instructions to "Allowing/denying lints" section.

changelog: none

readme: move "how to run single lint" instructions to "Allowing/denying lints" section.

git attribute macros not allowed in submodules

This change simply moves the `rust` macro definition directly into the
attributes for `*.rs` files.

git commands that recurse from the rust toplevel tree into submodules
produce errors in clippy due to the fact that:

  "Custom macro attributes can be defined only in top-level
  gitattributes files"

For example, from the toplevel `rust` directory in a rustc development
build, try:

  $ git grep "search string" --recurse-submodules

Embedded within the actual results is the error message:

  [attr]rust text eol=lf whitespace=tab-in-indent,trailing-space,tabwidth=4
   not allowed: src/tools/clippy/.gitattributes:1

Auto merge of #5381 - lzutao:itertools, r=matthiaskrgr

Bump itertools

changelog: none

Deprecate REPLACE_CONSTS lint

Bump itertools

Auto merge of #5376 - flip1995:verbose_file_reads_restriction, r=matthiaskrgr

Move verbose_file_reads to restriction

cc #5368

Using `File::read` instead of  `fs::read_to_end` does make sense in multiple cases, so this lint is rather restriction, than complexity

changelog: Move [`verbose_file_reads`] to restriction

Auto merge of #5377 - matthiaskrgr:rustup_38, r=flip1995

rustup

changelog: none

remove redundant import

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

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

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

Move verbose_file_reads to restriction

Also deploy from the beta branch

Document how to backport changes to beta

Document how to create a Clippy release

Auto merge of #5334 - flip1995:backport_remerge, r=flip1995

Backport remerge

blocked on landing the backport

cc @ehuss

changelog: none

Auto merge of #5372 - matthiaskrgr:rpc_nursery, r=flip1995

move redundant_pub_crate to nursery

cc #5369
changelog: none

move redundant_pub_crate to nursery

cc #5369

Auto merge of #5371 - matthiaskrgr:rdm, r=phansch

readme: explain how to run only a single lint on a codebase

changelog: none

readme: explain how to run only a single lint on a codebase

Auto merge of #5370 - phansch:matches, r=matthiaskrgr

Remove dependency on `matches` crate

The std equivalent works exactly the same.

changelog: none

Remove dependency on `matches` crate

The std equivalent works exactly the same.

Auto merge of #5364 - flip1995:useless_transmute_quarantine, r=Manishearth

Move useless_transmute to nursery

cc #5343

@rust-lang/clippy anyone against moving this to nursery?

changelog: Move [`useless_transmute`] to nursery

Auto merge of #5319 - 1tgr:master, r=flip1995

Lint for `pub(crate)` items that are not crate visible due to the visibility of the module that contains them

changelog: Add `redundant_pub_crate` lint

Closes #5274.

Code clean-up and formatting

Provide appropriate suggestion

Move useless_transmute to nursery

Initial lint without suggestion