Improve needless_doctest_main by using the parser

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

Rustup

r? @ghost

changelog: none

Auto merge of #5905 - ErichDonGubler:patch-1, r=flip1995

docs: typo in `temporary_cstring_as_ptr`: s/point/&s

changelog: none

docs: typo in `temporary_cstring_as_ptr`: s/point/&s

merge `as_local_hir_id` with `local_def_id_to_hir_id`

Auto merge of #5901 - giraffate:add_reference_to_rustc_dev_guide, r=matthiaskrgr

Add reference to rustc-dev-guide about lint message

I think it would be better to add lint message convention to documentation. I referred to https://github.com/rust-lang/rust-clippy/pull/5888 and https://github.com/rust-lang/rust-clippy/pull/5893.

changelog: none

Auto merge of #5899 - JarredAllen:rc-box-suggestion, r=yaahc

Change Rc<Box<T>> recommendation to be Rc<T> instead of Box<T>

Fixes #5722

changelog: Suggest `Rc<Box<T>>` -> `Rc<T>` in [`redundant_allocation`] lint

Auto merge of #5900 - ThibsG:ParensUselessConversion4750, r=phansch

Fix: keep parenthesis for suggestion in `useless_conversion` lint

Note: this lint was previously named `identity_conversion`.

fixes: #4750

changelog: fix parenthesis for `useless_conversion` lint suggestion

Add reference to rustc-dev-guide about lint message

Replace panics with early returns

Fix: keep parenthesis for suggestion in `useless_conversion` lint

Auto merge of #5897 - matthiaskrgr:no_clone_tts, r=flip1995

write.rs: don't clone TokenStream

changelog: none

Change Rc<Box<T>> recommendation to be Rc<T> instead of Box<T>

Auto merge of #5884 - Ryan1729:patch-1, r=flip1995

Add the other overloadable operations to suspicious_arithmetic_impl

In #2268 I idly mused that the other user-overloadable operations could be added to this lint. Knowing that the lint was arguably incomplete was gnawing at the back of my mind, so I figured that I might as well make this PR, particularly given the change needed was so small.

changelog: Start warning on suspicious implementations of the `BitAnd`, `BitOr`, `BitXor`, `Rem`, `Shl`, and `Shr` traits.

run tests/ui/update-references.sh to update 'suspicious_arithmetic_impl.rs'

Auto merge of #5893 - matthiaskrgr:lint_msg, r=yaahc

fix remaining lint messages

r? @yaahc

changelog:  make remaining lint messages adhere to rustc dev guide lint message convention.

add tests for Rem, BitAnd, BitOr, BitXor, Shl, and Shr

trailing comma

Should have actually ran rustfmt on it, rather than attempting to fix it manually

formatting

Add the other overloadable operations to suspicious_arithmetic_impl

In #2268 I idly mused that the other user-overloadable operations could be added to this lint. Knowing that the lint was arguably incomplete was gnawing at the back of my mind, so I figured that I might as well make this PR, particularly given the change needed was so small.

Auto merge of #5885 - Ryan1729:patch-2, r=flip1995

Add example of false positive to PTR_ARG docs.

Addresses #214

changelog: Add example of false positive to `ptr_arg` docs.

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

Rustup

r? @ghost

changelog: none

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

fix typo pointed out in review comment

Co-authored-by: Philipp Krones <hello@philkrones.com>

write.rs: don't clone TokenStream

Auto merge of #75405 - flip1995:clippyup, r=Manishearth

Update Clippy

