Move some utils to clippy_utils::source module

Move some utils to ty_utils

Auto merge of #6831 - camsteffen:suspicious-map, r=Y-Nak,flip1995

Fix suspicious_map false positives

changelog: Fix suspicious_map false positives

Fixes #5253
Replaces #5375

Improve `find_binding_init` docs

Fix suspicious_map false positives

Auto merge of #6895 - iobtl:reformat_unnecessary_cast, r=llogiq

replace span_lint with span_lint_and_sugg along with error message

fixes: #6874
changelog: none

apologies if this may not be the most idiomatic way of doing it, any advice on changes (if any) would be greatly appreciated.

use lint_unnecessary_cast for literals, suggest `_` if not present

Auto merge of #6859 - magurotuna:if_then_some_else_none, r=giraffate

Implement new lint: if_then_some_else_none

Resolves #6760

changelog: Added a new lint: `if_then_some_else_none`

Auto merge of #6820 - mgacek8:issue_6562_enhance_mem_replace_with_default_with_other_ctors, r=phansch

mem_replace_with_default: recognize some std library ctors

fixes #6562
changelog: mem_replace_with_default: recognize some common constructors equivalent to `Default::default()`

Auto merge of #6865 - Jarcho:explicit_deref_methods, r=llogiq

`explicit_deref_methods` improvements

Breaking up #6837

changelog: `explicit_deref_methods` will lint chained `deref` calls and ufcs style calls

Properly lint macro arguments for `explicit_deref_methods`

Keep track of whether `deref` or `deref_mut` was called
Remove more unnecessary code

Remove unneeded code.

Make `explicit_deref_methods` check for multiple deref calls
Fix suggestion for `explicit_deref_methods`. Sometimes `&**` is needed, sometimes nothing is needed.
Allow `explicit_deref_methods` to trigger in a few new contexts.
`explicit_deref_methods` will now consider ufcs calls

replace span_lint with span_lint_and_sugg along with error message

Auto merge of #6892 - matthiaskrgr:inc_struct_ctor, r=Y-Nak

inconsistent_struct_constructor: try to make message and lint description a bit clearer

changelog: inconsistent_struct_constructor: try to make message and lint description a bit clearer

r? `@ghost`

Auto merge of #6891 - Y-Nak:use-iterator-sym, r=Manishearth

Use sym::Iterator instead of paths::ITERATOR

Since `sym::Iterator` was added to diagnostic_item, it's time to remove `paths::ITERATOR`.
ref: #5393

changelog: Add `is_trait_method` to `clippy_utils`
changelog: Remove `paths::ITERATOR`

mem_replace_with_default: use diagnostic items intead of paths

mem_replace_with_default: recognize some std library ctors

inconsistent_struct_constructor: try to make message and lint description a bit clearer

Use sym::Iterator instead of paths::ITERATOR

Improve suggestion and make it work for macros

Auto merge of #6826 - TaKO8Ki:refactor-methods-mod, r=phansch

Refactor: arrange lints in `methods` module

This PR arranges methods lints so that they can be accessed more easily.
Basically, I refactored them following the instruction described in #6680.

changelog: Move lints in methods module into their own modules.

Auto merge of #6884 - matthiaskrgr:lintcheck_crate, r=Manishearth

move lintcheck into its own crate

This pr:
* moves lintcheck out of `clippy dev` and into its own crate (`lintcheck`)  (I should have done this earlier :D)
* makes lintcheck terminate if it is not launched from the repo root (to prevent problems with wrong paths when using `cargo run` in the crate root)
* fixes json lint messages leaking the runners `$HOME` when a lint messages comes from a proc macro that originates from a crate inside the `$CARGO_CACHE`
* adds more documentation to lintchecks `README.md` and mentions lintcheck in `docs/basics.md`

changelog: none

doc line length fixes

docs: basics.md: mention lintcheck

lintcheck: fix clippy warnings

lintcheck: fix --fix and document it in the readme.

also hook lintcheck into clippy-dev so that `clippy dev fmt` formats it.

lintcheck: update logs

lintcheck: fix bug where lint messages about macros coming from crate deps would sneak in absolute paths to registry sources.

make the path a relative path that starts at the CARGO_HOME to not print the users home location in the log

lintcheck: update readme and remove the now redundant readme from clippy-dev

move testfiles to "lintcheck" and fix more paths

gitignore: add lintchecks target dir

lintcheck: make sure we lauch from the repo root

This will terminate the program if run via "cargo run".
"cargo run" does currently not work because at least a bunch of paths do not take that into account.

lintcheck: move out of clippy-dev into own crate

fix interning-defined-symbol error

remove unused imports

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