Add a test for `doc(include)`
Tiny code improvement.

Add `doc(include = ...)` detection to `missing_docs_in_private_items`

Auto merge of #3880 - phansch:uicleanup9001, r=oli-obk

UI test cleanup: Extract manual_memcpy tests

cc #2038

UI test cleanup: Extract manual_memcpy tests

Auto merge of #3873 - phansch:hiridification, r=flip1995

Some more HirId-ification

* 8a59f81: Rename span_lint_node* functions to span_lint_hir*
* a457258: Use `HirId` instead of `NodeId` for lookup

cargo fmt

Auto merge of #3869 - taiki-e:use_self, r=flip1995

Fix `use_self` false positive on nested functions

Related to https://github.com/rust-lang/rust-clippy/pull/3640

The current `use_self` warns the following code.

```rust
#![warn(clippy::use_self)]
struct Foo {}
impl Foo {
    fn bar() {
        fn baz() -> Foo { //^ warning: unnecessary structure name repetition
            Foo {} //^ warning: unnecessary structure name repetition
         }
    }
}
```

Auto merge of #3872 - phansch:some_feature_cleanup, r=oli-obk

Remove some unused features and `error-pattern`s

None

Auto merge of #3794 - mikerite:fix-3739, r=phansch

Fix `boxed_local` suggestion

Don't warn about an argument that is moved into a closure.

ExprUseVisitor doesn't walk into nested bodies so use a new
visitor that collects the variables that are moved into closures.

Fixes #3739

Use `HirId` instead of `NodeId` for lookup

Remove some unused features and `error-pattern`s

Auto merge of #3871 - taiki-e:needless_continue, r=phansch

Fix `needless_continue` false positive

If the `continue` has a label, check it matches the label of the loop.

Fixes https://github.com/rust-lang/rust-clippy/issues/2329

Rename span_lint_node* functions to span_lint_hir*

Because they now take a `hir_id` instead of a `node_id` argument.

Auto merge of #3865 - phansch:run_more_doc_tests, r=flip1995

Run more doc tests

This executes some more doc tests that were ignored before.

Merge branch 'master' into fix-3739

Auto merge of #3868 - taiki-e:needless_pass_by_value, r=phansch

Filter out proc_macro and proc_macro_attribute

Related to https://github.com/rust-lang/rust-clippy/pull/1617

Fixes https://github.com/rust-lang/rust-clippy/issues/3067 (this issue has already been closed, but in fact the false positive in `#[proc_macro]` and `#[proc_macro_attribute]` has not been fixed yet)

Run more doc tests

This executes some more doc tests that were ignored before.

Fix `needless_continue` false positive

Fix `use_self` false positive on nested functions

Filter out proc_macro and proc_macro_attribute

Auto merge of #3767 - alexreg:cosmetic-2, r=flip1995

Various cosmetic improvements

Related to the larger effort of https://github.com/rust-lang/rust/pull/58036.

Fix/Ignore doc tests

Run rustfmt

Update tests

Reblessed test outputs.

Addressed points raised in review.

Various cosmetic improvements.

Auto merge of #3733 - Zoxc:rustc-interface, r=oli-obk

Use the new rustc interface

Shows the changes required to compile with https://github.com/rust-lang/rust/pull/56732

rustfmt

Use the new rustc interface

Auto merge of #3863 - rust-lang:get_unwrap, r=oli-obk

Move get_unwrap to restriction

fixes #3862

r? @oli-obk

Move get_unwrap to restriction

fixes #3862

Auto merge of #3824 - phansch:adding_lints, r=phansch

Add lint writing documentation

