Extract lints of unit_types group from types group

Auto merge of #6916 - camsteffen:diagnostics-util, r=Manishearth

Don't re-export `clippy_utils::diagnostics::*`

changelog: none

Continues #6907

Don't re-export clippy_utils::diagnostics::*

Auto merge of #6914 - camsteffen:source-utils, r=Manishearth

Move some utils to `clippy_utils::source`

changelog: none

Continues #6907

Auto merge of #6828 - mgacek8:issue_6758_enhance_wrong_self_convention, r=flip1995

wrong_self_convention: fix lint in case of `to_*_mut` method

fixes #6758
changelog: wrong_self_convention: fix lint in case of `to_*_mut` method. When a method starts with `to_` and ends with `_mut`, clippy expects a `&mut self` parameter, otherwise `&self`.

Any feedback is welcome. I was also thinking if shouldn't we treat `to_` the same way as `as_`. Namely to accept `self` taken:  `&self` or `&mut self`.

Auto merge of #6907 - camsteffen:ty-utils, r=flip1995

Move some utils to `clippy_utils::ty`

changelog: none

`clippy_utils::*` has become a giant junk drawer. This is one step to clean it up a bit. One motivation is that I believe the long import statements cause more merge conflicts.

Update clippy tests

Move some utils to clippy_utils::source module

Move some utils to ty_utils

Fix error after rebase

Add comments with examples and tests

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

Use `rustc_interface::interface::Config::parse_sess_created` in Clippy

Do not show docs link when lint doesn't start with "clippy::"

Add fake_read() to clippy

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

Clippy: HACK! Fix bootstrap error

This will be removed in the next sync, once beta is at 1.52. Until then
this hack avoids to put `cfg(bootstrap)` into Clippy.

Merge commit '6ed6f1e6a1a8f414ba7e6d9b8222e7e5a1686e42' into clippyup

Improve suggestion and make it work for macros

wrong_self_convention: fix lint in case of `to_*_mut` method

When a method starts with `to_` and ends with `_mut`, it should expect a `&mut self` parameter,
otherwise `&self`.

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.~~

Deprecate items that accidentally weren't deprecated

Fixes #82080

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.