Remove force-host and explain no-prefer-dynamic

cargo fmt

Fix ICE in suspicious_else_formatting

Merge pull request #3922 from rust-lang/run-pass

Add missing `// run-pass` annotations to ICE tests

Add missing `// run-pass` annotations to ICE tests

compiletest UI tests do not fail when encountering panics and ICEs
unless the `// run-pass` flag is used.

(This was forgotten in https://github.com/rust-lang/rust-clippy/pull/3743)

Merge pull request #3916 from matthiaskrgr/rustup

rustup https://github.com/rust-lang/rust/pull/59657

rustup https://github.com/rust-lang/rust/pull/59657

commit: 4122d2221ec65232bc211a266d0796a4713e0d39

Merge pull request #3905 from rust-lang/compiletest

Hacky rustup

Run rustfmt

Set level of identity_conversion FP to warn

Rustup to https://github.com/rust-lang/rust/pull/58805

rustup 41316f0449025394fdca6606d3fdb3b8f37a9872

Updated source to match with recent rustc `master` toolchain changes

Fix dogfood error of question_mark lint fix

Fix question_mark lint+test

Fix some test failures

Hacky rustup

Merge pull request #3902 from matthiaskrgr/rustup

rustup https://github.com/rust-lang/rust/pull/59096/

run cargo fmt

rustup https://github.com/rust-lang/rust/pull/59096/

Auto merge of #3888 - phansch:extract_mistyped_literal_suffix_tests_and_rustfix, r=flip1995

Add rustfix tests for mistyped_literal_suffix lint

This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc #3603, #2038

Based on #3887

Auto merge of #3894 - markcatley:adding_lint_documentation_fixes, r=phansch

Add implementation for the EarlyLintPass trait into the Adding Lints …

…documentation.

To follow along with the adding lint documentation you need to implement this trait in order to register the lint pass.

cargo fmt

Add rustfix tests for mistyped_literal_suffix lint

This moves all `mistyped_literal_suffix` tests to their own file and
enables rustfix tests for them.

cc #3603, #2038

Based on #3887

Move some `unreadable_literal` ui tests to correct file

The moved tests were part of `tests/ui/literals.rs` before. Now they are
placed in `tests/ui/unreadable_literal.rs`, which now also runs
`rustfix` on these 4 extra let statements.

Add implementation for the EarlyLintPass trait into the Adding Lints documentation.

To follow along with the adding lint documentation you need to implement this trait in order to register the lint pass.

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

Rustup

Supersedes https://github.com/rust-lang/rust-clippy/pull/3889

Addresses some review comments from previous PR and rustups to https://github.com/rust-lang/rust/pull/58899

Cargo fmt

Drop range_contains feature

Fix rustfmt::skip detection

Rework clippy detection in attribute lint

name -> check_name

span -> span()

name -> ident_str

NestedMetaItemKind -> NestedMetaItem

Drop redundant &

Fix path_qualified

As suggested by eddyb

Replace most of ty:Ty with Ty

Feed dog

Adapt paths to new formatting

cargo fmt

Use LocalInternedString inside of AbsolutePathPrinter

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