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

Rustup

r? `@ghost`

changelog: none

Fix dogfood errors

Properly deprecate panic_params lint

Clippy: Match on assert!() expansions without an inner block.

Remove the clippy::panic-params lint.

Rustc itself now warns for all cases that triggered this lint.

Auto merge of #6345 - giraffate:follow_up_of_rustup, r=flip1995

Revert "Add `rustfmt::skip` as a work around"

This reverts commit 0e803417f997ba35c0045704dd347e64c2a1786c.

Fixed by rustfmt v1.4.27 available in the latest nightly (2020-11-18). Refer to https://github.com/rust-lang/rustfmt/issues/4528.

changelog: none

Rollup merge of #79145 - camelid:clippy-fix-panics, r=flip1995

Fix handling of panic calls

This should make Clippy more resilient and will unblock #78343.

This PR is made against rust-lang/rust to avoid the need for a subtree
sync at ``@flip1995's`` suggestion in rust-lang/rust-clippy#6310.

r? ``@flip1995``
cc ``@m-ou-se``

Revert "Add `rustfmt::skip` as a work around"

This reverts commit 0e803417f997ba35c0045704dd347e64c2a1786c.

Fixed by https://github.com/rust-lang/rustfmt/issues/4528.

Auto merge of #6340 - giraffate:improve_doc_for_map_clone, r=Manishearth

Improve doc about `map_clone`

A follow up of https://github.com/rust-lang/rust-clippy/issues/6239#issuecomment-719100677.

