Auto merge of #5952 - 1c3t3a:1c3t3a-dev-5819, r=Manishearth

Added a lint which corrects expressions like (a - b) < f32::EPSILON, according to #5819

Fixes #5819
changelog: none

Auto merge of #5947 - ebroto:rustup_doc_improvements, r=flip1995

Improve documentation related to the sync process

[Rendered](https://github.com/ebroto/rust-clippy/blob/rustup_doc_improvements/CONTRIBUTING.md)

r? @flip1995

changelog: none

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

Rustup

r? @ghost

changelog: none

Added F32::EPSILON and F64::EPSILON to paths.rs

Auto merge of #5920 - giraffate:fix_fp_for_redundant_closure_call, r=mikerite

Fix FP for `redundant_closure_call`

Fix #5916

changelog: Fix FP for `redundant_closure_call` when called in function body

Unbreak the clippy test

Auto merge of #5935 - matthiaskrgr:unstable_sort_ty_name, r=flip1995

stable_sort_primitive: print the type that is being sorted in the lint message

changelog: stable_sort_primitive: print the type that is being sorted in the lint message

stable_sort_primitive: print the type that is being sorted in the lint message

Added a lint which corrects expressions like (a - b) < f32::EPSILON

Avoid period in lint message according to convention

Fix FP for `redundant_closure_call`

Visit the nested things like function body when checking closure call.

Apply suggestions from PR review

Auto merge of #5948 - ebroto:5944_repeat_ice, r=Manishearth

Fix ICE in `repeat_once`

changelog: Fix ICE in [`repeat_once`]

Fixes #5944

Auto merge of #5930 - ebroto:dev_newlint_fix_lt, r=matthiaskrgr

Fix cargo dev new_lint for late lint passes

Since https://github.com/rust-lang/rust-clippy/commit/30c046ede47ea45a772d2656b0365a4807263bca, `LateLintPass` has only one lifetime parameter.

I'm not sure how to easily test this, I think adding this kind of tests would be probably part of #5394

changelog: none

Fix ICE in `repeat_once` lint

Improve documentation related to the sync process

Fix cargo dev new_lint for late lint passes

Auto merge of #5941 - ThibsG:InlineInCopyPassByRef, r=yaahc

Don't lint if it has always inline attribute

Don't trigger the lint `trivially_copy_pass_by_ref` if it has `#[inline(always)]` attribute.

Note: I am not particularly familiar with `inline` impacts, so I implemented this the way that if only `#[inline]` attribute is here (without `always`), the lint will still trigger. Also, it will still trigger if it has `#[inline(never)]`.
Just tell me if it sounds too much conservative.

Fixes: #5876
changelog: none

Auto merge of #5929 - stchris:issues/5917, r=ebroto

Widen understanding of prelude import

Prelude imports are exempt from wildcard import warnings. Until now only
imports of the form

```
use ...::prelude::*;
```

were considered. This change makes it so that the segment `prelude` can
show up anywhere, for instance:

```
use ...::prelude::v1::*;
```

Fixes #5917

changelog: Allow `prelude` to appear in any segment of the import path in [`wildcard_imports`]

Use more elegant way to check for prelude string

Run sh tests/ui/update-all-references.sh

Widen understanding of prelude import

Prelude imports are exempt from wildcard import warnings. Until now only
imports of the form

```
use ...::prelude::*;
```

were considered. This change makes it so that the segment `prelude` can
show up anywhere, for instance:

```
use ...::prelude::v1::*;
```

Fixes #5917

Don't lint if it has always inline attribute

Auto merge of #5934 - ThibsG:WrongSelfTestCase4037, r=ebroto

Add async test case for FP in `wrong_self_convention` lint

Closes: #4037
changelog: none

r? @ebroto

Add async test case for `wrong_self_convention` lint

Auto merge of #5926 - giraffate:improve_lint_message_in_to_string_in_display, r=yaahc

Improve lint message in `to_string_in_display`

This is a follow-up of https://github.com/rust-lang/rust-clippy/pull/5831.

changelog: none

Auto merge of #5933 - mikerite:fix-5927, r=matthiaskrgr

Fix false negative in `option_as_ref_deref`

Closes #5927

changelog: Fix false negative in `option_as_ref_deref`

Auto merge of #5938 - ebroto:sync-from-rust, r=ebroto

Rustup

r? @ghost

changelog: none

run cargo dev fmt

Merge remote-tracking branch 'upstream/master' into sync-from-rust

Fix false negative in `option_as_ref_deref`

Auto merge of #5928 - mikerite:fix-5924, r=ebroto

Fix false positive in `PRECEDENCE` lint

Extend the lint to handle chains of methods combined with unary negation.

Closes #5924

changelog: Fix false negative in `PRECEDENCE` lint

Auto merge of #5919 - ebroto:5747_from_bits_in_const, r=matthiaskrgr

transmute: avoid suggesting from/to bits in const

changelog: Avoid suggesting from/to bits in const contexts in [`transmute_int_to_float`]

Fixes #5747

Capture tokens for Pat used in macro_rules! argument

This extends PR #73293 to handle patterns (Pat). Unlike expressions,
patterns do not support custom attributes, so we only need to capture
tokens during macro_rules! argument parsing.

Fix false positive in `PRECEDENCE` lint

Extend the lint to handle chains of methods combined with unary negation.

Closes #5924

Improve lint message in `to_string_in_display`

no from/to bits in const: add tests cases for f64

transmute: avoid suggesting from/to bits in const

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

Rustup

r? @ghost

changelog: none

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

Auto merge of #75120 - JulianKnodt:rm_reps, r=oli-obk

rust_ast::ast => rustc_ast

Rework of #71199 which is a rework #70621

Still working on this but just made the PR to track progress

r? @Dylan-DPC

rust_ast::ast => rustc_ast

Auto merge of #5908 - giraffate:fix_fp_for_same_item_push, r=flip1995

Fix FP for `same_item_push`

Fixes https://github.com/rust-lang/rust-clippy/issues/5902

changelog: Fix FP for `same_item_push` where the pushed variable is mutated.

clippy: support `QPath::LangItem`

This commit updates clippy with the introduction of `QPath::LangItem` so
that it still compiles.

Signed-off-by: David Wood <david@davidtw.co>

Auto merge of #5915 - matthiaskrgr:deps_, r=Manishearth

deps: bump cargo_metadata and semver

cargo_metadata 0.9.1 -> 0.11.1
semver 0.9.0 -> 0.10.0

changelog: none

deps: bump cargo_metadata and semver

cargo_metadata 0.9.1 -> 0.11.1
semver 0.9.0 -> 0.10.0

Improve code style

Auto merge of #5894 - tmiasko:self-assignment, r=Manishearth

Warn about explicit self-assignment

Warn about assignments where left-hand side place expression is the same
as right-hand side value expression. For example, warn about assignment in:

```rust
pub struct Event {
    id: usize,
    x: i32,
    y: i32,
}

pub fn copy_position(a: &mut Event, b: &Event) {
    a.x = b.x;
    a.y = a.y;
}
```

changelog: New lint `self_assignment`, checks for explicit self-assignments.

Warn about explicit self-assignment

Warn about assignments where left-hand side place expression is the same
as right-hand side value expression. For example, warn about assignment in:

```rust
pub struct Event {
    id: usize,
    x: i32,
    y: i32,
}

pub fn copy_position(a: &mut Event, b: &Event) {
    a.x = b.x;
    a.y = a.y;
}
```

Expresions with Assign / AssignOp have side effects

Rename SpanlessEq::ignore_fn to deny_side_effects

No functional changes intended.

Introduce function for comparing expression values

Introduce `eq_expr_value(cx, a, b)` as a shortcut for
`SpanlessEq::new(cx).ignore_fn().eq_expr(cx, a, b)`.

No functional changes intended.

Auto merge of #5720 - bugadani:new-lint, r=flip1995,yaahc

Add unnecessary lazy evaluation lint

changelog: Add [`unnecessary_lazy_evaluations`] lint that checks for usages of `unwrap_or_else` and similar functions that can be simplified.

Closes #5715

Rename lint to use plural form

Fix missed rename

Auto merge of #5913 - HactarCE:patch-1, r=flip1995

Fix typo in description of unnecessary_mut_passed

changelog: Correct "immutable reference" to "mutable reference" in description of `unnecessary_mut_passed`

Fix dogfooding test errors

run cargo dev fmt

Cleanup, explain return value

Moved to submodule, don't trigger if map_unwrap_or does

Fix typo in description of unnecessary_mut_passed

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

Fix FP for `same_item_push`

Don't emit a lint when `pushed_item` was declared as mutable variable.

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