[Rendered](https://github.com/phansch/rust-clippy/blob/adding_lints/doc/adding_lints.md)

This adds a new documentation page that explains how to write Clippy
lints. It guides the reader through creating a `foo` function lint.

I plan to iterate a bit more on the prose of some sections, but I think the
general structure is fine now, so I'm looking forward to feedback =)

One thing I'm not sure about: I felt like this is too big for CONTRIBUTING.md
so I put it into a new `doc/` directory. I can imagine having more
documentation in the future, so we might even want to create a book using
mdbook instead? Or should everything go into CONTRIBUTING.md?

Further things left to do:

- [x] Link from CONTRIBUTING.md
- [x] Remove things covered in this guide from CONTRIBUTING.md
- [x] Section about `clippy::author` attribute
- [x] Run `remark-lint` on CI over the `doc` directory and fix things

Auto merge of #3860 - phansch:refactor_out_opt_def_id, r=flip1995

Refactor: Remove utils::opt_def_id

This removes some indirection. Probably this method was uplifted to
rustc at some point?

Auto merge of #3861 - flip1995:rollup, r=flip1995

Rollup of 3 pull requests

Successful merges:

 - #3851 (Refactor: Extract `trait_ref_of_method` function)
 - #3852 (Refactor: Cleanup one part of assign_ops lint)
 - #3857 (Document match_path, improve match_qpath docs)

Failed merges:

r? @ghost

Rollup merge of #3857 - phansch:document_path_qpath, r=flip1995

Document match_path, improve match_qpath docs

Inching towards enabling `#[deny(missing_docs)]` in utils :scroll:

Rollup merge of #3852 - phansch:refactor_assign_ops, r=flip1995

Refactor: Cleanup one part of assign_ops lint

Removes a lot of indentation and separates lint emission from lint
logic. Only touches the `hir::ExprKind::AssignOp` part of the lint.

Rollup merge of #3851 - phansch:refactor_trait_stuff, r=flip1995

Refactor: Extract `trait_ref_of_method` function

This pattern was used in three places after #3844, so I think it's worth moving it into `utils/mod.rs` and documenting it.

Auto merge of #3854 - ljedrz:rustc_58992, r=phansch

Align with rust-lang/rust/#58992

Some adjustments needed after the most recent round of HirIdification.

Refactor: Remove utils::opt_def_id

This removes some indirection. Probably this method was uplifted to
rustc at some point?

Document match_path, improve match_qpath docs

cargo fmt

fmt

Update clippy_lints/src/utils/mod.rs

Co-Authored-By: phansch <dev@phansch.net>

Fix doctest

Refactor: Cleanup one part of assign_ops lint

Removes a lot of indentation and separates lint emission from lint
logic. Only touches the `hir::ExprKind::AssignOp` part of the lint.

Update comment location

Auto merge of #3856 - mikerite:clippy-dev-enchancement-1, r=phansch

Improve Clippy dev help

+ Print help if no subcommand is supplied
+ Make a short version of `update_lints` help for the subcommand listing

Auto merge of #3855 - rchaser53:issue-3849, r=flip1995

Remove an unused binary file

related https://github.com/rust-lang/rust-clippy/pull/3853
I'm sorry.

Improve Clippy dev help

+ Print help if no subcommand is supplied
+ Make a short version of `update_lints` help for the subcommand listing

Remove an unused binary file

align with rust-lang/rust/#58992

Auto merge of #3853 - rchaser53:issue-3849, r=flip1995

fix missing a semicolon

related: https://github.com/rust-lang/rust-clippy/issues/3849

fix missing a semicolon

Refactor: Extract `trait_ref_of_method` function

Auto merge of #3850 - mikerite:rustfmt-tests, r=phansch

More test improvements

Enable rustfix on `unused_unit` tests

Add `[rustfmt::skip]` to `unused_unit` test

Add `[rustfmt::skip]` to `doc_markdown` tests

Auto merge of #3803 - felix91gr:master, r=oli-obk

Cognitive Complexity (step 1 out of 3+): name changes

Following up on #3793

**Overall checklist:**

1. **Name changes**
2. MVP of functionality
3. Tests

After this PR, we will start working on the implementation itself.

Renamed: Cyclomatic Complexity -> Cognitive Complexity

* Ran automatic naming update

* Formalized rename of `cyclomatic_complexity` to `cognitive_complexity`
** Added the rename to `lib.rs`
** Added rename test

* Added warning for deprecated key `cyclomatic_complexity_threshold` and tests for it

* Added deprecation status for Clippy's builtin attribute

* Updated tests for new builtin attribute renaming

Auto merge of #3845 - euclio:unused-comments, r=phansch

move lint documentation into macro invocations

This PR moves lint documentation inside `declare_clippy_lint!` macro invocations, to avoid triggering the `unused_doc_comments` lint once it's modified in rust-lang/rust#57882. This PR is necessary to unblock that work, since the large number of warnings generated in `clippy_lints` causes Travis to hit the log length limit.

This PR also updates the documentation and website generation script.

It would be nice to get a clippy update in the Rust repo once this is merged.

cc @phansch

Auto merge of #3846 - mikerite:fix-3839, r=phansch

Ensure `expect_fun_call` bad suggestion is fixed

Closes #3839

Ensure `expect_fun_call` bad suggestion is fixed

Closes #3839

fix or ignore failing doc tests

move lint documentation into macro invocations

Auto merge of #3844 - phansch:const_fn_external_macro, r=oli-obk

Fix two missing_const_for_fn false positives

Fixes #3841

* Fixes false positive in external macros
* Fixes false positive when implement trait methods

Fix missing_const_for_fn for impl trait methods

Don't trigger missing_const_for_fn in external macros

As reported in #3841. Only fixes the part where it triggers on the
`derive`.

Add rustc nightly docs to cheatsheet

s/you/we; One more checklist item

Auto merge of #3838 - mati865:rustup, r=phansch

Upgrade compiletest_rs to 0.3.19

Upgrade compiletest_rs to 0.3.19

Add the rustc guide to the cheatsheet

More text fixes after Test section was moved to top

Reword 'Setup' a bit

Note that stderr files don't have to be created manually

Author lint usage via playground

Run remark in `doc` directory, too.

Cleanup 'Writing code' in CONTRIBUTING.md

This is now all included in `doc/adding_lints.md`

Copy from CONTRIBUTING.md: author lint, rustfix tests, rustfmt

And adapt the wording a bit so that it fits in the overall guide.

Mention Clippy utils

Remind to run update-all-references.sh at the end

Use positive if condition for readability

Expand on lint implementation section, wrap lines

Implicit return

Co-Authored-By: phansch <dev@phansch.net>

Implicit return

Co-Authored-By: phansch <dev@phansch.net>

Add lint writing documentation

This adds a new documentation page that explains how to write Clippy
lints. It guides the reader through creating a `foo` function lint.

Auto merge of #3834 - ljedrz:HirIdification_fix, r=phansch

HirIdification fixes

Supersedes https://github.com/rust-lang/rust-clippy/pull/3828, enables https://github.com/rust-lang/rust/pull/58836.

As usual, requesting a branch.

fmt fixes

align with rust-lang/rust/#58836

enable rust-lang/rust#58754

Auto merge of #3823 - spastorino:place_base, r=flip1995

Place::Local(x) is now Place::Base(PlaceBase::Local(x))

We need to merge this after the beta cut for this https://github.com/rust-lang/rust/pull/58631 to work.
/cc @oli-obk

Place::Local(x) is now Place::Base(PlaceBase::Local(x))

Auto merge of #3833 - mati865:rustup, r=oli-obk

Fix build with the master

https://github.com/rust-lang/rust/pull/58689 broke compiletest.

Use temporary branch for compiletest

https://github.com/rust-lang/rust/pull/58689 broke it

Auto merge of #3831 - martin-t:master, r=flip1995

Use the same <title> as <h1> on gh-pages

I think this makes the page easier to find using firefox's URL bar - it (afaik) searches based on the page's URL and title so if somebody doesn't have it bookmarked but remembers that the page contained "ALL lints" or something similar it's easy to go to it directly without resorting to a search engine.

Auto merge of #3830 - flip1995:unknown_attribute, r=oli-obk

Error on unknown or deprecated attributes

The reason I decided to not make this a lint is, that unknown attributes are always a hard error in rustc and also in rustfmt, which cannot be turned off by an `allow`. This could have been done by introducing a category which would be forbid-by-default. But that would have introduced a new lint category+group for just one lint and also wouldn't be consistent with rustfmt and rustc.

Add test for unknown Clippy attributes

Use the same <title> as <h1> on gh-pages