AbsolutePathBuffer -> AbsolutePathPrinter

Remove ty::TyKind from eta_reduction and replace it with ty::Ty

parent_def_id -> parent

item_path_str -> def_path_str

Reimplement AbsolutePathBuffer

Auto merge of #3886 - phansch:rustfix_useless_asref, r=flip1995

Enable rustfix for `useless_asref` lint tests

cc #3630

Enable rustfix for `useless_asref` lint tests

Auto merge of #3883 - daxpedda:missing_docs_in_private_items, r=phansch

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

The whole `missing documentation in crate` part doesn't have any tests. If I should add test cases tell me.

Another test for missing crate documentation.

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

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

Auto merge of #3879 - phansch:rustfix_string_lit_as_bytes, r=flip1995

Run rustfix for string_lit_as_bytes tests

This moves the `string_lit_as_bytes` tests into a new file and enables
rustfix tests for them.

cc #3603, #2038

Auto merge of #3877 - rink1969:3842, r=flip1995

casting integer literal to float is unnecessary

fix issue https://github.com/rust-lang/rust-clippy/issues/3842

casting integer literal to float is unnecessary

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

Run rustfix for string_lit_as_bytes tests

This moves the `string_lit_as_bytes` tests into a new file and enables
rustfix tests for them.

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>