Skip to content

Commit 74ed601

Browse files
chenyukangchenyukang
committed
Fix redundant import errors for preload extern crate
1 parent bdde2a8 commit 74ed601

17 files changed

+113
-16
lines changed

‎compiler/rustc_lint/src/context/diagnostics.rs

+12-2
Original file line numberDiff line numberDiff line change
@@ -103,11 +103,21 @@ pub(super) fn builtin(sess: &Session, diagnostic: BuiltinLintDiag, diag: &mut Di
103103
);
104104
}
105105
}
106-
BuiltinLintDiag::RedundantImport(spans, ident) => {
106+
BuiltinLintDiag::RedundantImport(spans, ident, import_span) => {
107107
for (span, is_imported) in spans {
108108
let introduced = if is_imported { "imported" } else { "defined" };
109-
diag.span_label(span, format!("the item `{ident}` is already {introduced} here"));
109+
let span_msg = if span.is_dummy() { "by prelude" } else { "here" };
110+
diag.span_label(
111+
span,
112+
format!("the item `{ident}` is already {introduced} {span_msg}"),
113+
);
110114
}
115+
diag.span_suggestion(
116+
import_span,
117+
"remove this import",
118+
"",
119+
Applicability::MachineApplicable,
120+
);
111121
}
112122
BuiltinLintDiag::DeprecatedMacro(suggestion, span) => {
113123
stability::deprecation_suggestion(diag, "macro", suggestion, span)

‎compiler/rustc_lint_defs/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -577,7 +577,7 @@ pub enum BuiltinLintDiag {
577577
ElidedLifetimesInPaths(usize, Span, bool, Span),
578578
UnknownCrateTypes(Span, String, String),
579579
UnusedImports(String, Vec<(Span, String)>, Option<Span>),
580-
RedundantImport(Vec<(Span, bool)>, Ident),
580+
RedundantImport(Vec<(Span, bool)>, Ident, Span),
581581
DeprecatedMacro(Option<Symbol>, Span),
582582
MissingAbi(Span, Abi),
583583
UnusedDocComment(Span),

‎compiler/rustc_resolve/src/imports.rs

+1-3
Original file line numberDiff line numberDiff line change
@@ -1334,9 +1334,7 @@ impl<'a, 'tcx> Resolver<'a, 'tcx> {
13341334
}
13351335

13361336
let mut is_redundant = true;
1337-
13381337
let mut redundant_span = PerNS { value_ns: None, type_ns: None, macro_ns: None };
1339-
13401338
self.per_ns(|this, ns| {
13411339
if is_redundant && let Ok(binding) = source_bindings[ns].get() {
13421340
if binding.res() == Res::Err {
@@ -1373,7 +1371,7 @@ impl<'a, 'tcx> Resolver<'a, 'tcx> {
13731371
id,
13741372
import.span,
13751373
format!("the item `{source}` is imported redundantly"),
1376-
BuiltinLintDiag::RedundantImport(redundant_spans, source),
1374+
BuiltinLintDiag::RedundantImport(redundant_spans, source, import.use_span),
13771375
);
13781376
}
13791377
}

‎tests/ui/imports/auxiliary/aux-issue-121915.rs

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
pub fn item() {}

‎tests/ui/imports/redundant-import-issue-121915-2015.rs

+11
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
//@ compile-flags: --extern aux_issue_121915 --edition 2015
2+
//@ aux-build: aux-issue-121915.rs
3+
4+
extern crate aux_issue_121915;
5+
6+
#[deny(unused_imports)]
7+
fn main() {
8+
use aux_issue_121915;
9+
//~^ ERROR the item `aux_issue_121915` is imported redundantly
10+
aux_issue_121915::item();
11+
}

‎tests/ui/imports/redundant-import-issue-121915-2015.stderr

+17
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
error: the item `aux_issue_121915` is imported redundantly
2+
--> $DIR/redundant-import-issue-121915-2015.rs:8:9
3+
|
4+
LL | extern crate aux_issue_121915;
5+
| ------------------------------ the item `aux_issue_121915` is already imported here
6+
...
7+
LL | use aux_issue_121915;
8+
| ----^^^^^^^^^^^^^^^^- help: remove this import
9+
|
10+
note: the lint level is defined here
11+
--> $DIR/redundant-import-issue-121915-2015.rs:6:8
12+
|
13+
LL | #[deny(unused_imports)]
14+
| ^^^^^^^^^^^^^^
15+
16+
error: aborting due to 1 previous error
17+

‎tests/ui/imports/redundant-import-issue-121915.rs

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
//@ compile-flags: --extern aux_issue_121915 --edition 2018
2+
//@ aux-build: aux-issue-121915.rs
3+
4+
#[deny(unused_imports)]
5+
fn main() {
6+
use aux_issue_121915;
7+
//~^ ERROR the item `aux_issue_121915` is imported redundantly
8+
aux_issue_121915::item();
9+
}

‎tests/ui/imports/redundant-import-issue-121915.stderr

+17
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
error: the item `aux_issue_121915` is imported redundantly
2+
--> $DIR/redundant-import-issue-121915.rs:6:9
3+
|
4+
LL | use aux_issue_121915;
5+
| ----^^^^^^^^^^^^^^^^-
6+
| | |
7+
| | the item `aux_issue_121915` is already defined by prelude
8+
| help: remove this import
9+
|
10+
note: the lint level is defined here
11+
--> $DIR/redundant-import-issue-121915.rs:4:8
12+
|
13+
LL | #[deny(unused_imports)]
14+
| ^^^^^^^^^^^^^^
15+
16+
error: aborting due to 1 previous error
17+

‎tests/ui/imports/suggest-remove-issue-121315.fixed

+8
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
//@ run-rustfix
2+
//@ compile-flags: --edition 2021
3+
#[deny(unused_imports)]
4+
//~^ ERROR the item `TryFrom` is imported redundantly
5+
6+
fn main() {
7+
let _ = u32::try_from(5i32);
8+
}

‎tests/ui/imports/suggest-remove-issue-121315.rs

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
//@ run-rustfix
2+
//@ compile-flags: --edition 2021
3+
#[deny(unused_imports)]
4+
use std::convert::TryFrom;
5+
//~^ ERROR the item `TryFrom` is imported redundantly
6+
7+
fn main() {
8+
let _ = u32::try_from(5i32);
9+
}

‎tests/ui/imports/suggest-remove-issue-121315.stderr

+17
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
error: the item `TryFrom` is imported redundantly
2+
--> $DIR/suggest-remove-issue-121315.rs:4:5
3+
|
4+
LL | use std::convert::TryFrom;
5+
| ----^^^^^^^^^^^^^^^^^^^^^- help: remove this import
6+
--> $SRC_DIR/std/src/prelude/mod.rs:LL:COL
7+
|
8+
= note: the item `TryFrom` is already defined here
9+
|
10+
note: the lint level is defined here
11+
--> $DIR/suggest-remove-issue-121315.rs:3:8
12+
|
13+
LL | #[deny(unused_imports)]
14+
| ^^^^^^^^^^^^^^
15+
16+
error: aborting due to 1 previous error
17+

‎tests/ui/lint/unused/issue-59896.stderr

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ LL | struct S;
55
| --------- the item `S` is already defined here
66
...
77
LL | use S;
8-
| ^
8+
| ----^- help: remove this import
99
|
1010
note: the lint level is defined here
1111
--> $DIR/issue-59896.rs:1:9

‎tests/ui/lint/use-redundant/use-redundant-glob-parent.stderr

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ LL | use bar::*;
55
| ------ the item `Foo` is already imported here
66
...
77
LL | use bar::Foo;
8-
| ^^^^^^^^
8+
| ----^^^^^^^^- help: remove this import
99
|
1010
note: the lint level is defined here
1111
--> $DIR/use-redundant-glob-parent.rs:2:9

‎tests/ui/lint/use-redundant/use-redundant-glob.stderr

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ warning: the item `Foo` is imported redundantly
44
LL | use bar::*;
55
| ------ the item `Foo` is already imported here
66
LL | use bar::Foo;
7-
| ^^^^^^^^
7+
| ----^^^^^^^^- help: remove this import
88
|
99
note: the lint level is defined here
1010
--> $DIR/use-redundant-glob.rs:2:9

‎tests/ui/lint/use-redundant/use-redundant-prelude-rust-2015.stderr

+4-4
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ warning: the item `Some` is imported redundantly
22
--> $DIR/use-redundant-prelude-rust-2015.rs:5:5
33
|
44
LL | use std::option::Option::Some;
5-
| ^^^^^^^^^^^^^^^^^^^^^^^^^
5+
| ----^^^^^^^^^^^^^^^^^^^^^^^^^- help: remove this import
66
--> $SRC_DIR/std/src/prelude/mod.rs:LL:COL
77
|
88
= note: the item `Some` is already defined here
@@ -17,7 +17,7 @@ warning: the item `None` is imported redundantly
1717
--> $DIR/use-redundant-prelude-rust-2015.rs:6:5
1818
|
1919
LL | use std::option::Option::None;
20-
| ^^^^^^^^^^^^^^^^^^^^^^^^^
20+
| ----^^^^^^^^^^^^^^^^^^^^^^^^^- help: remove this import
2121
--> $SRC_DIR/std/src/prelude/mod.rs:LL:COL
2222
|
2323
= note: the item `None` is already defined here
@@ -26,7 +26,7 @@ warning: the item `Ok` is imported redundantly
2626
--> $DIR/use-redundant-prelude-rust-2015.rs:8:5
2727
|
2828
LL | use std::result::Result::Ok;
29-
| ^^^^^^^^^^^^^^^^^^^^^^^
29+
| ----^^^^^^^^^^^^^^^^^^^^^^^- help: remove this import
3030
--> $SRC_DIR/std/src/prelude/mod.rs:LL:COL
3131
|
3232
= note: the item `Ok` is already defined here
@@ -35,7 +35,7 @@ warning: the item `Err` is imported redundantly
3535
--> $DIR/use-redundant-prelude-rust-2015.rs:9:5
3636
|
3737
LL | use std::result::Result::Err;
38-
| ^^^^^^^^^^^^^^^^^^^^^^^^
38+
| ----^^^^^^^^^^^^^^^^^^^^^^^^- help: remove this import
3939
--> $SRC_DIR/std/src/prelude/mod.rs:LL:COL
4040
|
4141
= note: the item `Err` is already defined here

‎tests/ui/lint/use-redundant/use-redundant-prelude-rust-2021.stderr

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ warning: the item `TryFrom` is imported redundantly
22
--> $DIR/use-redundant-prelude-rust-2021.rs:5:5
33
|
44
LL | use std::convert::TryFrom;
5-
| ^^^^^^^^^^^^^^^^^^^^^
5+
| ----^^^^^^^^^^^^^^^^^^^^^- help: remove this import
66
--> $SRC_DIR/std/src/prelude/mod.rs:LL:COL
77
|
88
= note: the item `TryFrom` is already defined here
@@ -17,7 +17,7 @@ warning: the item `TryInto` is imported redundantly
1717
--> $DIR/use-redundant-prelude-rust-2021.rs:6:5
1818
|
1919
LL | use std::convert::TryInto;
20-
| ^^^^^^^^^^^^^^^^^^^^^
20+
| ----^^^^^^^^^^^^^^^^^^^^^- help: remove this import
2121
--> $SRC_DIR/std/src/prelude/mod.rs:LL:COL
2222
|
2323
= note: the item `TryInto` is already defined here

‎tests/ui/lint/use-redundant/use-redundant.stderr

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ LL | use crate::foo::Bar;
2323
| --------------- the item `Bar` is already imported here
2424
...
2525
LL | use crate::foo::Bar;
26-
| ^^^^^^^^^^^^^^^
26+
| ----^^^^^^^^^^^^^^^- help: remove this import
2727

2828
warning: 3 warnings emitted
2929

0 commit comments

Comments
 (0)