Auto merge of #3645 - phansch:remove_copyright_headers, r=oli-obk

Remove all copyright license headers

Discussion previously happened in https://github.com/rust-lang/rust/pull/43498 and here: https://github.com/rust-lang/rust-clippy/pull/3642#issuecomment-452251727

Remove all copyright license headers

Discussion previously happened in https://github.com/rust-lang/rust/pull/43498

Auto merge of #3640 - detrumi:nested_use_self, r=flip1995

Restrict `use_self` on nested items

Fixes #3637
Fixes #3463

These changes make it so that nested items aren't visited any more by the `use_self` lint.

I think visiting nested items should be possible (so that it uses a different `item_path` for the nested item), but I'm not sure whether it's viable and what the best approach would be.
- Can `item_path` be changed to a new `Self` path before visiting the item, and then changing it back afterwards?
- Alternatively, could a new visitor be created, re-using `check_trait_method_impl_decl`?

Auto merge of #3600 - xfix:cast-ref-to-mut, r=flip1995

cast_ref_to_mut lint

I see this pattern way too often, and it's completely wrong. In fact, due to how common this incorrect pattern is, [the Rustonomicon specifically points this out](https://doc.rust-lang.org/nomicon/transmutes.html).

> - Transmuting an & to &mut is UB
>   - Transmuting an & to &mut is always UB
>   - No you can't do it
>   - No you're not special

This is my first lint.

Auto merge of #3641 - hellow554:patch-1, r=flip1995

Add missing ` in default lint

Move cast_ref_to_mut list to correctness group

Rustftmt

Don't import ty::Ref in cast_ref_to_mut lint

Move a hint to an error message in cast_ref_to_mut lint

This matches mem::transmute::<&T, &mut T> lint in rustc.

Add a note to cast_ref_to_mut lint

Use ty::Ref instead of ty::TyKind::Ref

cast_ref_to_mut lint

Add missing ` in default lint

Improve tests and exclude nested impls

Auto merge of #3638 - mikerite:fix-3625, r=oli-obk

Improve `get_unwrap` suggestion

Handle case where a reference is immediately dereferenced.

Fixes #3625

Update `unwrap_get` code review suggestions

Update known problems

Restrict use_self on nested items

Improve `get_unwrap` suggestion

Handle case where a reference is immediately dereferenced.

Fixes 3625

Auto merge of #3635 - matthiaskrgr:revert_random_state_3603, r=xfix

Revert the random_state lint.

