Mirrors/rust-clippy
2
0
Fork 0
mirror of https://github.com/rust-lang/rust-clippy synced 2024-12-12 06:12:46 +00:00
Code Issues Projects Releases Packages Wiki Activity

Improve redundant_locals help message

Browse source
This commit is contained in:
koka 2023-10-06 22:18:11 +09:00
parent 9554e477c2
commit 48d2770e52
No known key found for this signature in database
GPG key ID: A5917A40697774CD
2 changed files with 86 additions and 62 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
clippy_lints/src/redundant_locals.rs
View file

@ -74,10 +74,10 @@ impl<'tcx> LateLintPass<'tcx> for RedundantLocals {
span_lint_and_help( span_lint_and_help(
cx, cx,
REDUNDANT_LOCALS, REDUNDANT_LOCALS,
vec![binding_pat.span, local.span], local.span,
"redundant redefinition of a binding", &format!("redundant redefinition of a binding `{ident}`"),
None, Some(binding_pat.span),
&format!("remove the redefinition of `{ident}`"), &format!("`{ident}` is initially defined here"),
); );
} }
} }

140
tests/ui/redundant_locals.stderr
View file

@ -1,148 +1,172 @@
error: redundant redefinition of a binding error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:12:5
|
LL | let x = x;
| ^^^^^^^^^^
|
help: `x` is initially defined here
--> $DIR/redundant_locals.rs:11:9 --> $DIR/redundant_locals.rs:11:9
| |
LL | let x = 1; LL | let x = 1;
| ^ | ^
LL | let x = x;
| ^^^^^^^^^^
|
= help: remove the redefinition of `x`
= note: `-D clippy::redundant-locals` implied by `-D warnings` = note: `-D clippy::redundant-locals` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::redundant_locals)]` = help: to override `-D warnings` add `#[allow(clippy::redundant_locals)]`
error: redundant redefinition of a binding error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:17:5
|
LL | let mut x = x;
| ^^^^^^^^^^^^^^
|
help: `x` is initially defined here
--> $DIR/redundant_locals.rs:16:9 --> $DIR/redundant_locals.rs:16:9
| |
LL | let mut x = 1; LL | let mut x = 1;
| ^^^^^ | ^^^^^
LL | let mut x = x;
| ^^^^^^^^^^^^^^
|
= help: remove the redefinition of `x`
error: redundant redefinition of a binding error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:47:5
|
LL | let x = x;
| ^^^^^^^^^^
|
help: `x` is initially defined here
--> $DIR/redundant_locals.rs:46:14 --> $DIR/redundant_locals.rs:46:14
| |
LL | fn parameter(x: i32) { LL | fn parameter(x: i32) {
| ^ | ^
error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:52:5
|
LL | let x = x; LL | let x = x;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `x` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:51:9 --> $DIR/redundant_locals.rs:51:9
| |
LL | let x = 1; LL | let x = 1;
| ^ | ^
error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:53:5
|
LL | let x = x; LL | let x = x;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `x` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:52:9 --> $DIR/redundant_locals.rs:52:9
| |
LL | let x = x; LL | let x = x;
| ^ | ^
error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:54:5
|
LL | let x = x; LL | let x = x;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `x` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:53:9 --> $DIR/redundant_locals.rs:53:9
| |
LL | let x = x; LL | let x = x;
| ^ | ^
error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:55:5
|
LL | let x = x; LL | let x = x;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `x` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:54:9 --> $DIR/redundant_locals.rs:54:9
| |
LL | let x = x; LL | let x = x;
| ^ | ^
LL | let x = x;
error: redundant redefinition of a binding `a`
--> $DIR/redundant_locals.rs:61:5
|
LL | let a = a;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `a` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:59:9 --> $DIR/redundant_locals.rs:59:9
| |
LL | let a = 1; LL | let a = 1;
| ^ | ^
LL | let b = 2;
LL | let a = a; error: redundant redefinition of a binding `b`
--> $DIR/redundant_locals.rs:62:5
|
LL | let b = b;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `a` help: `b` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:60:9 --> $DIR/redundant_locals.rs:60:9
| |
LL | let b = 2; LL | let b = 2;
| ^ | ^
LL | let a = a;
LL | let b = b;
| ^^^^^^^^^^
|
= help: remove the redefinition of `b`
error: redundant redefinition of a binding error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:68:9
|
LL | let x = x;
| ^^^^^^^^^^
|
help: `x` is initially defined here
--> $DIR/redundant_locals.rs:67:13 --> $DIR/redundant_locals.rs:67:13
| |
LL | let x = 1; LL | let x = 1;
| ^ | ^
error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:75:9
|
LL | let x = x; LL | let x = x;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `x` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:74:13 --> $DIR/redundant_locals.rs:74:13
| |
LL | let x = 1; LL | let x = 1;
| ^ | ^
error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:78:9
|
LL | let x = x; LL | let x = x;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `x` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:77:6 --> $DIR/redundant_locals.rs:77:6
| |
LL | |x: i32| { LL | |x: i32| {
| ^ | ^
error: redundant redefinition of a binding `x`
--> $DIR/redundant_locals.rs:97:9
|
LL | let x = x; LL | let x = x;
| ^^^^^^^^^^ | ^^^^^^^^^^
| |
= help: remove the redefinition of `x` help: `x` is initially defined here
error: redundant redefinition of a binding
--> $DIR/redundant_locals.rs:94:9 --> $DIR/redundant_locals.rs:94:9
| |
LL | let x = 1; LL | let x = 1;
| ^ | ^
...
LL | let x = x;
| ^^^^^^^^^^
|
= help: remove the redefinition of `x`
error: redundant redefinition of a binding error: redundant redefinition of a binding `a`
--> $DIR/redundant_locals.rs:144:5
|
LL | let a = a;
| ^^^^^^^^^^
|
help: `a` is initially defined here
--> $DIR/redundant_locals.rs:142:9 --> $DIR/redundant_locals.rs:142:9
| |
LL | let a = WithoutDrop(1); LL | let a = WithoutDrop(1);
| ^ | ^
LL | let b = WithoutDrop(2);
LL | let a = a;
| ^^^^^^^^^^
|
= help: remove the redefinition of `a`
error: aborting due to 14 previous errors error: aborting due to 14 previous errors