move expect_fun_call to its own module

move or_fun_call to its own module

move map_unwrap_or to its own module

move option_map_or_none to its own module

unnecessary_fold to its own module

move useless_asref to its own module

move clone_on_copy to its own module

move iter_skip_next to its own module

move map_flatten and search_is_some to their own modules

move flat_map_identity to its own module

move filter_map_flat_map to its own module

move filter_flat_map to its own module

move filter_map_next to its own module

move filter_map_map to its own module

move filter_map to its own module

move from_iter_insteam_of_collect to its own module

move into_iter_on_ref and single_char_pattern to their own modules

move single_char_push_string to its own module

move single_char_insert_string to its own module

move string_extend_chars and clone_on_ref_ptr to their own module

move wrong_self_convention to its own module

move iter_nth, iter_nth_zero and iterator_step_by_zero to their own module

move iter_next_slice to its own module

move map_collect_result_unit to its own module

move zst_offset to its own module

replace `lints` and `lint` with `check`

move iter_cloned_collect to its own module

move uninit_assumed_init to its own module

move filetype_is_file to its own module

move suspicious_map to its own module

move option_as_ref_deref to its own module

move skip_while_next to its own module

move expect_used, filter_next, get_unwrap, ok_expect and unwrap_used to their own modules

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

Rustup

changelog: none

Fix remaining dogfood errors (internal lints)

Bump nightly version => 2021-03-11

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

Auto merge of #6814 - hyd-dev:issue-6486, r=flip1995

Fix false positives on procedural macros of `missing_inline_in_public_items` lint

Fixes #6486.

changelog: Fix false positives on procedural macros of `missing_inline_in_public_items` lint.

Auto merge of #6794 - camsteffen:needless-borrowed-ref, r=flip1995

Improve needless_borrowed_ref docs

changelog: none

I think "borrowed ref" is a confusing description for this lint. Destructuring a reference is the opposite of borrowing. So I updated the wording throughout the docs. Unfortunately this nit applies to the name of the lint itself, but I won't bother changing that. One motivation for these changes is to clarify the difference between this lint and `needless_borrow` (they are actually quite different). Let me know if I need to clarify anything or if you disagree with any changes.

Auto merge of #79519 - cjgillot:noattr, r=wesleywiser

Store HIR attributes in a side table

Same idea as #72015 but for attributes.
The objective is to reduce incr-comp invalidations due to modified attributes.
Notably, those due to modified doc comments.

Implementation:
- collect attributes during AST->HIR lowering, in `LocalDefId -> ItemLocalId -> &[Attributes]` nested tables;
- access the attributes through a `hir_owner_attrs` query;
- local refactorings to use this access;
- remove `attrs` from HIR data structures one-by-one.

Change in behaviour:
- the HIR visitor traverses all attributes at once instead of parent-by-parent;
- attribute arrays are sometimes duplicated: for statements and variant constructors;
- as a consequence, attributes are marked as used after unused-attribute lint emission to avoid duplicate lints.

~~Current bug: the lint level is not correctly applied in `std::backtrace_rs`, triggering an unused attribute warning on `#![no_std]`. I welcome suggestions.~~

Auto merge of #6878 - flip1995:authors, r=Manishearth

Set Clippy authors to "The Rust Clippy Developers"

Clippy has grown enough, that putting specific people in the "authors"
field isn't warranted anymore.

As a heads-up: `@Manishearth` `@llogiq` `@birkenfeld` `@mcarton` `@oli-obk` `@phansch` `@matthiaskrgr` your names will be removed from one or more of the `Cargo.toml` files of Clippy. This of course does not mean that we value your previous work on Clippy any less :heart:

As per our discussion in today's meeting: https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/Meeting.202021-03-09/near/229502514

changelog: none

Remove hir::Expr::attrs.

Remove hir::Item::attrs.

Remove hir::ImplItem::attrs.

Remove hir::TraitItem::attrs.

Remove hir::StructField::attrs.

Remove hir::Variant::attrs.

Remove hir::Arm::attrs.

Remove hir::Crate::attrs.

Remove hir::Local::attrs.

Remove hir::StmtKind::attrs.

