Fix missed rename

Fix dogfooding test errors

run cargo dev fmt

Cleanup, explain return value

Moved to submodule, don't trigger if map_unwrap_or does

Apply suggested change

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

Apply suggested change

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

Removed the extra lifetime parameter

Catch function calls in argument lists, add tests that tuples don't get linted

Run cargo dev update_lints

Also simplify if the closure body is an index expression

Add note to `or_fun_call`, list checked methods

Rename the changelog footnote as well

Implement new lint

Auto merge of #5725 - montrivo:should-impl-trait, r=flip1995

should_impl_trait - ignore methods with lifetime params

Fixes: #5617
changelog: don't lint should_implement_trait when an `Iterator::next` case has explicit parameters

Auto merge of #5907 - wiomoc:feature/useless-vec-max-size, r=flip1995

appreciative too_large_for_stack in useless `vec!`

Fixes: #5847
changelog: Add `too_large_for_stack ` configuration option for `USELESS_VEC`

Auto merge of #5903 - jrqc:needless_return, r=ebroto,flip1995

Needless return

Fixes #5858
changelog: fix false positive [`needless_return`]

Auto merge of #5881 - wiomoc:feature/single-char-push_str, r=ebroto,flip1995

Lint `push_str` with a single-character string literal

Fixes #5875
changelog:  `* [single_char_push_str]`

Auto merge of #5831 - chansuke:to_string_in_display, r=flip1995

Don't use `to_string` in impl Display

fixes #3876

this PR is derived from [Toxyxer's implementation](https://github.com/rust-lang/rust-clippy/pull/5574).
changelog: add [`to_string_in_display`] lint

Auto merge of #5912 - ebroto:needless_doctest_main_improvements, r=Manishearth,flip1995

Parse doctests in needless_doctest_main

This switches from text-based search to running the parser to avoid false positives. Inspired by how [rustdoc](https://github.com/rust-lang/rust/blob/3f3250500fe152b5759c21453ba9a9129808d0d8/src/librustdoc/test.rs#L366) handles this and by #4729.

cc @llogiq

changelog: Fix multiple false positives in [`needless_doctest_main`].

Fixes #5879
Fixes #4906
Fixes #5103
Fixes #4698

Improve needless_doctest_main by using the parser

early return removed

cargo dev update_lints

Modifications according to the code review

Known problems changed

Borrow checker added

Visitor added

Make needless_return a late lint pass

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

Rustup

r? @ghost

changelog: none

elide lifetimes

Merge lint with `single_char_pattern`

Lint `push_str` with a single-character string literal

Fixes #5875

Don't use `to_string` in impl Display

appreciative too_large_for_stack in useless `vec!`

Fixes: #5847

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