`map_clone` works with not only `Iterator` but `Option` although not written in [doc](https://rust-lang.github.io/rust-clippy/master/#map_clone). Also, an example in the doc shows  a usage of dereferencing, but this isn't also written.

changelog: Improve doc about `map_clone`

Improve doc about `map_clone`

Auto merge of #6070 - matsujika:unnecessary_wrap, r=flip1995

Add new lint `unnecessary_wrap`

Fixes #5969

changelog: New lint [`unnecessary_wraps`]

Fix handling of panic calls

This should make Clippy more resilient and will unblock #78343.

This PR is made against rust-lang/rust to avoid the need for a subtree
sync at @flip1995's suggestion in rust-lang/rust-clippy#6310.

Auto merge of #6337 - ThibsG:FixIce6332, r=Manishearth

Remove `expect()` calls to avoid ICEs in `deref_addrof` lint

Fixes: #6332
changelog: none

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

Rustup

r? `@ghost`

changelog: none

Run cargo dev fmt

Pluralize lint name

Fix embarrassing grammatical error

Ignore trait implementations

Update stderr files

Update stderr files

Fix clippy error

Skip functions in PartialOrd

Remove wildcard use

Downgrade applicability to MaybeIncorrect

Allow this lint on lint tests

Skip function with no exprs contained

Make lint skip macros

Add support for methods

Make lint skip closures

Run `cargo dev fmt`

Improve lint message

Update clippy_lints/src/unnecessary_wrap.rs

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

Move `find_all_ret_expressions` into `utils`

Add test case

Split lint suggestion into two

Run rustfmt

Call `diag.multipart_suggestion` instead

Fix typo

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

Optout rustfix test

Refactor code according to reivews

Add test cases

Fix lint example

Add suggestion on type signatures

Add new lint to detect unnecessarily wrapped value

Auto merge of #78779 - LeSeulArtichaut:ty-visitor-return, r=oli-obk

Introduce `TypeVisitor::BreakTy`

Implements MCP rust-lang/compiler-team#383.
r? `@ghost`
cc `@lcnr` `@oli-obk`

~~Blocked on FCP in rust-lang/compiler-team#383.~~

Remove `expect()` calls to avoid ICEs in `deref_addrof` lint

clippy: fold by value

Auto merge of #6119 - rsulli55:find_is_some_on_strs, r=flip1995

Add a case to `lint_search_is_some` to handle searching strings

Fixes: #6010
This adds a lint which recommends using `contains()` instead of `find()` followed by `is_some()` on strings as suggested in #6010.

This was added as an additional case to
https://github.com/rust-lang/rust-clippy/blob/5af88e3c2d8cc4fb74a0e455381669930ee3a31a/clippy_lints/src/methods/mod.rs#L3037

I would really appreciate any comments/suggestions for my code!

changelog: Added case to `lint_search_is_some` to handle searching strings

Auto merge of #6336 - giraffate:sync-from-rust, r=flip1995

Rustup

changelog: none

Add `rustfmt::skip` as a work around

because comments are checked and removed by rustfmt for some reason

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

Rustup

r? `@ghost`

changelog: none

Rollup merge of #79016 - fanzier:underscore-expressions, r=petrochenkov

Make `_` an expression, to discard values in destructuring assignments

This is the third and final step towards implementing destructuring assignment (RFC: rust-lang/rfcs#2909, tracking issue: #71126). This PR is the third and final part of #71156, which was split up to allow for easier review.

With this PR, an underscore `_` is parsed as an expression but is allowed *only* on the left-hand side of a destructuring assignment. There it simply discards a value, similarly to the wildcard `_` in patterns. For instance,
```rust
(a, _) = (1, 2)
```
will simply assign 1 to `a` and discard the 2. Note that for consistency,
```
_ = foo
```
is also allowed and equivalent to just `foo`.

Thanks to ````@varkor```` who helped with the implementation, particularly around pre-expansion gating.

r? ````@petrochenkov````

Set the default `BreakTy` to `!`

Introduce `TypeVisitor::BreakTy`

Auto merge of #78809 - vn-ki:fix-issue-76064, r=oli-obk

add error_occured field to ConstQualifs,

fix #76064

I wasn't sure what `in_return_place` actually did and not sure why it returns `ConstQualifs` while it's sibling functions return `bool`. So I tried to make as minimal changes to the structure as possible. Please point out whether I have to refactor it or not.

r? `@oli-obk`
cc `@RalfJung`

Add underscore expressions for destructuring assignments

Co-authored-by: varkor <github@varkor.com>

Auto merge of #6320 - giraffate:fix_suggestion_in_manual_range_contains_using_float, r=llogiq

Fix suggestion in `manual_range_contains` when using float

Fix #6315

changelog: Fix suggestion in `manual_range_contains` when using float

update clippy test ouput

Auto merge of #6329 - giraffate:sync-from-rust, r=matthiaskrgr

Rustup

changelog: none

Fix dogfood test

Rollup merge of #78836 - fanzier:struct-and-slice-destructuring, r=petrochenkov

Implement destructuring assignment for structs and slices

This is the second step towards implementing destructuring assignment (RFC: rust-lang/rfcs#2909, tracking issue: #71126). This PR is the second part of #71156, which was split up to allow for easier review.

Note that the first PR (#78748) is not merged yet, so it is included as the first commit in this one. I thought this would allow the review to start earlier because I have some time this weekend to respond to reviews. If ``@petrochenkov`` prefers to wait until the first PR is merged, I totally understand, of course.

This PR implements destructuring assignment for (tuple) structs and slices. In order to do this, the following *parser change* was necessary: struct expressions are not required to have a base expression, i.e. `Struct { a: 1, .. }` becomes legal (in order to act like a struct pattern).

Unfortunately, this PR slightly regresses the diagnostics implemented in #77283. However, it is only a missing help message in `src/test/ui/issues/issue-77218.rs`. Other instances of this diagnostic are not affected. Since I don't exactly understand how this help message works and how to fix it yet, I was hoping it's OK to regress this temporarily and fix it in a follow-up PR.

Thanks to ``@varkor`` who helped with the implementation, particularly around the struct rest changes.

r? ``@petrochenkov``

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

Rustup

r? `@ghost`

changelog: none

Auto merge of #78782 - petrochenkov:nodoctok, r=Aaron1011

Do not collect tokens for doc comments

Doc comment is a single token and AST has all the information to re-create it precisely.
Doc comments are also responsible for majority of calls to `collect_tokens` (with `num_calls == 1` and `num_calls == 0`, cc https://github.com/rust-lang/rust/pull/78736).

(I also moved token collection into `fn parse_attribute` to deduplicate code a bit.)

r? `@Aaron1011`

Fix suggestion in `manual_range_contains` when using float

Implement destructuring assignment for structs and slices

Co-authored-by: varkor <github@varkor.com>

Fix typo in comment

occurences -> occurrences

Change variable named `foo` and rerun `update-all-references`

Update clippy_lints/src/methods/mod.rs

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

Cleaned up message and suggestion for `lint_search_is_some`

Added period back to lint `search_is_some` and ran
`update-all-references.sh`

Remove borrow

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

Remove `to_string` on msg

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

Ran `tests/ui/update-all-references.sh" and `cargo dev fmt`

Moved the tests for lint `search_is_some` to new files
`search_is_some.rs` and `search_is_some_fixable.rs`

Added a lint as suggested in 6010 which recommends using `contains()`
instead of `find()` follows by `is_some()` on strings

Update clippy_lints/src/find_is_some_on_strs.rs
Co-authored-by: Takayuki Nakata <f.seasons017@gmail.com>
Update clippy_lints/src/methods/mod.rs
Co-authored-by: Philipp Krones <hello@philkrones.com>

Auto merge of #6269 - camsteffen:map-clone-deref, r=ebroto

Fix map_clone with deref and clone

changelog: Fix map_clone false positive with deref coercion

Fixes #6239

Fix map_clone with deref and clone

Auto merge of #6303 - ThibsG:OptionOptionSerde, r=ebroto

Remove `allow` in `option_option` lint test

As it is not triggering locally anymore, I propose to remove `#[allow(clippy::option_option)]` from the test.

The goal is also to see what happens on CI.

closes: #4298

changelog: none

Auto merge of #6319 - ebroto:rustup, r=ebroto

Rustup

changelog: none

r? `@ghost`

Run cargo dev fmt

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

Auto merge of #6305 - smoelius:master, r=flip1995

Add `let_underscore_drop`

This line generalizes `let_underscore_lock` (#5101) to warn about any initializer expression that implements `Drop`.

So, for example, the following would generate a warning:
```rust
struct Droppable;
impl Drop for Droppable {
    fn drop(&mut self) {}
}
let _ = Droppable;
```

I tried to preserve the original `let_underscore_lock` functionality in the sense that the warning generated for
```rust
let _ = mutex.lock();
```
should be unchanged.

*Please keep the line below*
changelog: Add lint [`let_underscore_drop`]

Rollup merge of #78710 - petrochenkov:macvisit, r=davidtwco

rustc_ast: Do not panic by default when visiting macro calls

Panicking by default made sense when we didn't have HIR or MIR and everything worked on AST, but now all AST visitors run early and majority of them have to deal with macro calls, often by ignoring them.

The second commit renames `visit_mac` to `visit_mac_call`, the corresponding structures were renamed earlier in https://github.com/rust-lang/rust/pull/69589.

Allow `let_underscore_drop` in `filter_methods` test

Auto merge of #6278 - ThibsG:DerefAddrOf, r=llogiq

Fix bad suggestions for `deref_addrof` and `try_err` lints

Fix bad suggestions when in macro expansion for `deref_addrof` and `try_err` lints.

Fixes: #6234
Fixes: #6242
Fixes: #6237
changelog: none

r? `@llogiq`

Update references

Do not collect tokens for doc comments

Update lints

Update clippy_lints/src/let_underscore.rs

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

Update clippy_lints/src/let_underscore.rs

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

Update clippy_lints/src/let_underscore.rs

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

Auto merge of #6267 - camsteffen:or-fun-idx, r=flip1995

Fix or_fun_call for index operator

changelog: Fix or_fun_call for index operator

Fixes #6266

Fix or_fun_call for index operator

Make KNOW_TYPES static

Auto merge of #6288 - flip1995:changelog_internal, r=matthiaskrgr

Require the `changelog:` entry in the PR body to be modified

r? `@matthiaskrgr`

changelog: none

Force contributors/reviewers to set _some_ changelog entry

Auto merge of #6293 - Urcra:lint-example-fix, r=flip1995

Fix example used in cargo_common_metadata

The previous example used in `cargo_common_metadata`  included an authors field, even though the comment says it doesn't. And thus doesn't actually demonstrate an example of how the lint fails.

This removes that authors field from the _bad_ example and suggest to fix the _bad_ example by adding the authors field

changelog: Fix example used in `cargo_common_metadata`

Update references

Auto merge of #6271 - camsteffen:vec-box-import, r=flip1995

Fix vec_box scope error

changelog: Fix vec_box suggestion with wrong type scope

Fixes #6236

Add let_underscore_drop