Remove the random_state lint until it or rustc has been fixed to no longer crash with debug assertions (see #3628)
We can't update clippy in the rustc repo because of this which is blocking nightlies because toolstate is already broken.

fixes #3628

Auto merge of #3626 - phansch:rustfix_works, r=oli-obk

Add run-rustfix where it already works

This PR adds `// run-rustfix` headers to tests for `MachineApplicable` lints where
applying the suggestions works without any errors.

Revert "tests: used_underscore_binding_macro: disable random_state lint."

This reverts commit 2b80829fe0311fc5cdbc731fa0b67f103db4c004.

Revert "Auto merge of #3603 - xfix:random-state-lint, r=phansch"

This reverts commit 0a6593cd1b12c06d4ff2f2ad0ccf20e77f1ec5f6, reversing
changes made to 5277a1fb6c1be42ec0c57a6f60d82fc18a962259.

This hopefully fixes #3628

Auto merge of #3627 - detrumi:use_self_local_macro, r=phansch

Trigger `use_self` lint in local macros

Closes #2098

The test currently only covers local macros. #2098 suggested this:
> You could add the macro in question into the `mini_macro` subcrate

But that doesn't work for a `macro_rules`:
```
error: cannot export macro_rules! macros from a `proc-macro` crate type currently
```

So I suggest leaving out the test for external macros, as using `in_external_macro` seems straigtforward enough. Alternatives would be to use to add an additional crate (overkill if you ask me), or test with a `proc-macro`.

Auto merge of #3610 - phansch:method_rs_cleanup, r=flip1995

UI test cleanup: Extract lint from methods.rs test

Extracts the `result_map_unwrap_or_else` lint into a separate test file.

This also extracts the `IteratorFalsePositives` struct and impl into
`auxiliary/option_helpers.rs`.

cc #2038

Auto merge of #3633 - matthiaskrgr:rustup, r=phansch

rustup (don't know the exact PR unfortunately)

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

rustup (don't know the exact PR unfortunately)

Auto merge of #3631 - phansch:add_itertools_to_integration_tests, r=phansch

Add itertools to integration tests

Mainly because it causes an ICE in https://github.com/rust-lang/rust/issues/57298
and it's a good library to test anyway.

Auto merge of #3629 - matthiaskrgr:crash_workaround, r=phansch

tests: used_underscore_binding_macro: disable random_state lint.

Trying to work around a crash (see https://github.com/rust-lang/rust-clippy/issues/3628) in
https://github.com/rust-lang/rust/pull/57303

Add itertools to integration tests

tests: used_underscore_binding_macro: disable random_state lint.

Trying to work around a crash (see https://github.com/rust-lang/rust-clippy/issues/3628) in
https://github.com/rust-lang/rust/pull/57303

Auto merge of #3601 - xfix:move-constant-write-lint, r=flip1995

Move constant write checks to temporary_assignment lint

They make more sense here

cc #3595

Trigger `use_self` lint in local macros

Add run-rustfix where it already passes

Auto merge of #3623 - phansch:rustup, r=flip1995

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

None

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

Auto merge of #3622 - m-cat:master, r=phansch

Add ui/for_kv_map test for false positive in #1279

Fixes #1279

Auto merge of #3621 - Zoxc:parallel, r=phansch

Make clippy work with parallel rustc

Make clippy work with parallel rustc

Add ui/for_kv_map test for false positive in #1279

Auto merge of #3519 - phansch:brave_newer_ui_tests, r=flip1995

Integrate rustfix into Clippy test suite

Once the [PR to compiletest-rs](https://github.com/laumann/compiletest-rs/pull/151) is reviewed and merged this fixes #2376.

I will create a separate tracking issue for adding `run-rustfix` to all tests.

Update to latest compiletest-rs release

Auto merge of #3617 - matthiaskrgr:3462_test, r=phansch

add testcase for #3462

add testcase for #3462

Auto merge of #3615 - matthiaskrgr:rtuVbump, r=phansch

deps: bump rustc_tools_util version from 0.1.0 to 0.1.1 just in case...

I'm surprised there is no warning since the root cargo.toml specifies 0.1.0 but the subcrate toml actually states 0.1.1.

I should have noticed this in previous pull request but oh well...

deps: bump rustc_tools_util version from 0.1.0 to 0.1.1 just in case...

Auto merge of #3603 - xfix:random-state-lint, r=phansch

random_state lint

Auto merge of #3614 - matthiaskrgr:rustc_tool_utils, r=phansch

rustc_tool_utils: fix failure to create proper non-repo version string when used in crates on crates.io, bump version

Auto merge of #3607 - detrumi:limit_infinite_iter_to_known_types, r=phansch

Only trigger `infinite_iter` lint for infinitely allocating `collect()` calls

Fixes  #3538

~Oh, I guess this should actually check other methods like `count` as well, not only `collect()`.~
Never mind, `collect` is the only of these functions that allocates a data structure.

Use compiletest's aux-build header instead of include macro

rustc_tool_utils: fix failure to create proper non-repo version string when used in crates on crates.io, bump version

Auto merge of #3609 - codeworm96:fix_test, r=phansch

Fix test for rust-lang/rust#57250

Part of rust-lang/rust#57250.
~~Do not merge. Waiting a nightly with that PR merged.~~

Auto merge of #3612 - phansch:copies_cleanup, r=flip1995

UI test cleanup: Extract ifs_same_cond tests

cc #2038

Auto merge of #3611 - phansch:some_for_loop_cleanup, r=flip1995

UI test cleanup: Extract for_kv_map lint tests

cc #2038

rustfmt

UI test cleanup: Extract ifs_same_cond tests

Extract IteratorFalsePositives into option_helpers.rs

This was previously duplicated in #3605

UI test cleanup: Extract for_kv_map lint tests

UI test cleanup: Extract lint from methods.rs test

Fix test for rust-lang/rust#57250

Limit infinite_iter collect() check to known types

Auto merge of #3608 - phansch:improve_util_docs, r=oli-obk

Some improvements to util documentation

None

Some improvements to util documentation

Auto merge of #3606 - detrumi:blacklisted_name_hashset, r=phansch

Use hashset in `blacklisted_name` lint

Use hashset for name blacklist

Auto merge of #3590 - jorpic:i3559-if_same_then_else, r=phansch

Fix if_same_then_else false positive

This fixes false positive in #3559.
The problem was that `SpanlessEq` does not check patterns in declarations. So this two blocks considered equal.
```rust
if true {
    let (x, y) = foo();
} else {
   let (y, x) = foo();
}
```
Not sure if the proposed change is safe as `SpanlessEq` is used extensively in other lints, but I tried hard to come up with counterexample and failed.

Auto merge of #3599 - xfix:use-hash-set-for-valid-idents, r=oli-obk

Use an FxHashSet for valid idents in documentation lint

Auto merge of #3605 - phansch:small_methods_ui_test_improvement, r=flip1995

UI test cleanup: Extract iter_skip_next from methods.rs

cc #2038

Reformat random_state tests

UI test cleanup: Extract iter_skip_next from methods.rs

cc #2038

Use node_id_to_type_opt instead of node_it_to_type in random_state

Auto merge of #3593 - mikerite:readme-syspath-2, r=phansch

Update README local run command to remove syspath

Since #3257 was reverted, including the sysroot in RUSTFLAGS gives the
error `Option 'sysroot' given more than once`

Auto merge of #3588 - detrumi:tuple_struct_use_self, r=phansch

`use_self` for tuple structs

Fixes #3498

Check pattern equality while checking declaration equality

Merge branch 'master' of https://github.com/rust-lang/rust-clippy into random-state-lint

Auto merge of #3597 - xfix:match-ergonomics, r=phansch

Match ergonomics (lints from A to B)

Auto merge of #3602 - matthiaskrgr:rustup, r=phansch

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

item.name -> item.ident.name

Update test output after rebase

Remove false negatives from known problems

Implement use_self for tuple structs

Document known problems

random_state lint

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

item.name -> item.ident.name

Move constant write checks to temporary_assignment lint

They make more sense here

Auto merge of #3598 - xfix:apply-cargo-fix-edition-idioms, r=phansch

Apply cargo fix --edition-idioms fixes

Auto merge of #3558 - russelltg:new_without_default_merge, r=flip1995

Merge new_without_default_derive into new_without_default

Closes #3525, deprecating new_without_default_derive and moving both lints into new_without_default.

Use an FxHashSet for valid idents in documentation lint

Remove unnecessary `use` statements after `cargo fix`

Apply cargo fix --edition-idioms fixes

Use match ergonomics for booleans lint

Use match ergonomics for block_in_if_condition lint

Use match ergonomics for bit_mask lint

Use match ergonomics for attrs lint

Use match ergonomics for assign_ops lint

Use match ergonomics for artithmetic lint

Use match ergonomics for approx_const lint

Auto merge of #3596 - xfix:remove-crate-from-paths, r=flip1995

Remove crate:: prefixes from crate paths

This is somewhat misleading, as those are actually external crates,
and don't need a crate:: prefix.

Auto merge of #3595 - xfix:mark-writes-to-constant-as-sideeffectless, r=oli-obk

Mark writes to constants as side-effect-less

This is my first real contribution, so I probably did it wrong.

Remove crate:: prefixes from crate paths

This is somewhat misleading, as those are actually external crates,
and don't need a crate:: prefix.