Skip to content

Commit ac477db

Browse files
committed
Merge remote-tracking branch 'upstream/master' into rustup
2 parents ed892e7 + c3fb102 commit ac477db

File tree

751 files changed

+13764
-6992
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

751 files changed

+13764
-6992
lines changed

.github/workflows/clippy_changelog.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
- name: Check Changelog
2525
if: ${{ github.event_name == 'pull_request' }}
2626
run: |
27-
body=$(curl -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" -s "https://api.github.com/repos/rust-lang/rust-clippy/pulls/$PR_NUMBER" | \
27+
body=$(curl -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" -s "https://api.github.com/repos/${{ github.repository }}/pulls/$PR_NUMBER" | \
2828
python -c "import sys, json; print(json.load(sys.stdin)['body'])")
2929
output=$(awk '/^changelog:\s*\S/ && !/changelog: \[.*\]: your change/' <<< "$body" | sed "s/changelog:\s*//g")
3030
if [ -z "$output" ]; then

.github/workflows/clippy_mq.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ jobs:
6666
run: cargo test --features internal -- --skip dogfood
6767

6868
- name: Test clippy_lints
69-
run: cargo test --features internal
69+
run: cargo test
7070
working-directory: clippy_lints
7171

7272
- name: Test clippy_utils

.github/workflows/clippy_pr.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ jobs:
4242
run: cargo test --features internal
4343

4444
- name: Test clippy_lints
45-
run: cargo test --features internal
45+
run: cargo test
4646
working-directory: clippy_lints
4747

4848
- name: Test clippy_utils

.github/workflows/deploy.yml

+4
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ on:
88
tags:
99
- rust-1.**
1010

11+
concurrency:
12+
group: ${{ github.workflow }}
13+
cancel-in-progress: false
14+
1115
env:
1216
TARGET_BRANCH: 'gh-pages'
1317
SHA: '${{ github.sha }}'

.github/workflows/lintcheck.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ jobs:
6666

6767
- name: Run lintcheck
6868
if: steps.cache-json.outputs.cache-hit != 'true'
69-
run: ./target/debug/lintcheck --format json --all-lints --crates-toml ./lintcheck/ci_crates.toml
69+
run: env CLIPPY_CONF_DIR="$PWD/lintcheck/ci-config" ./target/debug/lintcheck --format json --all-lints --crates-toml ./lintcheck/ci_crates.toml
7070

7171
- name: Upload base JSON
7272
uses: actions/upload-artifact@v4
@@ -97,7 +97,7 @@ jobs:
9797
run: cargo build --manifest-path=lintcheck/Cargo.toml
9898

9999
- name: Run lintcheck
100-
run: ./target/debug/lintcheck --format json --all-lints --crates-toml ./lintcheck/ci_crates.toml
100+
run: env CLIPPY_CONF_DIR="$PWD/lintcheck/ci-config" ./target/debug/lintcheck --format json --all-lints --crates-toml ./lintcheck/ci_crates.toml
101101

102102
- name: Upload head JSON
103103
uses: actions/upload-artifact@v4

CHANGELOG.md

+67-3
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,68 @@ document.
66

77
## Unreleased / Beta / In Rust Nightly
88

9-
[609cd310...master](https://github.com/rust-lang/rust-clippy/compare/609cd310...master)
9+
[3e3715c3...master](https://github.com/rust-lang/rust-clippy/compare/3e3715c3...master)
10+
11+
## Rust 1.86
12+
13+
Current stable, released 2025-04-03
14+
15+
[View all 108 merged pull requests](https://github.com/rust-lang/rust-clippy/pulls?q=merged%3A2024-12-27T15%3A11%3A38Z..2025-02-06T13%3A57%3A58Z+base%3Amaster)
16+
17+
### New Lints
18+
19+
* Added [`unneeded_struct_pattern`] to `style` [#13465](https://github.com/rust-lang/rust-clippy/pull/13465)
20+
* Added [`doc_overindented_list_items`] to `style` [#13711](https://github.com/rust-lang/rust-clippy/pull/13711)
21+
* Added [`manual_ok_err`] to `complexity` [#13740](https://github.com/rust-lang/rust-clippy/pull/13740)
22+
* Added [`non_std_lazy_statics`] to `pedantic` [#13770](https://github.com/rust-lang/rust-clippy/pull/13770)
23+
* Added [`manual_repeat_n`] to `style` [#13858](https://github.com/rust-lang/rust-clippy/pull/13858)
24+
* Added [`manual_option_as_slice`] to `complexity` [#13901](https://github.com/rust-lang/rust-clippy/pull/13901)
25+
* Added [`double_ended_iterator_last`] to `perf` [#13922](https://github.com/rust-lang/rust-clippy/pull/13922)
26+
* Added [`useless_nonzero_new_unchecked`] to `complexity` [#13993](https://github.com/rust-lang/rust-clippy/pull/13993)
27+
* Added [`sliced_string_as_bytes`] to `perf` [#14002](https://github.com/rust-lang/rust-clippy/pull/14002)
28+
* Added [`unnecessary_semicolon`] to `pedantic` [#14032](https://github.com/rust-lang/rust-clippy/pull/14032)
29+
* Added [`return_and_then`] to `restriction` [#14051](https://github.com/rust-lang/rust-clippy/pull/14051)
30+
* Added [`manual_slice_fill`] to `style` [#14082](https://github.com/rust-lang/rust-clippy/pull/14082)
31+
* Added [`precedence_bits`] to `restriction` [#14115](https://github.com/rust-lang/rust-clippy/pull/14115)
32+
33+
### Moves and Deprecations
34+
35+
* Moved [`redundant_locals`] to `suspicious` (from `correctness`, now warn-by-default)
36+
[#13747](https://github.com/rust-lang/rust-clippy/pull/13747)
37+
* Moved [`format_push_string`] to `pedantic` (from `restriction`)
38+
[#13894](https://github.com/rust-lang/rust-clippy/pull/13894)
39+
* Moved [`format_collect`] to `pedantic` (from `perf`, now allow-by-default)
40+
[#13894](https://github.com/rust-lang/rust-clippy/pull/13894)
41+
* Moved [`mutex_integer`] to `restriction` (from `nursery`) [#14110](https://github.com/rust-lang/rust-clippy/pull/14110)
42+
43+
### Enhancements
44+
45+
* Add `lint-inconsistent-struct-field-initializers` configuration option to [`inconsistent_struct_constructor`]
46+
[#13737](https://github.com/rust-lang/rust-clippy/pull/13737)
47+
* [`len_zero`] now also triggers if deref target implements `is_empty()`
48+
[#13871](https://github.com/rust-lang/rust-clippy/pull/13871)
49+
* [`obfuscated_if_else`] now also triggers for the `.then(..).unwrap_or(..)` pattern
50+
[#14021](https://github.com/rust-lang/rust-clippy/pull/14021)
51+
52+
### False Positive Fixes
53+
54+
* [`trailing_empty_array`] no longer triggers in tests [#13844](https://github.com/rust-lang/rust-clippy/pull/13844)
55+
* [`missing_const_for_fn`] no longer triggers in tests [#13945](https://github.com/rust-lang/rust-clippy/pull/13945)
56+
* [`significant_drop_in_scrutinee`]: do not falsely warn for temporaries created by `.await` expansion
57+
[#13985](https://github.com/rust-lang/rust-clippy/pull/13985)
58+
59+
### ICE Fixes
60+
61+
* [`borrow_interior_mutable_const`] Fix an ICE that can occur when taking a reference to a tuple/`struct` field of an
62+
interior mutable `const` [#13877](https://github.com/rust-lang/rust-clippy/pull/13877)
63+
64+
### Others
65+
66+
* Clippy now uses Rust edition 2024 [#13751](https://github.com/rust-lang/rust-clippy/pull/13751)
1067

1168
## Rust 1.85
1269

13-
Current stable, released 2025-02-20
70+
Released 2025-02-20
1471

1572
[View all 72 merged pull requests](https://github.com/rust-lang/rust-clippy/pulls?q=merged%3A2024-11-15T19%3A31%3A08Z..2024-12-26T13%3A59%3A48Z+base%3Amaster)
1673

@@ -5516,6 +5573,7 @@ Released 2018-09-13
55165573
[`cast_slice_different_sizes`]: https://rust-lang.github.io/rust-clippy/master/index.html#cast_slice_different_sizes
55175574
[`cast_slice_from_raw_parts`]: https://rust-lang.github.io/rust-clippy/master/index.html#cast_slice_from_raw_parts
55185575
[`cfg_not_test`]: https://rust-lang.github.io/rust-clippy/master/index.html#cfg_not_test
5576+
[`char_indices_as_byte_indices`]: https://rust-lang.github.io/rust-clippy/master/index.html#char_indices_as_byte_indices
55195577
[`char_lit_as_u8`]: https://rust-lang.github.io/rust-clippy/master/index.html#char_lit_as_u8
55205578
[`chars_last_cmp`]: https://rust-lang.github.io/rust-clippy/master/index.html#chars_last_cmp
55215579
[`chars_next_cmp`]: https://rust-lang.github.io/rust-clippy/master/index.html#chars_next_cmp
@@ -5681,6 +5739,7 @@ Released 2018-09-13
56815739
[`if_same_then_else`]: https://rust-lang.github.io/rust-clippy/master/index.html#if_same_then_else
56825740
[`if_then_some_else_none`]: https://rust-lang.github.io/rust-clippy/master/index.html#if_then_some_else_none
56835741
[`ifs_same_cond`]: https://rust-lang.github.io/rust-clippy/master/index.html#ifs_same_cond
5742+
[`ignore_without_reason`]: https://rust-lang.github.io/rust-clippy/master/index.html#ignore_without_reason
56845743
[`ignored_unit_patterns`]: https://rust-lang.github.io/rust-clippy/master/index.html#ignored_unit_patterns
56855744
[`impl_hash_borrow_with_str_and_bytes`]: https://rust-lang.github.io/rust-clippy/master/index.html#impl_hash_borrow_with_str_and_bytes
56865745
[`impl_trait_in_params`]: https://rust-lang.github.io/rust-clippy/master/index.html#impl_trait_in_params
@@ -5783,12 +5842,14 @@ Released 2018-09-13
57835842
[`macro_metavars_in_unsafe`]: https://rust-lang.github.io/rust-clippy/master/index.html#macro_metavars_in_unsafe
57845843
[`macro_use_imports`]: https://rust-lang.github.io/rust-clippy/master/index.html#macro_use_imports
57855844
[`main_recursion`]: https://rust-lang.github.io/rust-clippy/master/index.html#main_recursion
5845+
[`manual_abs_diff`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_abs_diff
57865846
[`manual_assert`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_assert
57875847
[`manual_async_fn`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_async_fn
57885848
[`manual_bits`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_bits
57895849
[`manual_c_str_literals`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_c_str_literals
57905850
[`manual_clamp`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_clamp
57915851
[`manual_contains`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_contains
5852+
[`manual_dangling_ptr`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_dangling_ptr
57925853
[`manual_div_ceil`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_div_ceil
57935854
[`manual_filter`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_filter
57945855
[`manual_filter_map`]: https://rust-lang.github.io/rust-clippy/master/index.html#manual_filter_map
@@ -6055,6 +6116,7 @@ Released 2018-09-13
60556116
[`redundant_pub_crate`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pub_crate
60566117
[`redundant_slicing`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_slicing
60576118
[`redundant_static_lifetimes`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes
6119+
[`redundant_test_prefix`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_test_prefix
60586120
[`redundant_type_annotations`]: https://rust-lang.github.io/rust-clippy/master/index.html#redundant_type_annotations
60596121
[`ref_as_ptr`]: https://rust-lang.github.io/rust-clippy/master/index.html#ref_as_ptr
60606122
[`ref_binding_to_reference`]: https://rust-lang.github.io/rust-clippy/master/index.html#ref_binding_to_reference
@@ -6156,6 +6218,7 @@ Released 2018-09-13
61566218
[`suspicious_unary_op_formatting`]: https://rust-lang.github.io/rust-clippy/master/index.html#suspicious_unary_op_formatting
61576219
[`suspicious_xor_used_as_pow`]: https://rust-lang.github.io/rust-clippy/master/index.html#suspicious_xor_used_as_pow
61586220
[`swap_ptr_to_ref`]: https://rust-lang.github.io/rust-clippy/master/index.html#swap_ptr_to_ref
6221+
[`swap_with_temporary`]: https://rust-lang.github.io/rust-clippy/master/index.html#swap_with_temporary
61596222
[`tabs_in_doc_comments`]: https://rust-lang.github.io/rust-clippy/master/index.html#tabs_in_doc_comments
61606223
[`temporary_assignment`]: https://rust-lang.github.io/rust-clippy/master/index.html#temporary_assignment
61616224
[`temporary_cstring_as_ptr`]: https://rust-lang.github.io/rust-clippy/master/index.html#temporary_cstring_as_ptr
@@ -6346,6 +6409,7 @@ Released 2018-09-13
63466409
[`await-holding-invalid-types`]: https://doc.rust-lang.org/clippy/lint_configuration.html#await-holding-invalid-types
63476410
[`cargo-ignore-publish`]: https://doc.rust-lang.org/clippy/lint_configuration.html#cargo-ignore-publish
63486411
[`check-incompatible-msrv-in-tests`]: https://doc.rust-lang.org/clippy/lint_configuration.html#check-incompatible-msrv-in-tests
6412+
[`check-inconsistent-struct-field-initializers`]: https://doc.rust-lang.org/clippy/lint_configuration.html#check-inconsistent-struct-field-initializers
63496413
[`check-private-items`]: https://doc.rust-lang.org/clippy/lint_configuration.html#check-private-items
63506414
[`cognitive-complexity-threshold`]: https://doc.rust-lang.org/clippy/lint_configuration.html#cognitive-complexity-threshold
63516415
[`disallowed-macros`]: https://doc.rust-lang.org/clippy/lint_configuration.html#disallowed-macros
@@ -6362,7 +6426,7 @@ Released 2018-09-13
63626426
[`future-size-threshold`]: https://doc.rust-lang.org/clippy/lint_configuration.html#future-size-threshold
63636427
[`ignore-interior-mutability`]: https://doc.rust-lang.org/clippy/lint_configuration.html#ignore-interior-mutability
63646428
[`large-error-threshold`]: https://doc.rust-lang.org/clippy/lint_configuration.html#large-error-threshold
6365-
[`lint-inconsistent-struct-field-initializers`]: https://doc.rust-lang.org/clippy/lint_configuration.html#lint-inconsistent-struct-field-initializers
6429+
[`lint-commented-code`]: https://doc.rust-lang.org/clippy/lint_configuration.html#lint-commented-code
63666430
[`literal-representation-threshold`]: https://doc.rust-lang.org/clippy/lint_configuration.html#literal-representation-threshold
63676431
[`matches-for-let-else`]: https://doc.rust-lang.org/clippy/lint_configuration.html#matches-for-let-else
63686432
[`max-fn-params-bools`]: https://doc.rust-lang.org/clippy/lint_configuration.html#max-fn-params-bools

Cargo.toml

+12-4
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
[package]
22
name = "clippy"
33
# begin autogenerated version
4-
version = "0.1.87"
4+
version = "0.1.88"
55
# end autogenerated version
66
description = "A bunch of helpful lints to avoid common pitfalls in Rust"
77
repository = "https://github.com/rust-lang/rust-clippy"
@@ -27,6 +27,7 @@ clippy_config = { path = "clippy_config" }
2727
clippy_lints = { path = "clippy_lints" }
2828
clippy_utils = { path = "clippy_utils" }
2929
rustc_tools_util = { path = "rustc_tools_util", version = "0.4.2" }
30+
clippy_lints_internal = { path = "clippy_lints_internal", optional = true }
3031
tempfile = { version = "3.3", optional = true }
3132
termize = "0.1"
3233
color-print = "0.3.4"
@@ -43,7 +44,7 @@ walkdir = "2.3"
4344
filetime = "0.2.9"
4445
itertools = "0.12"
4546
pulldown-cmark = { version = "0.11", default-features = false, features = ["html"] }
46-
rinja = { version = "0.3", default-features = false, features = ["config"] }
47+
askama = { version = "0.13", default-features = false, features = ["alloc", "config", "derive"] }
4748

4849
# UI test dependencies
4950
clippy_utils = { path = "clippy_utils" }
@@ -58,8 +59,8 @@ tokio = { version = "1", features = ["io-util"] }
5859
rustc_tools_util = { path = "rustc_tools_util", version = "0.4.2" }
5960

6061
[features]
61-
integration = ["tempfile"]
62-
internal = ["clippy_lints/internal", "tempfile"]
62+
integration = ["dep:tempfile"]
63+
internal = ["dep:clippy_lints_internal", "dep:tempfile"]
6364

6465
[package.metadata.rust-analyzer]
6566
# This package uses #[feature(rustc_private)]
@@ -72,3 +73,10 @@ harness = false
7273
[[test]]
7374
name = "dogfood"
7475
harness = false
76+
77+
# quine-mc_cluskey makes up a significant part of the runtime in dogfood
78+
# due to the number of conditions in the clippy_lints crate
79+
# and enabling optimizations for that specific dependency helps a bit
80+
# without increasing total build times.
81+
[profile.dev.package.quine-mc_cluskey]
82+
opt-level = 3

rinja.toml renamed to askama.toml

File renamed without changes.

book/src/SUMMARY.md

+1
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
- [Updating the Changelog](development/infrastructure/changelog_update.md)
3131
- [Release a New Version](development/infrastructure/release.md)
3232
- [The Clippy Book](development/infrastructure/book.md)
33+
- [Benchmarking Clippy](development/infrastructure/benchmarking.md)
3334
- [Proposals](development/proposals/README.md)
3435
- [Roadmap 2021](development/proposals/roadmap-2021.md)
3536
- [Syntax Tree Patterns](development/proposals/syntax-tree-patterns.md)

book/src/development/adding_lints.md

+16-6
Original file line numberDiff line numberDiff line change
@@ -99,19 +99,22 @@ struct A;
9999
impl A {
100100
pub fn fo(&self) {}
101101
pub fn foo(&self) {}
102+
//~^ foo_functions
102103
pub fn food(&self) {}
103104
}
104105

105106
// Default trait methods
106107
trait B {
107108
fn fo(&self) {}
108109
fn foo(&self) {}
110+
//~^ foo_functions
109111
fn food(&self) {}
110112
}
111113

112114
// Plain functions
113115
fn fo() {}
114116
fn foo() {}
117+
//~^ foo_functions
115118
fn food() {}
116119

117120
fn main() {
@@ -122,17 +125,24 @@ fn main() {
122125
}
123126
```
124127

125-
Now we can run the test with `TESTNAME=foo_functions cargo uibless`, currently
126-
this test is meaningless though.
128+
Note that we are adding comment annotations with the name of our lint to mark
129+
lines where we expect an error. Except for very specific situations
130+
(`//@check-pass`), at least one error marker must be present in a test file for
131+
it to be accepted.
132+
133+
Once we have implemented our lint we can run `TESTNAME=foo_functions cargo
134+
uibless` to generate the `.stderr` file. If our lint makes use of structured
135+
suggestions then this command will also generate the corresponding `.fixed`
136+
file.
127137

128138
While we are working on implementing our lint, we can keep running the UI test.
129139
That allows us to check if the output is turning into what we want by checking the
130140
`.stderr` file that gets updated on every test run.
131141

132-
Running `TESTNAME=foo_functions cargo uitest` should pass on its own. When we
133-
commit our lint, we need to commit the generated `.stderr` files, too. In
134-
general, you should only commit files changed by `cargo bless` for the
135-
specific lint you are creating/editing.
142+
Once we have implemented our lint running `TESTNAME=foo_functions cargo uitest`
143+
should pass on its own. When we commit our lint, we need to commit the generated
144+
`.stderr` and if applicable `.fixed` files, too. In general, you should only
145+
commit files changed by `cargo bless` for the specific lint you are creating/editing.
136146

137147
> _Note:_ you can run multiple test files by specifying a comma separated list:
138148
> `TESTNAME=foo_functions,test2,test3`.

book/src/development/basics.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ following:
147147

148148
First, take note of the toolchain
149149
[override](https://rust-lang.github.io/rustup/overrides.html) in
150-
`/rust-toolchain`. We will use this override to install Clippy into the right
150+
`/rust-toolchain.toml`. We will use this override to install Clippy into the right
151151
toolchain.
152152

153153
> Tip: You can view the active toolchain for the current directory with `rustup

book/src/development/common_tools_writing_lints.md

+6-4
Original file line numberDiff line numberDiff line change
@@ -159,19 +159,21 @@ paths for Clippy can be found in [paths.rs][paths]
159159
To check if our type defines a method called `some_method`:
160160

161161
```rust
162-
use clippy_utils::ty::is_type_diagnostic_item;
163-
use clippy_utils::return_ty;
162+
use clippy_utils::ty::is_type_lang_item;
163+
use clippy_utils::{sym, return_ty};
164164

165165
impl<'tcx> LateLintPass<'tcx> for MyTypeImpl {
166166
fn check_impl_item(&mut self, cx: &LateContext<'tcx>, impl_item: &'tcx ImplItem<'_>) {
167167
// Check if item is a method/function
168168
if let ImplItemKind::Fn(ref signature, _) = impl_item.kind
169169
// Check the method is named `some_method`
170-
&& impl_item.ident.name.as_str() == "some_method"
170+
//
171+
// Add `some_method` to `clippy_utils::sym` if it's not already there
172+
&& impl_item.ident.name == sym::some_method
171173
// We can also check it has a parameter `self`
172174
&& signature.decl.implicit_self.has_implicit_self()
173175
// We can go further and even check if its return type is `String`
174-
&& is_type_diagnostic_item(cx, return_ty(cx, impl_item.hir_id), sym!(string_type))
176+
&& is_type_lang_item(cx, return_ty(cx, impl_item.hir_id), LangItem::String)
175177
{
176178
// ...
177179
}

book/src/development/defining_lints.md

+1-2
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ lint involves some boilerplate code.
99

1010
A lint type is the category of items and expressions in which your lint focuses on.
1111

12-
As of the writing of this documentation update, there are 12 _types_ of lints
12+
As of the writing of this documentation update, there are 11 _types_ of lints
1313
besides the numerous standalone lints living under `clippy_lints/src/`:
1414

1515
- `cargo`
@@ -23,7 +23,6 @@ besides the numerous standalone lints living under `clippy_lints/src/`:
2323
- `transmute`
2424
- `types`
2525
- `unit_types`
26-
- `utils / internal` (Clippy internal lints)
2726

2827
These types group together lints that share some common behaviors. For instance,
2928
`functions` groups together lints that deal with some aspects of functions in

0 commit comments

Comments
 (0)