Do not store attrs in FnKind.

Simplify clippy author.

Switch to changing cp_non_overlap in tform

It was suggested to lower this in MIR instead of ssa, so do that instead.

Update cranelift

Update match branches

This updates all places where match branches check on StatementKind or UseContext.
This doesn't properly implement them, but adds TODOs where they are, and also adds some best
guesses to what they should be in some cases.

Set Clippy authors to "The Rust Clippy Developers"

Clippy has grown enough, that putting specific people in the "authors"
field is warranted anymore.

Auto merge of #6876 - giraffate:improve_doc_on_map_flatten, r=flip1995

Improve doc on `map_flatten`

Fix https://github.com/rust-lang/rust-clippy/issues/6870. It's because this doc lacks the description that this lint is also used for `Option`.

changelog: none

Improve doc on `map_flatten`

Auto merge of #6866 - anall:ice6840, r=flip1995

Fix ICE 6840 - make is_normalizable more strict

fixes #6840

make `is_normalizable` more strict, which should catch this ICE and related cases

changelog: Fix ICE in [`zero_sized_map_values`]

add comment for when  can be removed

Auto merge of #6868 - Jarcho:lang_item, r=flip1995

Don't assume lang items will exist.

~~Should fix lintcheck warnings caused by #6823~~
See below

changelog: None

Auto merge of #6873 - Y-Nak:refactor-casts-lint, r=flip1995

Refactor casts lint

Ref: #6724

Changes:
1. Separate the `casts` group from the `types` group.
2. Reorganize the lints of the `casts` group into their own modules.

Notes:
1. I didn't `fix` #6874 in order to maintain this PR as small as possible.

---
changelog: none

Move ptr_as_ptr to its own module

Rollup merge of #82048 - mark-i-m:or-pat-type-ascription, r=petrochenkov

or-patterns: disallow in `let` bindings

~~Blocked on https://github.com/rust-lang/rust/pull/81869~~

Disallows top-level or-patterns before type ascription. We want to reserve this syntactic space for possible future generalized type ascription.

r? ``@petrochenkov``

Move char_lit_as_u8 to its own module

Move cast_ref_to_mut to its own module

Move cast_ptr_alignment to its own module

Move fn_to_numeric_cast_with_truncation to its own module

Move fn_to_numeric_cast to its own module

Remove 'lint_numeric_casts'

Move unnecessary_cast to its own module

Move cast_possible_wrap to its own module

Move cast_sign_loss to its own module

Move cast_possible_truncation to its own module

Auto merge of #6869 - DJMcNab:rust-analyzer-private, r=phansch

Opt-in to rustc_private for `rust-analyzer`

rust-analyzer/rust-analyzer#7891

changelog: none

This will also help priroda and any other package which depends on the `miri` library crate.

use `.all` instead of negative use of `.any`

use TyS::walk

Move cast_lossless to its own module

Move cast_precision_loss to its own module

Move 'is_isize_or_usize' to clippy_utils

Separate lints of casts group from types group

Auto merge of #6871 - camsteffen:redundant-closure-macro, r=Manishearth

Fix redundant closure with macros

changelog: Fix redundant_closure FPs with macros

Fixes #6732
Fixes #6850
Fixes #4354 (addresses the error message confusion)

Improve the redundant_closure message

clippy: fix or-pattern in let binding

Fix redundant closure with macros

Opt-in to rustc_private for `rust-analyzer`

rust-analyzer/rust-analyzer#7891

Don't assume lang items will exist.

Auto merge of #6725 - Y-Nak:refactor-types-lints, r=flip1995

Refactor types lints

Ref #6724.
As described in #6724, `types.rs` contains many groups inside it.
In this PR, I reorganize the lints of the `types` group into their own modules.

changelog: none

Fix some lints in types that fail dogfood

Import declared lints at the top of the file

Remove unused is_local from borrowed_box

Move borrowed_box to its own module

Move linked_list to its own module

Move option_option to its own module

Add flags to detect lints are triggered

Move vec_box to its own module

Move rc_buffer to its own module

Move redundant_allocation to its own module

Move box_vec to its own module