Simplify `clippy::default_trait_access`

Remove repeated matching on the same QPath.

Auto merge of #5994 - taiki-e:useless_attribute, r=phansch

useless_attribute: Permit wildcard_imports and enum_glob_use

Fixes #5918

changelog: `useless_attribute`: Permit `wildcard_imports` and `enum_glob_use` on `use` items

Auto merge of #6003 - mikerite:fix-tests-20200903, r=phansch

Update ui stderr with improved rustc output

Related rust pull request: rust-lang/rust#76160

changelog: none

Update ui stderr with improved rustc output

Related rust pull request: rust-lang/rust#76160

Auto merge of #5996 - Koxiaet:master, r=ebroto

Allow GraphQL in docs without backticks

changelog: Allow "GraphQL" in [`doc_markdown`] without backticks.

Add tests for allowed non-backticked identifiers in doc

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

Sync from rust

r? @ghost

changelog: none

Run cargo dev fmt

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

Auto merge of #5999 - rail-rain:fix_fp_transmute_ptr_to_ptr_in_consts, r=ebroto

Fix a fp in `transmute_ptr_to_ptr`

fixes #5959

changelog: Fix a false positive in `transmute_ptr_to_ptr` that the lint fires when `transmute` is used to cast a reference in const contexts although dereferencing raw pointers in consts is unstable.

Auto merge of #5993 - taiki-e:default_trait_access, r=phansch

default_trait_access: Fix wrong suggestion

https://github.com/rust-lang/rust-clippy/issues/5975#issuecomment-683751131
> I think the underlying problem is clippy suggests code with complete parameters, not clippy triggers this lint even for complex types. AFAIK, If code compiles with `Default::default`, it doesn't need to specify any parameters, as type inference is working. (So, in this case, `default_trait_access` should suggest `RefCell::default`.)

Fixes #5975 Fixes #5990

changelog: `default_trait_access`: fixed wrong suggestion

Allow GraphQL in doc without backticks

useless_attribute: Permit wildcard_imports and enum_glob_use

Fix a fp in `transmute_ptr_to_ptr`

Avoid firing the lint when `transmute` in const contexts
as dereferencing raw pointers in consts is unstable. cc #5959

Auto merge of #5992 - giraffate:fix_wrong_seggestion_in_collapsible_if, r=yaahc

Fix the wrong suggestion when using macro in `collapsible_if`

Fix #5962

changelog: Fix the wrong suggestion when using macro in `collapsible_if`

Auto merge of #5909 - khuey:async_yields_async, r=yaahc

Add a lint for an async block/closure that yields a type that is itself awaitable.

This catches bugs of the form

tokio::spawn(async move {
    let f = some_async_thing();
    f // Oh no I forgot to await f so that work will never complete.
});

See the two XXXkhuey comments and the unfixed `_l` structure for things that need more thought.

*Please keep the line below*
changelog: none

default_trait_access: Fix wrong suggestion

Fix the wrong suggestion when using macro in `collapsible_if`

Auto merge of #5984 - ebroto:5693_const_assoc_fn, r=flip1995

or_fn_call: ignore nullary associated const fns

The fix in #5889 was missing associated functions.

changelog: Ignore also `const fn` methods in [`or_fun_call`]

Fixes #5693

Fix clippy

Auto merge of #5988 - camelid:patch-2, r=ebroto

Syntax-highlight `single_char_push_str` lint

It wasn't being syntax highlighted in the online lint index:


changelog: none

Syntax-highlight `single_char_push_str` lint

Formatting changes requested by ThibsG.

Add a test for an async function.

Add a lint for an async block/closure that yields a type that is itself awaitable.

This catches bugs of the form

tokio::spawn(async move {
    let f = some_async_thing();
    f // Oh no I forgot to await f so that work will never complete.
});

or_fn_call: ignore nullary associated const fns

Merge commit '3d0b0e66afdfaa519d8855b338b35b4605775945' into clippyup

Auto merge of #5970 - ebroto:changelog_beta_1_47, r=flip1995

Update changelog to beta 1.47

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

changelog: none

Auto merge of #5774 - ThibsG:FixNewRetNoSelf, r=ebroto

Fix FP in `new_ret_no_self`: trigger in trait def instead of impl block

Lint in trait def instead of impl block.

Fixes: #5435
changelog: none

Update clippy_lints/src/methods/mod.rs

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

Update clippy_lints/src/utils/mod.rs

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

Update clippy_lints/src/utils/mod.rs

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

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`

Merge logic of looking for `Self` type

Remove expansion restriction + fix doc and tests naming

Fix FP in `new_ret_no_self`: trigger in trait def instead of impl block

Fix FP in `to_string_in_display`

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

Update changelog to beta 1.47

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