fix map import to rustc_middle

dev update_lints

fix internal clippy warnings

change visitor name to OppVisitor

use Visitor api to find Mutex::lock calls

add note about update-all-refs script, revert redundant pat to master

move closures to seperate fns, remove known problems

use span_lint_and_help, cargo dev fmt

creating suggestion

progress work on suggestion for auto fix

Auto merge of #5141 - xiongmao86:issue5095, r=flip1995

Fixes issue 5095

fixes #5095.

- [x] Followed [lint naming conventions][lint_naming]
- [x] Added passing UI tests (including committed `.stderr` file)
- [x] `cargo test` passes locally
- [x] Executed `cargo dev update_lints`
- [x] Added lint documentation
- [x] Run `cargo dev fmt`

[lint_naming]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints

changelog: (internal) warn about collapsible `span_lint_and_then` calls.

Update empty_enum.stderr

Auto merge of #5491 - smklein:borrowed_box, r=flip1995

Fix issue #2907.

Update the "borrow box" lint to avoid recommending the following
conversion:

```
  // Old
  pub fn f(&mut Box<T>) {...}

  // New
  pub fn f(&mut T) {...}
```

Given a mutable reference to a box, functions may want to change
"which" object the Box is pointing at.

This change avoids recommending removing the "Box" parameter
for mutable references.

changelog: Don't trigger [`borrow_box`] lint on `&mut Box` references

Formatting and naming

Formatting and naming

Auto merge of #5489 - phansch:node-id-hir-id, r=flip1995

Cleanup: `node_id` -> `hir_id`

This removes some more `node_id` terminology from Clippy and replaces one occurrence of `as_local_node_id` with `as_local_hir_id`, which should be doing the same for that particular case.

changelog: none

Auto merge of #5488 - phansch:future-not-send-ret-ty, r=flip1995

Cleanup: future_not_send: use `utils::return_ty` function

changelog: none

Auto merge of #5490 - sinkuu:toplevel_ref_arg_for, r=phansch

Don't trigger toplevel_ref_arg for `for` loops

The lint suggests turning `for ref x in 0..10 {` into `for ref x in let x = &0..10; {`.

---

changelog: none

Cleanup: `node_id` -> `hir_id`

Fix issue #2907.

Update the "borrow box" lint to avoid recommending the following
conversion:

```
  // Old
  pub fn f(&mut Box<T>) {...}

  // New
  pub fn f(&mut T) {...}
```

Given a mutable reference to a box, functions may want to change
"which" object the Box is pointing at.

This change avoids recommending removing the "Box" parameter
for mutable references.

Don't trigger toplevel_ref_arg for `for` loops

Cleanup: future_not_send: use `return_ty` method

Auto merge of #5486 - flip1995:badge_over, r=flip1995

Remove badge FIXME from Cargo.toml

cc https://github.com/rust-lang/crates.io/issues/2436

changelog: none

Remove badge FIXME from Cargo.toml

Change note_span argument for span_lint_and_note.

Add an Option<Span> argument to span_lint_and_help.

Fixes internal lint warning in code base.

Implement collapsible_span_lint_calls lint.

Auto merge of #5427 - pmk21:implicit-sat-sub, r=flip1995

Add lint named implicit_saturating_sub

Fixes: #5399
I've made a basic skeleton of the lint, would love more feedback on how to make it better.
changelog: Add lint [`implicit_saturating_sub`]

Polished lint and tests

Added final lint and tests

Added basic lint and tests

Auto merge of #5483 - alex-700:fix-redundant-pattern-matching, r=flip1995

fix redundant_pattern_matching lint

