Skip to content

Commit 23f52db

Browse files
chenyukangchenyukang
committed
Fix redundant import errors for preload extern crate
1 parent 0decdac commit 23f52db

17 files changed

+124
-27
lines changed

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

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,21 @@ pub(super) fn builtin(sess: &Session, diagnostic: BuiltinLintDiagnostics, diag:
108108
);
109109
}
110110
}
111-
BuiltinLintDiagnostics::RedundantImport(spans, ident) => {
111+
BuiltinLintDiagnostics::RedundantImport(spans, ident, import_span) => {
112112
for (span, is_imported) in spans {
113113
let introduced = if is_imported { "imported" } else { "defined" };
114-
diag.span_label(span, format!("the item `{ident}` is already {introduced} here"));
114+
let span_msg = if span.is_dummy() { "by prelude" } else { "here" };
115+
diag.span_label(
116+
span,
117+
format!("the item `{ident}` is already {introduced} {span_msg}"),
118+
);
115119
}
120+
diag.span_suggestion(
121+
import_span,
122+
"remove this import",
123+
"",
124+
Applicability::MachineApplicable,
125+
);
116126
}
117127
BuiltinLintDiagnostics::DeprecatedMacro(suggestion, span) => {
118128
stability::deprecation_suggestion(diag, "macro", suggestion, span)

‎compiler/rustc_lint_defs/src/lib.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -577,7 +577,7 @@ pub enum BuiltinLintDiagnostics {
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‎

Lines changed: 2 additions & 4 deletions
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 {
@@ -1371,9 +1369,9 @@ impl<'a, 'tcx> Resolver<'a, 'tcx> {
13711369
self.lint_buffer.buffer_lint_with_diagnostic(
13721370
UNUSED_IMPORTS,
13731371
id,
1374-
import.span,
1372+
import.use_span,
13751373
format!("the item `{source}` is imported redundantly"),
1376-
BuiltinLintDiagnostics::RedundantImport(redundant_spans, source),
1374+
BuiltinLintDiagnostics::RedundantImport(redundant_spans, source, import.use_span),
13771375
);
13781376
}
13791377
}

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
pub fn item() {}

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

Lines changed: 11 additions & 0 deletions
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‎

Lines changed: 17 additions & 0 deletions
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:5
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‎

Lines changed: 9 additions & 0 deletions
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‎

Lines changed: 17 additions & 0 deletions
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:5
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‎

Lines changed: 8 additions & 0 deletions
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‎

Lines changed: 9 additions & 0 deletions
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+
}

0 commit comments

Comments
 (0)