Fix clippy error

Add STD_FS_CREATE_DIR into paths

Add a new lint to prevent `create_dir` from being used

Auto merge of #5971 - giraffate:fix_fp_in_to_string_in_display, r=ebroto

Fix FP in `to_string_in_display`

Don't emit a lint when `.to_string()` on anything that is not `self`

Fix #5967

changelog: Fix FP in `to_string_in_display` when calling `.to_string()` on anything that is not `self`

Fix FP in `to_string_in_display`

Don't emit a lint when `.to_string()` on anything that is not `self`

Auto merge of #5957 - xvschneider:AddUnwrapInsideResultLint, r=yaahc

Adding new lint to prevent usage of 'unwrap' inside functions that re…

### Change
Adding a new lint that will emit a warning when using "unwrap" or "expect" inside a function that returns result.
### Motivation
These functions promote recoverable errors to non-recoverable errors which may be undesirable in code bases which wish to avoid panics.
### Test plan
Running:
`TESTNAME=unwrap_in_result cargo uitest
`---

changelog: none

Add new lint to prevent usage of unwrap in fns that return result

Auto merge of #5968 - rail-rain:fix_use_snippet_with_applicability, r=flip1995

Fix the wrong use of `snippet_with_applicability`

For the context, please see https://github.com/rust-lang/rust-clippy/issues/5822#issuecomment-680749728 and https://github.com/rust-lang/rust-clippy/issues/5822#issuecomment-680783381.

---

changelog: none

Fix the wrong use of `snippet_with_applicability`

This includes a workaround of the issue #5822,
the cause of this little mistake.

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

Corrects the float_equality_without_abs lint

Fixes an issue in the `float_equality_without_abs` lint. The lint suggestion was configured in a way that it lints the whole error and not just the subtraction part. In the current configuration the lint would suggest to change the expression in a wrong way, e.g.
```rust
let _ = (a - b) < f32::EPSILON; // before
let _ = (a - b).abs(); // after
```
This was dicovered by @flip1995. (See discussion of PR #5952).

Also the suggestion is now formatted via `utils::sugg`.
changelog: none

Changed the location of the suggestion as well as the way the suggestion is assembled

Auto merge of #5946 - mikerite:fix-5729, r=flip1995

Fix `let_and_return` bad suggestion

Add a cast to the suggestion when the return expression has adjustments.
These adjustments are lost when the suggestion is applied.

This is similar to the problem in issue #4437.

Closes #5729

changelog: Fix `let_and_return` bad suggestion

Auto merge of #5949 - rail-rain:fix_fp_borrow_interior_mutable_const, r=oli-obk

Fix fp in `borrow_interior_mutable_const`

fixes #5796

changelog: fix false positive in `borrow_interior_mutable_const` when referencing a field behind a pointer.

Auto merge of #5964 - matsujika:matsujika-patch-1, r=ebroto

Fix typo

changelog: none

Fix typo

Auto merge of #5951 - ThibsG:FixMacroCloneOnRefPtr2076, r=ebroto

Fix incorrect suggestion when `clone_on_ref_ptr` is triggered in macros

In the lint `clone_on_ref_ptr`, if the `span` is in a macro, don't expand it for suggestion.

Fixes: #2076
changelog: none

r? @ebroto

Update clippy_lints/src/methods/mod.rs

Co-authored-by: Eduardo Broto <ebroto@tutanota.com>

Auto merge of #5961 - scottmcm:master, r=Manishearth

Re-enable len_zero for ranges now that `is_empty` is stable on them

Fixes #5956

Completed stabilization PR: https://github.com/rust-lang/rust/pull/75132

changelog: len_zero: re-enable linting ranges now that range_is_empty is stable

Re-enable len_zero for ranges now that `is_empty` is stable on them

Auto merge of #5958 - mikerite:fix-fmt-20200825, r=flip1995

Various fmt test improvements

Various fmt test improvements.

changelog: none

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

Fix typo

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

Fix intermittent build error

The fmt test will cause clippy dev to be compiled and run. The clippy dev
dependencies are currently stored at `target/debug/deps` when this happens.
This location sometimes causes conflicts with ui tests which result in the
build error "thread 'compile_test' panicked at 'Found multiple rlibs for crate
`regex` ...".

This commit forces the clippy_dev dependencies to be stored under
`clippy_dev/target/` which seems to resolve this problem.

Simplify fmt test by using the clippy dev alias

Fix incorrect comment

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

Fix incorrect suggestion when `clone_on_ref_ptr` is triggered in macros

typo

Co-authored-by: Thibaud <ThibsG@users.noreply.github.com>

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 fp in `borrow_interior_mutable_const`

Fix false positive when referencing a field behind a pointer.

Fix ICE in `repeat_once` lint

Improve documentation related to the sync process

Fix cargo dev new_lint for late lint passes

Fix `let_and_return` bad suggestion

Add a cast to the suggestion when the return expression has adjustments.
These adjustments are lost when the suggestion is applied.

This is similar to the problem in issue #4437.

Closes #5729

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`