- now it handles `while let` case  (related to #5462)
- better suggestions in `if let` case

changelog: Fix suggestion in `redundant_pattern_matching` and also apply this lint to the `while let` case

fix redundant_pattern_matching lint

- now it handles `while let` case
- better suggestions in `if let` case

Auto merge of #5423 - rkuhn:add_futures_not_send, r=flip1995

add lint futures_not_send

changelog: add lint futures_not_send

fixes #5379

~Remark: one thing that can (should?) still be improved is to directly include the error message from the `Send` check so that the programmer stays in the flow. Currently, getting the actual error message requires a restructuring of the code to make the `Send` constraint explicit.~
It now shows all unmet constraints for allowing the Future to be Send.

Auto merge of #5445 - logan-dev-oss:master, r=flip1995

Fixes issue #4892.

First contribution here 😊 ! Do not hesitate to correct me.

This PR is related to issue #4892 .

# Summary

```rust
-literal.method_call(args)
```
The main idea is to not trigger `clippy::precedence` when the method call is an odd function.

# Example

```rust
// should trigger lint
let _ = -1.0_f64.abs() //precedence of method call abs() and neg ('-') is ambiguous

// should not trigger lint
let _ = -1.0_f64.sin() // sin is an odd function => -sin(x) = sin(-x)
```

# Theory

Rust allows following literals:
- char
- string
- integers
- floats
- byte
- bool

Only integers/floats implements the relevant `std::ops::Neg`.
Following odd functions are implemented on i[8-128] and/or f[32-64]:
- `asin`
- `asinh`
- `atan`
- `atanh`
- `cbrt`
- `fract`
- `round`
- `signum`
- `sin`
- `sinh`
- `tan`
- `tanh `
- `to_degrees`
- `to_radians`

# Implementation

As suggested by `flip1995` in [comment](https://github.com/rust-lang/rust-clippy/issues/4892#issuecomment-568249683), this PR add a whitelist of odd functions and compare method call to the the whitelist before triggering lint.

changelog: Don't trigger [`clippy::precedence`] on odd functions.

add lint futures_not_send

Integrate more idiomatic rust changes.

Fix issue #4892.

Auto merge of #5482 - phansch:diag, r=matthiaskrgr

Cleanup: Rename 'db' variables to 'diag'

Did the same in rustc a while ago: https://github.com/rust-lang/rust/pull/65428

changelog: none

Auto merge of #5481 - sinkuu:no_as_ref, r=phansch

question_mark: don't add `as_ref()` for a call expression

If a call returns a `!Copy` value, it does so regardless of whether `as_ref()` is added. For example, `foo.into_option().as_ref()?` can be simplified to `foo.into_option()?`.

---

changelog: Improved `question_mark` lint suggestion so that it doesn't add redundant `as_ref()`

cargo dev fmt

Cleanup: Rename 'db' variable to 'diag'

question_mark: don't add `as_ref()` for a call expression

Auto merge of #5480 - sinkuu:unit_arg_app, r=flip1995

unit_arg suggestion is not really machine-applicable

This lint suggests replacing any code returining `()`-type with `()` literal, which can change the behavior. e.g. `Ok(do_something())` -> `OK(())`.

---

changelog: none

unit_arg suggestion may be incorrect

Auto merge of #5479 - matthiaskrgr:readme_auto_fix, r=flip1995

readme: update to cargo clippy --fix command

changelog: none

Auto merge of #5476 - ThibsG:FixMatchesInExternalMacros, r=flip1995

Do not lint in macros for match lints

Don't lint in macros for match lints, more precisely in `check_pat` and `check_local` where it was not the case.

changelog: none

fixes: #5362

Auto merge of #5475 - woshilapin:woshilapin-patch-1, r=flip1995

[fix] Minor typo in GH Actions 'clippy_dev'

changelog: minor: fixes file name in Github Actions workflow `clippy_dev.yml`

I was looking for something else and ended up on that typo, just thought I would fix it.

readme: update to cargo clippy --fix command

Auto merge of #5473 - Toxyxer:map-flatten-for-option, r=flip1995

Map flatten for option

Fixes #5175

changelog:
- Trigger the map_flatten lint when map is called on an option
- Add test for such case

Auto merge of #5474 - phansch:reenable-rustfmt, r=flip1995

Reenable rustfmt integration test

changelog: none

Auto merge of #5478 - matthiaskrgr:apt_ci_debug_5477, r=phansch,flip1995

 perform system upgrade fixes ci issue

Perhaps doing a system upgrade magically solves the problem?

changelog: none

CI: performing system upgrade fixes broken apt deps on ubuntu 32bit

See #5477 for details

Do not lint in macros for match lints

[fix] Minor typo in GH Actions 'clippy_dev'

Reenable rustfmt integration test

Add test to map_flatten with an Option

Lint map_flatten if caller is an Option

Auto merge of #5363 - yaahc:clippy-fix, r=phansch,flip1995

add --fix support to `cargo-clippy`

Prior to this we had started work on integrating clippy as a subcommand directly into cargo in the form of `cargo clippy-preview` and `cargo fix --clippy`. In the course of that work it was decided that the best approach would be to strictly add the features clippy needed to cargo in order to insert `clippy-driver` only for workspace crates. This was accomplished by adding a `RUSTC_WORKSPACE_WRAPPER` env variable to cargo that will override the normal `RUSTC_WRAPPER` when both are present and the current crate is a workspace crate.

This change adds support to clippy to use this by setting the `RUSTC_WORKSPACE_WRAPPER` env variable instead `RUSTC_WRAPPER` and by detecting `--fix` as an arg and swapping out the `check` cargo command for `fix` when it is present.

WIP, here are the current issues that I still need to resolve

- [x] Detect if we're running on nightly rust
  - [x] Set `RUSTC_WORKSPACE_WRAPPER` on nightly, and `RUSTC_WRAPPER` on stable
  - [x] Error out on stable when `--fix` is specified, because stable currently hasn't landed the PR for `RUSTC_WORKSPACE_WRAPPER` so if we set this it just runs check and silently fails
- [ ] Update the help text
  - [ ] The current plan is to shell out to `cargo check --help` and then postprocess the output to mention clippy instead of check where appropriate and to add the extra info about `--fix` and the `-- -A lint` options.
- [x] tests?

changelog: add `--fix` arg to `cargo-clippy`

Apply suggestions from code review

Co-Authored-By: Philipp Krones <hello@philkrones.com>

manually fixing formatting at this point lol

fmt

rename field

Auto merge of #5470 - flip1995:rollup-cvkuiza, r=flip1995

Rollup of 5 pull requests

Successful merges:

 - #5226 (Add lint for explicit deref and deref_mut method calls)
 - #5248 (Add lint on large non scalar const)
 - #5430 (Disallow bit-shifting in integer_arithmetic)
 - #5466 (large_enum_variant: Report sizes of variants)
 - #5468 (Zero single char names)

Failed merges:

r? @ghost

changelog: rollup

Rollup merge of #5468 - Toxyxer:zero-single-char-names, r=flip1995

Zero single char names

Fixes: #4086
changelog:
- Make the inequality strict

Rollup merge of #5466 - phansch:large-enum-variant-output, r=flip1995

large_enum_variant: Report sizes of variants

This reports the sizes of the largest and second-largest variants.

Closes #5459

changelog: `large_enum_variant`: Report the sizes of the largest and second-largest variants.

Rollup merge of #5430 - michaelsproul:integer-arithmetic, r=flip1995

Disallow bit-shifting in integer_arithmetic

Make the `integer_arithmetic` lint detect all the operations that are defined as being capable of overflow in the [Rust Reference](https://doc.rust-lang.org/reference/expressions/operator-expr.html#overflow), by also linting for bit-shifting operations (`<<`, `>>`).

changelog: Disallow bit-shifting in `integer_arithmetic`

Rollup merge of #5248 - ThibsG:ConstValues, r=flip1995

Add lint on large non scalar const

This PR adds the new lint `non_scalar_const` that aims to warn against `const` declaration of large arrays. For performance, because of inlining, large arrays should be preferably declared as `static`.

Note: i made this one to warn on all const arrays, whether they are in a body function or not. I don't know if this is really necessary, i could just reduce this lint to variables out of function scope.

Fixes: #400
changelog: add new lint for large non-scalar types declared as const

Rollup merge of #5226 - ThibsG:DerefExplicit1566, r=flip1995

Add lint for explicit deref and deref_mut method calls

This PR adds the lint `explicit_deref_method` that suggests replacing `deref()` and `deref_mut()` with `&*a` and `&mut *a`.

It doesn't lint inside macros.

This PR is the continuation of  #3258.

changelog: Add lint `explicit_deref_method`.

Fixes: #1566

Auto merge of #5467 - matthiaskrgr:rustup_43, r=flip1995

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

changelog: none

Rename dummy_hir_id -> parent_hir_id

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

revert the damn fmt changes

add some tests

split it up for testing but the merge broke tests

Change default many single char names threshold

Merge remote-tracking branch 'upstream/master' into clippy-fix

Better precedence case management + more tests

Use only check_expr with parent expr and precedence

Check for Deref trait impl + add fixed version

Report using stmts and expr + tests

Global rework + fix imports

Working basic dereference clip

Add test for zero single char names

Make the single char threshold strict inequality

large_enum_variant: Report sizes of variants

This reports the sizes of the largest and second-largest variants.

Auto merge of #5464 - matthiaskrgr:dep_compiletest, r=phansch

deps: bump compiletest-rs from 0.4 to 0.5

changelog: none

Auto merge of #5345 - Toxyxer:add-lint-for-float-in-array-comparison, r=flip1995

Add lint for float in array comparison

Fixes #4277
changelog:
- Added new handler for expression of index kind (e.g. `arr[i]`). It returns a constant when both array and index are constant, or when the array is constant and all values are equal.
- Trigger float_cmp and float_cmp_const lint when comparing arrays. Allow for comparison when one of the arrays contains only zeros or infinities.
- Added appropriate tests for such cases.

Auto merge of #5452 - phansch:match_def_path_refactor, r=matthiaskrgr

Refactor: Use rustc's `match_def_path`

This replaces our match_def_path implementation with the rustc one.

Note that we can't just use it in all call sites because of the
`&[&str]` / `&[Symbol]` difference in Clippy/rustc.

changelog: none

Refactor: Use rustc's `match_def_path`

This replaces our match_def_path implementation with the rustc one.

Note that we can't just use it in all call sites because of the
`&[&str]` / `&[Symbol]` difference in Clippy/rustc.

Auto merge of #5449 - phansch:diagnostic-items, r=matthiaskrgr

Make use of more diagnostic items

This makes use of some (not all) already existing diagnostic items. Specifically:

* 79982a2: `core::mem::uninitialized`, `core::mem::zeroed`, `alloc::sync::Arc`, `alloc::sync::Rc`
* 83874d0: `Option` and `Result`

cc #5393

changelog: none

Auto merge of #5460 - phansch:fix_incorrect_tests, r=matthiaskrgr

result_map_unit_fn: Fix incorrect UI tests

`x` and the `HasResult` struct were missing in this file.

changelog: none

deps: bump compiletest-rs from 0.4 to 0.5

Auto merge of #5457 - phansch:sym, r=matthiaskrgr

Cleanup: Use our `sym!` macro more

It's much shorter than Symbol::intern and the effect should still be clear

---

changelog: none

Auto merge of #5463 - matthiaskrgr:rustup_42, r=phansch

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

changelog: none

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

Auto merge of #5453 - rabisg0:fix/redundant_clone, r=phansch

Fixes #5405: redundant clone false positive with arrays

Check whether slice elements implement Copy before suggesting to drop
the clone method

changelog: add a check for slice indexing on redundant_clone lint

Auto merge of #5458 - phansch:is-proc-macro-attr, r=matthiaskrgr

Cleanup: Use rustc's is_proc_macro_attr

It's doing exactly the same: https://doc.rust-lang.org/nightly/nightly-rustc/src/rustc_ast/expand/mod.rs.html#8-12

changelog: none

Auto merge of #5448 - Emerentius:update_new_ret_no_self_docs, r=phansch

Update documentation for new_ret_no_self

changelog: Update documentation for lint new_ret_no_self to reflect that the return type must only contain `Self`, not be `Self`

The lint was changed to be more lenient than the documentation implies in PR #3338 (Related issue #3313)