Auto merge of #6313 - giraffate:fix_fp_needless_collect, r=ebroto

Fix FP in indirect `needless_collect` when used multiple times

Fix https://github.com/rust-lang/rust-clippy/issues/5991
Fix https://github.com/rust-lang/rust-clippy/issues/6297

changelog: Fix FP in indirect `needless_collect` when used multiple times

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

Rustup

changelog: none

r? `@ghost`

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

Auto merge of #78439 - lzutao:rm-clouldabi, r=Mark-Simulacrum

Drop support for all cloudabi targets

`cloudabi` is a tier-3 target, and [it is no longer being maintained upstream][no].

This PR drops supports for cloudabi targets. Those targets are:
* aarch64-unknown-cloudabi
* armv7-unknown-cloudabi
* i686-unknown-cloudabi
* x86_64-unknown-cloudabi

Since this drops supports for a target, I'd like somebody to tag `relnotes` label to this PR.

Some other issues:
* The tidy exception for `cloudabi` crate is still remained because
  * `parking_lot v0.9.0` and `parking_lot v0.10.2` depends on `cloudabi v0.0.3`.
  * `parking_lot v0.11.0` depends on `cloudabi v0.1.0`.

[no]: https://github.com/NuxiNL/cloudabi#note-this-project-is-unmaintained

Auto merge of #6369 - camsteffen:cast-cfg, r=Manishearth

Disable unnecessary_cast for cfg-dependant types

changelog: Disable unnecessary_cast for cfg-dependant types

Fix  #6331

Auto merge of #6317 - chansuke:add-external-macro, r=llogiq

Add exteranal macros for as_conversions

Added external macros to this PR https://github.com/rust-lang/rust-clippy/pull/4888.

r? `@llogiq`

changelog: none

Disable unnecessary_cast for cfg-dependant types

Drop support for cloudabi targets

Auto merge of #6361 - integer32llc:doc-style, r=carols10cents

Small grammar, punctuation, and code style improvements to docs

changelog: Made small grammar, punctuation, and code style improvements to docs

I recently found some places in rust-lang/rust that had lists without spaces after commas, which led me to look for more places, which led me over here to find:

- Some similar lists in code examples that could use spaces after commas to be idiomatic Rust style
- Some lists in documentation text that didn't have spaces after commas, needed an Oxford comma (fight me), or were otherwise misformatted
- Some other grammar improvements in the area of the other changes

These changes should only be in user-facing documentation or output.

Apply suggestions from code review to change "that" to "which"

Co-authored-by: oliver <16816606+o752d@users.noreply.github.com>

Small grammar, punctuation, and code style improvements to docs

Auto merge of #6363 - o752d:patch-2, r=flip1995

a typo

typo

changelog: none

Auto merge of #6364 - o752d:master, r=flip1995

revisiting a typo

changelog: none

update

a typo

typo

Thread `Constness` through selection

Auto merge of #6354 - Daniel-B-Smith:refcell_ref_await, r=flip1995

Downgrade the await holding lints from correctness

We found a false positive in these lints (see https://github.com/rust-lang/rust-clippy/issues/6353 for more details). As a short-term mitigation, this downgrades the lints from correctness to limit the noise.

changelog: downgrade AWAIT_HOLDING_REFCELL_REF and AWAIT_HOLDING_LOCK to pedantic. From rustup earlier, where I forgot the changlog: deprecate [`panic_params`] (uplifted)

Revert "Convert the await holding lints to correctness"

This reverts commit d8c6bce4407b1c99ed961f75a093ffe767818069.

Auto merge of #6347 - flip1995:changelog, r=Manishearth

Changelog Rust 1.48

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

This changelog is **big**. Hacktoberfest was pretty good for us, even though we had too few reviewers.

changelog: none

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.

Improve changlog update documentation

Fix trailing whitespaces in CHANGELOG file

Add CHANGELOG for 1.48

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``

Fix FP in indirect `needless_collect` when used multiple times

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