Skip to content

Commit 1edd63f

Browse files
authored
Rollup merge of #103953 - TaKO8Ki:remove-unused-arg-from-throw_unresolved_import_error, r=oli-obk
Remove unused argument from `throw_unresolved_import_error` `throw_unresolved_import_error` does not need the second argument.
2 parents ee3c9f7 + 97ddc63 commit 1edd63f

File tree

1 file changed

+10
-18
lines changed

1 file changed

+10
-18
lines changed

compiler/rustc_resolve/src/imports.rs

+10-18
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
473473
{
474474
// In the case of a new import line, throw a diagnostic message
475475
// for the previous line.
476-
self.throw_unresolved_import_error(errors, None);
476+
self.throw_unresolved_import_error(errors);
477477
errors = vec![];
478478
}
479479
if seen_spans.insert(err.span) {
@@ -505,29 +505,21 @@ impl<'a, 'b> ImportResolver<'a, 'b> {
505505
}
506506

507507
if !errors.is_empty() {
508-
self.throw_unresolved_import_error(errors, None);
508+
self.throw_unresolved_import_error(errors);
509509
}
510510
}
511511

512-
fn throw_unresolved_import_error(
513-
&self,
514-
errors: Vec<(String, UnresolvedImportError)>,
515-
span: Option<MultiSpan>,
516-
) {
512+
fn throw_unresolved_import_error(&self, errors: Vec<(String, UnresolvedImportError)>) {
513+
if errors.is_empty() {
514+
return;
515+
}
516+
517517
/// Upper limit on the number of `span_label` messages.
518518
const MAX_LABEL_COUNT: usize = 10;
519519

520-
let (span, msg) = if errors.is_empty() {
521-
(span.unwrap(), "unresolved import".to_string())
522-
} else {
523-
let span = MultiSpan::from_spans(errors.iter().map(|(_, err)| err.span).collect());
524-
525-
let paths = errors.iter().map(|(path, _)| format!("`{}`", path)).collect::<Vec<_>>();
526-
527-
let msg = format!("unresolved import{} {}", pluralize!(paths.len()), paths.join(", "),);
528-
529-
(span, msg)
530-
};
520+
let span = MultiSpan::from_spans(errors.iter().map(|(_, err)| err.span).collect());
521+
let paths = errors.iter().map(|(path, _)| format!("`{}`", path)).collect::<Vec<_>>();
522+
let msg = format!("unresolved import{} {}", pluralize!(paths.len()), paths.join(", "),);
531523

532524
let mut diag = struct_span_err!(self.r.session, span, E0432, "{}", &msg);
533525

0 commit comments

Comments
 (0)