Add tests for #1654

Don't trigger while_let_on_iterator when the iterator is recreated every iteration

Auto merge of #5520 - matthiaskrgr:rustup_44, r=flip1995,phansch

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

There's currently an crash in `ui/new_without_default.rs` that I need to figure out how to avoid.

changelog: none

Update issue_2356.stderr reference file

Update while_let_on_iterator tests

Fix while_let_on_iterator suggestion and make it MachineApplicable

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

Auto merge of #5513 - matthiaskrgr:reg, r=phansch

fix clippy_dev exit status and make regex match again

changelog: none

Fixes #5510

r? @phansch

Auto merge of #5498 - phansch:update_changelog, r=flip1995

Update CHANGELOG.md for Rust 1.43 and 1.44

[Rendered](https://github.com/phansch/rust-clippy/blob/update_changelog/CHANGELOG.md)

changelog: none

README: fix lit count line

It looks like after changing to "there are more than 120 lints", an older PR was merged
and resolving merge conflicts this was changed back to "there are 123 lints" causing the update-script to silently fail.

Changed back the README.md back to the new format fixes the problem.

clippy_dev: make it fatal when the regex for updating lint count does not match

Fixes #5510

Auto merge of #5508 - lzutao:rustup-71044, r=phansch

Rustup "Remove `BodyAndCache`"

cc https://github.com/rust-lang/rust/pull/71044
changelog: none

`predecessors_for` will be removed soon

Co-Authored-By: ecstatic-morse <ecstaticmorse@gmail.com>

Rustup "Remove `BodyAndCache`"

Auto merge of #5439 - rokob:lock-await, r=Manishearth

Lint for holding locks across await points

Fixes #4226

This introduces the lint await_holding_lock. For async functions, we iterate
over all types in generator_interior_types and look for types named MutexGuard,
RwLockReadGuard, or RwLockWriteGuard. If we find one then we emit a lint.

changelog: introduce the await_holding_lock lint

span_lint_and_note now takes an Option<Span> for the note_span instead of just a span

Make lint also capture blocks and closures, adjust language to mention other mutex types

don't test the code in the lint docs

Switch to matching against full paths instead of just the last element of the path

Lint for holding locks across await points

Fixes #4226

This introduces the lint await_holding_lock. For async functions, we iterate
over all types in generator_interior_types and look for types named MutexGuard,
RwLockReadGuard, or RwLockWriteGuard. If we find one then we emit a lint.

Also mention `--fix` for nightly users

Auto merge of #5499 - matthiaskrgr:crash_5497, r=flip1995

fix crash on issue-69020-assoc-const-arith-overflow.rs

Fixes #5497

changelog: fix crash on rustc test issue-69020-assoc-const-arith-overflow.rs

fix crash on issue-69020-assoc-const-arith-overflow.rs

Fixes #5497

Auto merge of #5496 - phansch:markdown-link, r=flip1995

util/fetch_prs_between.sh: Add Markdown formatted link

This can then be easily copy/pasted into the changelog :blue_heart:

changelog: none

Address review comments

Auto merge of #5495 - phansch:update_changelog_docs, r=flip1995

Update the changelog update documentation

I just started working on updating the changelog. Hopefully the docs are a bit clearer now?

r? @flip1995

changelog: none

remark fixes

Update CHANGELOG.md for Rust 1.43 and 1.44

Auto merge of #5332 - DevinR528:if-let-else-mutex, r=flip1995

If let else mutex

changelog: Adds lint to catch incorrect use of `Mutex::lock` in `if let` expressions with lock calls in any of the blocks.

closes: #5219

update stderr file

util/fetch_prs_between.sh: Add Markdown formatted Link

This can then be easily copy/pasted into the changelog :blue_heart:

factor ifs into function, add differing mutex test

Update the changelog update documentation

Auto merge of #5493 - ebroto:unsafe_derive_deserialize, r=flip1995

Implement unsafe_derive_deserialize lint

Added `unsafe_derive_deserialize` lint to check for cases when automatically deriving `serde::Deserialize` can be problematic, i.e. with types that have methods using `unsafe`.

Closes #5471

changelog: Add lint [`unsafe_derive_deserialize`]

Apply suggestions from PR review

* Move the lint to pedantic
* Import used types instead of prefixing with `hir::`

update span_lint_and_help call to six args

test for mutex eq, add another test case

use if chain

cargo dev fmt

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

Implement unsafe_derive_deserialize lint

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`