Restrict unnecessary_sort_by to non-ref copy types

needless-lifetime - fix nested elision site FPs

Auto merge of #5997 - giraffate:fix_fp_about_clone_in_same_item_push, r=ebroto

Fix FP in `same_item_push`

Don't emit a lint when the pushed item doesn't have Clone trait

Fix #5979

changelog: Fix FP in `same_item_push` not to emit a lint when the pushed item doesn't have Clone trait

Fix FP when coercion kicks in

Auto merge of #6009 - Ryan1729:show-line_count-and-max_lines-in-too_many_lines-lint-message, r=phansch

Show line count and max lines in too_many_lines lint message

This PR adds the current amount of lines and the current maximum number of lines in the message for the `too_many_lines` lint, in a similar way as the `too_many_arguments` lint currently does.

changelog: show the line count and the maximum lines in the message for the `too_many_lines` lint.

run the specific script suggested by the error message

```
./tests/ui-toml/update-references.sh './target/debug/test_build_base' 'functions_maxlines/test.rs'
```

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

add line_count and max_lines to too_many_lines lint message

update example to be more idiomatic

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

Rustup

r? @ghost

changelog: none

Run cargo dev fmt

Fix fallout from rustup

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

ty.flags -> ty.flags()

`ty.kind` -> `ty.kind()` in rustdoc and clippy

Auto merge of #6004 - mikerite:default-trait-access-20200904, r=ebroto

Simplify `clippy::default_trait_access`

Remove repeated matching on the same QPath.

changelog: none

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

Removed map_err suggestion in lint, and updated lint documentation example

unit-arg - pr remarks

Auto merge of #76160 - scileo:format-recovery, r=petrochenkov

Improve recovery on malformed format call

The token following a format expression should be a comma. However, when it is replaced with a similar token (such as a dot), then the corresponding error is emitted, but the token is treated as a comma, and the parsing step continues.

r? @petrochenkov

Merge branch 'master' into unit-arg

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

Improve recovery on malformed format call

If a comma in a format call is replaced with a similar token, then we
emit an error and continue parsing, instead of stopping at this point.

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.

Ran cargo dev update_lints

Added tests for map_err, ignored map_err lint on drop_ref tests

Working map_err_ignore lint

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

Fix FP in `same_item_push`

Don't emit a lint when the pushed item doesn't have Clone trait

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

fixed bug

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`

Use `match_def_path` instead of `match_qpath`

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

Update dataflow analyses to use new interface

uncommented fn

changed check_impl_item to check_fn and added a few more test cases

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.
});

Downgrade applicability of `create_dir`

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

Fix errors

Fix clippy error

Add STD_FS_CREATE_DIR into paths

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

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>

added lint for todo and removed option

ran cargo dev update lints

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`

edited documentation

removed unnecessary comment

added restriction lint that prohibits the usage of unimplemented, unreachable or panic in a function of type result or option

unit-arg - pr comments

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.