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

nursery group -> style

Auto merge of #5350 - ThibsG:FixSingleBindingClosure, r=flip1995

Fix single binding closure

Fix the `match_single_binding` lint when triggered inside a closure.

Fixes: #5347
changelog: Improve suggestion for [`match_single_binding`]

Auto merge of #5361 - matthiaskrgr:rustup_37, r=flip1995

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

changelog: none

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

Auto merge of #5353 - flip1995:conf_doc, r=Manishearth

Fix documentation generation for configurable lints

In #5135, the configuration macro changed, but the documentation generation script wasn't updated. This PR catches up on this.

[Preview](https://flip1995.github.io/rust-clippy/master/index.html)

r? @Manishearth

changelog: Document configuration options of lints again.

Fix documentation generation for configurable lints

Fix single binding in closure

Update for PR feedback

Auto merge of #5323 - rabisg0:fix/5284, r=flip1995

Improvement: Don't show function body in needless_lifetimes

Changes the span on which the lint is reported to point to only the
function return type instead of the entire function body.
Fixes #5284

changelog: none

Auto merge of #5342 - flip1995:rollup-77s2pmt, r=flip1995

Rollup of 4 pull requests

Successful merges:

 - #5326 (rustup https://github.com/rust-lang/rust/pull/69838)
 - #5333 (rustup https://github.com/rust-lang/rust/pull/69189)
 - #5336 (rustup https://github.com/rust-lang/rust/pull/69920/)
 - #5341 (Rustup to rust-lang/rust#66131)

Failed merges:

r? @ghost

changelog: none

Get integration test binary from deps dir

cc rust-lang/cargo#7965

Rollup merge of #5341 - flip1995:rustup, r=flip1995

Rustup to rust-lang/rust#66131

changelog: none

Rollup merge of #5336 - matthiaskrgr:rustup_35, r=flip1995

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

changelog: none

Rollup merge of #5333 - matthiaskrgr:rustup_34, r=flip1995

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

rustups https://github.com/rust-lang/rust/pull/69189 which is part of https://github.com/rust-lang/rust/pull/70085
(at least I think this is the only pr that changes clippy test stdout)

changelog: none

Rollup merge of #5326 - matthiaskrgr:rustup_33, r=flip1995

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

changelog: none

Rustup to rust-lang/rust#66131

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

Improvement: Don't show function body in needless_lifetimes

Changes the span on which the lint is reported to point to only the
function return type instead of the entire function body.
Fixes #5284

Auto merge of #5335 - flip1995:changelog, r=Manishearth

Update changelog to 1.43.0 beta

In the beta changelog update, I accidentally used the commit of the 1.43.0 beta, instead of the 1.42.0 beta. I fixed this in this PR.

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

r? @Manishearth

changelog: none

Auto merge of #5329 - matthiaskrgr:int_arith_on_ref_5328, r=flip1995

integer_arithmetic: detect integer arithmetic on references.

changelog: integer_arithmetic fix false negatives with references on integers

Fixes #5328

tests: arithmetic: split into integer_arithmetic and float_arithmetic files.

Auto merge of #5331 - matthiaskrgr:lint_docs1, r=flip1995

clean up a few lint docs

changelog: none

Fix limit-stderr-files test

Split up checked_unwrap test further

Auto merge of #5330 - matthiaskrgr:newlint_expl, r=flip1995

new_lint.rs: encourage authors to write more detailed code samples in lint descriptions (linted as well as fixed code)

changelog: none

Update changelog to 1.43.0 beta

integer_arithmetic: detect integer arithmetic on references.

Also fixes the same for float_arithmetic.

changelog: integer_arithmetic,float_arithmetic: fix false negatives with references on integers

Fixes #5328

Merge remote-tracking branch 'upstream/rust-1.43.0' into backport_remerge