Move cast_possible_wrap to its own module

Move cast_sign_loss to its own module

Move cast_possible_truncation to its own module

Move cast_lossless to its own module

Move cast_precision_loss to its own module

Move 'is_isize_or_usize' to clippy_utils

Separate lints of casts group from types group

Auto merge of #6871 - camsteffen:redundant-closure-macro, r=Manishearth

Fix redundant closure with macros

changelog: Fix redundant_closure FPs with macros

Fixes #6732
Fixes #6850
Fixes #4354 (addresses the error message confusion)

Improve the redundant_closure message

Fix redundant closure with macros

Auto merge of #6725 - Y-Nak:refactor-types-lints, r=flip1995

Refactor types lints

Ref #6724.
As described in #6724, `types.rs` contains many groups inside it.
In this PR, I reorganize the lints of the `types` group into their own modules.

changelog: none

Fix some lints in types that fail dogfood

Import declared lints at the top of the file

Remove unused is_local from borrowed_box

Move borrowed_box to its own module

Move linked_list to its own module

Move option_option to its own module

Add flags to detect lints are triggered

Move vec_box to its own module

Move rc_buffer to its own module

Move redundant_allocation to its own module

Move box_vec to its own module

Create types dir and move old module under it

Auto merge of #6834 - hyd-dev:clippy-args, r=phansch,flip1995,oli-obk

Let Cargo track CLIPPY_ARGS

This PR makes `clippy-driver` emit `CLIPPY_ARGS` in its `dep-info` output.

Just like #6441, this allows this workflow to work:
```shell
cargo clippy # warning: empty `loop {}` wastes CPU cycles
cargo clippy -- -A clippy::empty_loop # no warnings emitted
```
But without rebuilding all dependencies.

cc https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/CLIPPY_ARGS.20is.20not.20tracked.20by.20Cargo/near/228599088

changelog: Cargo now re-runs Clippy if arguments after `--` provided to `cargo clippy` are changed.

Don't panic if `CLIPPY_ARGS` is not Unicode

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

Rename `ClippyArgsCallbacks` to `RustcCallbacks`

Auto merge of #6864 - giraffate:fix_doc_adding_a_new_lint, r=Manishearth

Add msrv to contents in adding lints md

changelog: none

Add msrv to contents in adding lints md

Auto merge of #6807 - anall:feature/use_new_diagnostics, r=Manishearth

migrate paths to newly-added diagnostic items

This gets rid of the following paths:
  * OS_STRING
  * TO_OWNED
  * TO_STRING

Removes some usages of:
 * PATH_BUF

Per #5393

also removes unneeded `is_ty_param_path` from `clippy_lints::types` and relocates `is_ty_param_lang_item` and `is_ty_param_diagnostic_item` to `clippy_utils`.

changelog: none

relocate functions from `clippy_lints::types`

relocate `is_ty_param_lang_item` and `is_ty_param_diagnostic_item` to `clippy_utils`

migrate paths to newly-added diagnostic items

This gets rid of the following paths:
  * OS_STRING
  * TO_OWNED
  * TO_STRING

Also removes some usages of:
  * PATH_BUF

And the now completely unused `clippy_lints::types::is_ty_param_path`

Auto merge of #6853 - Jarcho:len_without_is_empty_fp, r=Manishearth

`len_without_is_empty` will now consider multiple impl blocks

fixes #1562

This also reverts #1559 as the `#[allow]` now works on the `len` method. A note has also been added to point out where the `empty` method is, if it exists.

changelog: `len_without_is_empty` will now consider multiple impl blocks
changelog: `len_without_is_empty` will now consider `#[allow]` on both the `len` method, and the type definition

`len_without_is_empty` will now consider multiple impl blocks
`len_without_is_empty` will now consider `#[allow]` on both the `len` method, and the type definition

Auto merge of #6823 - Jarcho:diagnostic_items, r=phansch

Use diagnostic or language items instead of paths

I think that gets everything except ones used in a list of paths to check.

changelog: none

Use `LanguageItems::require`

Auto merge of #6854 - mgacek8:6844_fix_doc_for_useless_format_lint, r=phansch

useless_format: fix examples in the description

fixes #6844
changelog: useless_format: fix examples in the description

Auto merge of #6848 - matthiaskrgr:lintcheck_clippyfix, r=llogiq

