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

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.

Support array indexing expressions in unused write to a constant

Mark writes to constants as side-effect-less

Auto merge of #3592 - xfix:remove-unsafe-from-consts-clippy-lints, r=flip1995

Remove unsafe from consts clippy lints

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`

Remove unsafe from consts clippy lints

Auto merge of #3561 - fuerstenau:master, r=oli-obk

Suggest `.as_ref()?` instead of `?` in certain circumstances.

Fix suggestion for unnecessary_ref lint

Update CONTRIBUTING.md for rustfix tests

Update .fixed files via update-references.sh

Run rustfix on first UI test

Use WIP branch for compiletest_rs

Fix formatting

Merge remote-tracking branch 'upstream/master'

Merge new_without_default_derive into new_without_default

Auto merge of #3587 - flip1995:ui-testing, r=phansch

Use the -Zui-testing flag for compile-test tests

Like `@petrochenkov` suggested in https://github.com/rust-lang/rust-clippy/pull/3581#discussion_r243925691

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

Match on ast/hir::ExprKind::Err

rustup rust-lang/rust#56999

Match on ast/hir::ExprKind::Err

Update *.stderr files

Use -Zui-testing flag

Auto merge of #3589 - flip1995:contrib_inactive_prs, r=phansch

Mention S-inactive-closed PRs in the CONTRIBUTING.md

Some of these PRs would be great if they could be completed. But I guess no-one would look through closed PRs if they want to contribute. Let's point (new) contributors there by mentioning these PRs in the `CONTRIBUTING.md`

Auto merge of #3586 - matthiaskrgr:test_fmt_fix, r=oli-obk

base tests: switch to nightly toolchain before checking formatting of tests with rustfmt

this errored because rustfmt is not available on the master toolchain

Mention S-inactive-closed PRs in the CONTRIBUTING.md