Biweekly Clippy update (2 days late, since I wanted to wait for https://github.com/rust-lang/rust/pull/75098)

r? @Manishearth

Merge commit '09bd400243ed6f7059fedc0c1623aae3792521d6' into clippyup

if_not_else: make lint adhere to lint message convention

if_let_some_result: make lint adhere to lint message convention

enum-variant-names: make lint adhere to lint message convention

enum-clike-unportable-variant: tweak message a bit (Clike -> C-like)

duration-subsec: make lint adhere to lint message convention

pub-enum-variant-names: make lint adhere to lint message convention

multiple_inherent_impl: make lint adhere to lint message convention

implicit-saturating-sub: make lint adhere to lint message convention

int_plus_one: make lint adhere to lint message convention

iter-next-slice: make lint adhere to lint message convention

mutex-atomic: make lint adhere to lint message convention

map_clone: make lint adhere to lint message convention

stable-sort-primitive: make lint adhere to lint message convention

Auto merge of #5892 - matthiaskrgr:redundant_mut, r=flip1995

unnecessary-mut-passed: make lint message say if fn is a function or a method

changelog: refine "unnecessary-mut-passed" lint message

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

Rustup

r? @ghost

Sync back rust-lang/rust#75098

changelog: none

Fix sync fallout

unnecessary-mut-passed: make lint message say if fn is a function or a method.

Rollup merge of #75226 - pnadon:miri-undef-uninit, r=RalfJung

Miri: Renamed "undef" to "uninit"

Renamed remaining references to "undef" to "uninit" when referring to Miri.

Impacted directories are:

- `src/librustc_codegen_llvm/consts.rs`
- `src/librustc_middle/mir/interpret/`
- `src/librustc_middle/ty/print/pretty.rs`
- `src/librustc_mir/`
- `src/tools/clippy/clippy_lints/src/consts.rs`

Upon building Miri based on the new changes it was verified that no changes needed to be made with the Miri project.

Related issue #71193

Rollup merge of #75098 - Ryan1729:clippy-pointer-cast-lint-experiment, r=oli-obk

Clippy pointer cast lint experiment

This PR is an experiment about exposing more parts of `rustc_typeck` for use in `clippy`. In particular, the code that checks where a cast is valid or not was exposed, which necessitated exposing [`FnCtxt`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc_typeck/check/struct.FnCtxt.html), and figuring out how to create an instance of that type inside `clippy`.

This was prompted by [this clippy issue](https://github.com/rust-lang/rust-clippy/issues/2064).

r? @oli-obk

Auto merge of #5888 - matthiaskrgr:lints, r=yaahc

make a bunch of lints texts adhere to rustc dev guide

According to the rustc-dev guide: "The text should be matter of fact and avoid capitalization and periods, unless multiple sentences are needed"

changelog: make some lint output adhere to the rustc-dev guide

update test stderr

Auto merge of #5889 - ebroto:5886_or_fun_call_const_0_args, r=Manishearth

Avoid or_fun_call for const_fn with no args

Based on #5682 by @lzutao

This avoids a subset of false positives, specifically those related to `const fn`s that take no arguments.
For the rest, a much more involved fix would be needed, see https://github.com/rust-lang/rust-clippy/pull/5682#issuecomment-638681210.

So this does *not* solve #5658

changelog: Avoid triggering [`or_fun_call`] with `const fn`s that take no arguments.

Fixes #5886

run cargo dev update-lints

Update clippy_lints/src/unwrap.rs

Co-authored-by: Jane Lusby <jlusby42@gmail.com>

Update clippy_lints/src/neg_cmp_op_on_partial_ord.rs

Co-authored-by: Jane Lusby <jlusby42@gmail.com>

neg-cmp-op-on-partial-ord: make lint adhere to lint message convention

neg-multiply: make lint adhere to lint message convention

overflow-check-conditional: make lint adhere to lint message convention

path-buf-push-overwrite: make lint adhere to lint message convention

range-zip-with-len: make lint adhere to lint message convention

redundant-static-lifetimes: make lint adhere to lint message convention

also change "deprecated-attribute" message

suspicious-arithmetic-impl: make lint adhere to lint message convention

unknown: make lint adhere to lint message convention

ref_in_deref: make lint adhere to lint message convention

unneeded-field-pattern: make lint adhere to lint message convention

empty-liner-after-outer-attr: make lint adhere to lint message convention

drop_bounds: make lint adhere to lint message convention

double-parens: make lint adhere to lint message convention and do minor refactoring

double-comparisons: make lint adhere to lint message convention

default-trait-access: make lint adhere to lint message convention

cmp_null: make lint adhere to lint message convention

unnecessary_unwrap, panicking_unwrap: make lints adhere to lint message convention

checked-conversions: make lint adhere to lint message convention

Avoid or_fun_call for const_fn with no args

naive_bytecount: make lint adhere to lint message convention

builtin-type-shadow: make lint adhere to lint message convention

bool-comparison: make lint adhere to lint message convention

too-many-lines: make lint adhere to lint message convention

Auto merge of #5887 - robojumper:patch-1, r=flip1995

Fix CHANGELOG's commit range links

Two most recent links linked to the wrong range

changelog: none

Fix CHANGELOG's commit range links

Two most recent links linked to the wrong range

changelog: none

formatting

Add example of false positive to PTR_ARG docs.

Fixes #214

Auto merge of #5883 - flip1995:rollup-x9mftxe, r=flip1995

Rollup of 5 pull requests

Successful merges:

 - #5825 (Add the new lint `same_item_push`)
 - #5869 (New lint against `Self` as an arbitrary self type)
 - #5870 (enable #[allow(clippy::unsafe_derive_deserialize)])
 - #5871 (Lint .min(x).max(y) with x < y)
 - #5874 (Make the docs clearer for new contributors)

Failed merges:

r? @ghost

changelog: rollup

Rollup merge of #5874 - camelid:patch-1, r=flip1995

Make the docs clearer for new contributors

It confused me before, so I made it extra obvious that you need to run
a script to set up your toolchain before you can build Clippy.

I also added a note so that new contributors aren't confused when
Clippy doesn't build as a result of a change in rustc's internals.

changelog: make `CONTRIBUTING.md` clearer for new contributors

Rollup merge of #5871 - wiomoc:feature/methodcall-minmax, r=flip1995

Lint .min(x).max(y) with x < y

Fixes  #5854

changelog: Also lint `ord.min(a).max(b)`, where `a < b` in [`min_max`] lint

Rollup merge of #5870 - ebroto:5789_allow_unsafe_derive_deserialize, r=flip1995

enable #[allow(clippy::unsafe_derive_deserialize)]

Before this change this lint could not be allowed as the code we are checking is automatically generated.

changelog: Enable using the `allow` attribute on top of an ADT linted by [`unsafe_derive_deserialize`].

Fixes: #5789

Rollup merge of #5869 - wiomoc:feature/implicit-self, r=ebroto,flip1995

New lint against `Self` as an arbitrary self type

Fixes #5861

changelog: * [`needless_arbitrary_self_type`] [#5869](https://github.com/rust-lang/rust-clippy/pull/5869)

Rollup merge of #5825 - giraffate:same_item_push, r=Manishearth

Add the new lint `same_item_push`

changelog: Add the new lint `same_item_push`

Fixed #4078. As I said in https://github.com/rust-lang/rust-clippy/issues/4078#issuecomment-658184195, I referrerd to https://github.com/rust-lang/rust-clippy/pull/4647.

Auto merge of #5882 - dima74:ra_setup-prevent-compile-rustc, r=Manishearth

Prevent compile parts of rustc when using `cargo dev ra-setup`

Currently after running `cargo dev ra-setup` the following lines are added to `Cargo.toml`:

```toml
[target]
rustc_data_structures = { path = ".../rust/src/librustc_data_structures" }
rustc_driver = { path = ".../rust/src/librustc_driver" }
rustc_errors = { path = ".../rust/src/librustc_errors" }
rustc_interface = { path = ".../rust/src/librustc_interface" }
rustc_middle = { path = ".../rust/src/librustc_middle" }
```

This pull request adds dependencies for `rustc` crates under `cfg(NOT_A_PLATFORM)`, thus preventing them from compiling together with clippy:

```toml
[target.'cfg(NOT_A_PLATFORM)'.dependencies]
rustc_data_structures = { path = ".../rust/src/librustc_data_structures" }
rustc_driver = { path = ".../rust/src/librustc_driver" }
rustc_errors = { path = ".../rust/src/librustc_errors" }
rustc_interface = { path = ".../rust/src/librustc_interface" }
rustc_middle = { path = ".../rust/src/librustc_middle" }
```

---

This approach was [originally proposed for IntelliJ Rust](https://github.com/intellij-rust/intellij-rust/issues/1618#issuecomment-459098749), and looks like it works for rust-analyzer too.

changelog: none

Prevent compile parts of rustc when using `cargo dev ra-setup`

add allow unused_unsafe and allow dead_code

fix unary minus on usize and unused variable errors in .fixed file

add a test example of where transmutes_expressible_as_ptr_casts should not suggest anything

update stderr for transmutes_expressible_as_ptr_casts

Auto merge of #5877 - ebroto:5872_loops_ice, r=Manishearth

Fix ICE in `loops` module

changelog: Fix ICE related to `needless_collect` when a call to `iter()` was not present.

I went for restoring the old suggestion of `next().is_some()` over `get(0).is_some()` given that `iter()` is not necessarily present (could be e.g. `into_iter()` or `iter_mut()`)  and that the old suggestion could change semantics, e.g. a call to `filter()` could be present between `iter()` and the collect part.

Fixes #5872

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

Rustup

r? @ghost

changelog: none

Run fmt

Fix ICE in `loops` module

Miri: Renamed "undef" to "uninit"

Renamed remaining references to "undef" to "uninit" when referring to Miri.

Impacted directories are:

- src/librustc_codegen_llvm/consts.rs
- src/librustc_middle/mir/interpret/
- src/librustc_middle/ty/print/pretty.rs
- src/librustc_mir/
- src/tools/clippy/clippy_lints/src/consts.rs

Upon building Miri based on the new changes it was verified that no changes needed to be made with the Miri project.

Related issue #71193

Eliminate the `SessionGlobals` from `librustc_ast`.

By moving `{known,used}_attrs` from `SessionGlobals` to `Session`. This
means they are accessed via the `Session`, rather than via TLS. A few
`Attr` methods and `librustc_ast` functions are now methods of
`Session`.

All of this required passing a `Session` to lots of functions that didn't
already have one. Some of these functions also had arguments removed, because
those arguments could be accessed directly via the `Session` argument.

`contains_feature_attr()` was dead, and is removed.

Some functions were moved from `librustc_ast` elsewhere because they now need
to access `Session`, which isn't available in that crate.
- `entry_point_type()` --> `librustc_builtin_macros`
- `global_allocator_spans()` --> `librustc_metadata`
- `is_proc_macro_attr()` --> `Session`

Make the docs clearer for new contributors

* Add an easy-to-see note at the top of `CONTRIBUTING.md` that points
  new contributors to the Basics docs
* Add a note about compiler errors as a result of internals changes
  that break Clippy

check impl Ord / is_float