Move `test_no_deps_ignores_path_deps_in_workspaces()` out of `dogfood_subprojects()`

Rename `ClippyArgsCallbacks` to `RustcCallbacks`

Let Cargo track `CLIPPY_ARGS`

Auto merge of #6824 - Y-Nak:refactor_loops_module, r=flip1995

Refactor: organize loops file into loops module (Delegated)

`@flip1995` `@nahuakang`

closes #6693
r? `@flip1995`

 As we talked about in the PM of Zulip,  this PR is a delegated PR from `@nahuakang.`

Changes from the last commit of #6693:
1. Unify the name of the main entries of all modules to check, that was pointed out [here](https://github.com/rust-lang/rust-clippy/pull/6693#discussion_r582897713)
2. Simplify ` check_for_loop_arg`, that was pointed out [here](https://github.com/rust-lang/rust-clippy/pull/6693#discussion_r582914044) and [here](https://github.com/rust-lang/rust-clippy/pull/6693#discussion_r582929910)
3. Resolve conflicts

changelog: Refactor `loops.rs` file into `loops` module.

Auto merge of #6801 - Jarcho:manual_match_fix, r=phansch

Fix `manual_map` false positives

fixes: #6795
fixes: #6797
fixes: #6811
fixes: #6819

changelog: Fix false positives for `manual_map` when `return`, `break`, `continue`, `yield`, `await`, and partially moved values are used.
changelog: Don't expand macros in suggestions  for `manual_map`

Remove "for_loop_arg.rs"

Auto merge of #6791 - TaKO8Ki:iter-count, r=matthiaskrgr

New lint: `iter_count`

This pull request adds a new lint named `iter_count`.

---

closes https://github.com/rust-lang/rust-clippy/issues/6262

changelog: new lint `iter_count`

Auto merge of #6716 - magurotuna:refactor-transmute-mod, r=flip1995

Refactor: arrange transmute lints

This PR arranges `transmute` lints so that they can be accessed more easily.
Basically, I followed the instruction described in #6680 as to how to do the refactoring.

- `declare_clippy_lint!` and `impl LintPass` is placed in `transmute/mod.rs`
- Uitlity functions is placed in `transmute/utils.rs`
- Each lint function about `transmute` is moved into its own module, like `transmute/useless_transmute.rs`

For ease of review, I refactored step by step, keeping each commit small. For instance, all I did in
2451781 was to move  `useless_transmute` into its own module.

---

changelog: Refactor `transmute.rs` file into `transmute` module.

Make check_cast private

Refactor calls to lint check functions

Cleanup imports

Move transmutes_expressible_as_ptr_casts to its own module

Move unsound_collection_transmute to its own module

Move transmute_float_to_int to its own module

Move transmute_int_to_float to its own module

Move transmute_int_to_bool to its own module

Move transmute_ptr_to_ptr to its own module

Move transmute_bytes_to_str and transmute_ptr_to_ptr to transmute_ref_to_ref module

Move transmute_int_to_char to its own module

Move transmute_ptr_to_ref to its own module

Move crosspointer_transmute to its own module

Move wrong_transmute to its own module

Move useless_transmute to its own module

Add transmute/utils.rs

Create transmute directory

Simplify check_for_loop_arg

Unify names of lint entry functions in loops to 'check'

Include loops.rs changes from PR#6698

Clean up: Rename some files to be consistent with lint names; import lints to each file

Move check_for_loop_arg back to mod; split into 4 lint files

Move MinifyingSugg into manual_memcpy

Refactor while let loop to its own module

Refactor empty loop to its own module

Refactor while let on iterator lint to its module; rename for loop explicit counter to explicit counter loop

Refactor never loop to its own module

Move detect_manual_memcpy to its module; split up utils structs

Add detect_same_item_push to its own module

Add check_needless_collect to its own module

Add check_infinite_loop to its own module

Refactor check_for_single_element_loop to its own module

Refactor check_for_loop_explicit_counter to its own module

Refactor check_for_loop_range into its module

Refactor check_for_mut_range_bound to its own module

Refactor check_for_loop_over_map_kv to its own module

Refactor check_for_loop_arg; rename manual_flatten's lint back to check_manual_flatten

create loops dir; arrange manual_flatten lint and utils

Auto merge of #6822 - camsteffen:rustfmt, r=llogiq

Rustfmt version "Two"

changelog: none

This enables some ~bug fixes~ changes from rustfmt. This is more consistent with rustc's config, and should be more forward-compatible. Also, the changes look good IMO. 😃

Don't move `yield` or inline assembly into closure

Add version = "Two" to rustfmt.toml

Ignore UI tests since this change makes rustfmt less friendly with UI
test comments.

Auto merge of #6813 - matthiaskrgr:lintcheck_refactor, r=flip1995

lintcheck, do some refactoring and add more sources

refactor: add a Config object
don't run in parallel mode by default (it didn't make sense because cargo would lock the shared target dir anyway)
show full paths (from repo root) to the source files in clippy warnings so we can just copy the path from the logfile
fix more bugs
add more crates by dtolnay and embark to the sources toml

changelog: lintcheck: refactor some code and add more sources

update lintcheck_crates logs and fix typos

lintcheck: add more embark crate sources to the sources toml

lintcheck: make download path and source path consts, move source directory from traget/lintcheck/crates to target/lintcheck/sources

also update logfile with the dtolnay crates

lintcheck: more fixes

fix a couple of issues when checking if lintcheck needed rerun after clippy binary changed.
I was apparently still comparing the times wrongly, but it should be fixed™ now...
I actually looked at the date of the sources.toml and not at the date of the log file!

Also fix progress report counter not advancing in squential mode

lintcheck: add a couple of dtolnays crates to the source list

lintcheck: put the full paths (target/lintcheck/sources/<crate>...) to the source files of a warning into the lintcheck log

This is more convenient when reviewing new lint warnings that popped up in the logs

lintcheck: uses consts for clippy driver and cargo clippy paths

lintcheck: don't run clippy in parallel by default

lintcheck: refactor: introduce a basic LintcheckConfig struct which holds the job limit and paths to the sources and log files

Auto merge of #6787 - matthiaskrgr:lint_msgs, r=llogiq

tests: add test that roughly ensures that our lint messages conform with the diagnostics convention of the rustc dev guide

lint message should not start with uppercase letters
lint messages should not have punctuation at the end of the last line

https://rustc-dev-guide.rust-lang.org/diagnostics.html#diagnostic-structure

The test reads through all the .stderr files in the testsuit and checks lint messages that start with "help: ", "error: " etc.
There is also an exception list for special messages that are deemed acceptable.

changelog: make sure lint messages conform with the rustc dev guide and add test

Fix `manual_map`: do not expand macros in suggestions

use different example (C-like) for valid capitalized start of lint message

lintcheck: update logs

change some lint messages and remove old entries from the ignorelist

disable lint_message_convention test inside the rustc test suite

update the lint messages and tests

fix clippy lint warnings

tests: add test that roughly ensures that our lint messages conform with the diagnostics convention of the rustc dev guide

lint message should not start with uppercase letters
lint messages should not have punctuation at the end of the last line

https://rustc-dev-guide.rust-lang.org/diagnostics.html#diagnostic-structure

The test reads through all the .stderr files in the testsuit and checks lint messages that start with "help: ", "error: " etc.
There is also an exception list for special messages that are deemed acceptable.

changelog: make sure lint messages conform with the rustc dev guide and add test

Auto merge of #6802 - camsteffen:dogfood-fix, r=llogiq

Remove workspace and fix dogfood (again)

changelog: none

In response to https://github.com/rust-lang/rust-clippy/pull/6733#issuecomment-785792060

Auto merge of #6812 - Y-Nak:fix-6792, r=matthiaskrgr

Fix ICEs 6792 and 6793

fixes #6792, fixes #6793.
r? `@matthiaskrgr`

Fixes the ICEs by replacing `TyCtxt::type_of` with `TypeckResults::expr_ty`.

changelog: none

Add test for ICE 6793

Use TypeckResults::expr_ty instead of TyCtxt::type_of to fix "Not a type" ICE

remove pub(crate)

export `derefs_to_slice` from methods module

receive iter method name as an argument

remove if_chain

return when the ty doesn't have `len()`

move `lints()` to `iter_count.rs`

lint for `into_iter().count()`

allow clippy::iter_count

new lint: iter_count

Auto merge of #6730 - anall:feature/owned_to_owned_methods, r=camsteffen

added new lint `owned_to_owned`

Adding new lint `owned_to_owned`

Creating draft PR to have this looked over.
I think this takes all advice I received into account.

I did have to update the `redundant_clone` test to ignore this lint -- I felt that was the safest action.

closes: #6715
changelog: added new lint `implicit_clone`

added new lint `implicit_clone`

Revert #6796: Downgrade manual_map to nursery

Fix `manual_map`: don't lint when partially moved values are used.
Fix `manual_map`: don't lint when `return`, `break`, and `continue` are used.

Auto merge of #6800 - matthiaskrgr:lintcheck_stats, r=llogiq

lintcheck: print stats how lint counts change

The stats look like this:
````
Stats:
clippy::manual_map 0 => 10
clippy::missing_panics_doc 54 => 56
clippy::upper_case_acronyms 18 => 4
````

changelog: lintcheck: print stats about changing lint counts in the log

Revert "Fix versioncheck test"

This reverts commit 1e7b1ccb2a05f80ae0a580401e7565fb1c0a4917.

Add package arguments to dogfood test

This is necessary after migrating to RUSTC_WORKSPACE_WRAPPER.

Revert "Add workspace to manifest"

This reverts commit 9bcb257985d5a16355bdd62998935f16d09b42e2.

Revert "Fix lintcheck by excluding checked crates from workspace"

This reverts commit aea55d2c6239e42a0a337d610d19061521565615.

Revert "Test workspace at once"

This reverts commit e355652fec704f307b50d42fb6a08172dc1e08bf.

Auto merge of #6796 - dtolnay-contrib:manualmap, r=flip1995

Downgrade manual_map to nursery

I believe #6795 should be considered a blocker for this lint to be considered for enabling by default.

---

changelog: remove manual_map from default list of enabled lints

lintcheck: update logs and do minor fixes

lintcheck: print stats how lint counts have changed between runs

lintcheck: fix bug when getting the config toml path.

In order to check if we need to recheck crates, I was getting the path from clap only
which is None if we don't pass it via cmdline args.

Use the dedicated lintcheck_config_toml() fnuction instead!

Downgrade manual_map to nursery

Auto merge of #6601 - mdm:fix-unit-args-false-positive, r=camsteffen

Fix false positive for unit_arg lint

Fixes #6447

To avoid false positives don't complain about unit args when they come from a path expression, e.g. a local variable.

**Note:** This is my first contribution to Clippy, so I might have messed up somewhere. Any feedback is welcome and I'm happy to work out any kinks.

---

changelog: Do not lint unit arguments when they come from a path expression.

Add original test case from issue

Auto merge of #6790 - mgacek8:issue_6748, r=Manishearth

or_fun_call: fix suggestion for `or_insert(vec![])`

fixes #6748
changelog: or_fun_call: fix suggestion for `or_insert(vec![])` on `std::collections::hash_map::Entry` or `std::collections::btree_map::Entry`

or_fun_call: fix suggestion for `or_insert(vec![])`

Applies for `std::collections::hash_map::Entry` and `std::collections::btree_map::Entry`

Example:
Previously, for the following code:
`let _ = hash_map.entry("test".to_owned()).or_insert(vec![]);`
clippy would suggest to use:
`or_insert_with(vec![])`, which causes a compiler error (E0277).

Now clippy suggests:
`or_insert_with(Vec::new)`