lintcheck: add --fix mode which tries to apply lint suggestions to th…

…e sources and prints a warning if that fails

Great for spotting false positives/broken suggestions of applicable lints.

There are false positives though because I'm not sure yet how to silence rustc warnings while keeping clippy warnings.
Sometimes rustc makes a suggestion that fails to apply and the implementation does not differentiate between clippy and rustc warnings when applying lint suggestions.

changelog: none

useless_format: fix examples in the description

"Good" example was something not acceptable by the useless_format lint.

Auto merge of #6852 - camsteffen:avoid-mir, r=Manishearth

Remove a couple MIR usages

changelog: none

We use MIR to get the return type of a closure/function in a couple places. But typeck seems like a better approach.

This is the easy part of #6080.

Also did a tiny cleanup with `typeck` -> `typeck_body`.

Use typeck_body

Avoid mir in missing_errors_doc

Avoid mir in implicit_return

lintcheck: add --fix mode which tries to apply lint suggestions to the sources and prints a warning if that fails

Great for spotting false positives/broken suggestions of applicable lints.

There are false positives though becasue I'm not sure yet how to silence rustc warnings while keeping clippy warnings.
Sometimes rustc makes a suggestion that fails to apply and the implementation does not differenciate between clippy and rustc warnings when applying lint suggestions.

changelog: none

Auto merge of #6849 - flip1995:dogfood-fix, r=matthiaskrgr

Dogfood and CI fixes

The CI fix is practically #6829 rebased and squashed into one commit

Dogfood fix is a follow up of #6802

r? `@matthiaskrgr` for lintcheck changes

(best reviewed with whitespace changes hidden)

changelog: none

Extract directory creation into its own function

Fix dogfood errors in clippy_dev

Fix dogfood errors in clippy_lints

Remove hack that forces dogfood to run on nightly

This isn't necessary anymore, since we don't use a custom toolchain anymore

Fix dogfood test

Since clippy cannot be a workspace, we have to check the sub-crates separately

ci: Sync clippy and clippy_bors workflows

Those workflows should always test exactly the same things

Auto merge of #6846 - matthiaskrgr:lintcheck_test, r=Manishearth

lintcheck: add test (but don't run on ci)

This is the rest of https://github.com/rust-lang/rust-clippy/pull/6829 but without adding anything to ci

*Please write a short comment explaining your change (or "none" for internal only changes)*
changelog: none

update lintcheck logs

lintcheck: add test

Auto merge of #6843 - Jarcho:match_on_same_arms_macro, r=Manishearth

Compare empty blocks for equality based on tokens

fixes: #1390

This only considers empty blocks for now, though we should also catch something like this:

```rust
match 0 {
    0 => {
        do_something();
        trace!(0);
        0
    }
    1 => {
        do_something();
        trace!(1);
        1
    }
    x => x,
}
```

As far as I can tell there aren't any negative effects on other lints. These blocks only happen to be the same for a given compilation, not all compilations.

changelog: Fix `match_on_same_arms` and others. Only consider empty blocks equal if the tokens contained are the same.

Auto merge of #6833 - daxpedda:use-self, r=Manishearth

Fix false-positive in `use_self`

Fixes  #6818.

changelog: Fix false positives for use_self in macros.

When checking if two empty hir blocks are equal also check to see if the tokens used are the same as well

Auto merge of #6839 - matthiaskrgr:lintcheck_clippy_warnings, r=flip1995

lintcheck: fix clippy warnings

split out from https://github.com/rust-lang/rust-clippy/pull/6829
changelog: none

lintcheck: fix clippy warnings

Auto merge of #6825 - djc:naive-bytecount-pedantic, r=Manishearth

Move naive_bytecount to pedantic

As discussed on Zulip, current best practice is to avoid recommending
external crates. This lint is from before that was enforced. Move it
to the pedantic group to avoid enabling it by default.

https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/naive_bytecount.20suggesting.20extra.20dependency

changelog: move [`naive_bytecount`] to pedantic

Let Cargo track `CLIPPY_ARGS`

Fix false-positive in `use_self`

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`

Move naive_bytecount to pedantic

As discussed on Zulip, current best practice is to avoid recommending
external crates. This lint is from before that was enforced. Move it
to the pedantic group to avoid enabling it by default.

https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/naive_bytecount.20suggesting.20extra.20dependency

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