bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

refactor(compiler/resolve): simplify some code

Browse source 
Removes unnecessary allocates and double-sorting the same vector,
makes the code a little nicer.
This commit is contained in:
Vitaliy Busko 2023-10-19 04:24:35 +07:00
parent cc705b8012
commit e68edb89ad
No known key found for this signature in database
GPG key ID: 8C1BC6C6D5747572
1 changed files with 10 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

21
compiler/rustc_resolve/src/check_unused.rs
View file

@ -338,7 +338,7 @@ impl Resolver<'_, '_> {
for unused in visitor.unused_imports.values() { for unused in visitor.unused_imports.values() {
let mut fixes = Vec::new(); let mut fixes = Vec::new();
let mut spans = match calc_unused_spans(unused, unused.use_tree, unused.use_tree_id) { let spans = match calc_unused_spans(unused, unused.use_tree, unused.use_tree_id) {
UnusedSpanResult::Used => continue, UnusedSpanResult::Used => continue,
UnusedSpanResult::FlatUnused(span, remove) => { UnusedSpanResult::FlatUnused(span, remove) => {
fixes.push((remove, String::new())); fixes.push((remove, String::new()));
@ -356,20 +356,19 @@ impl Resolver<'_, '_> {
} }
}; };
let len = spans.len(); let ms = MultiSpan::from_spans(spans);
spans.sort();
let ms = MultiSpan::from_spans(spans.clone()); let mut span_snippets = ms
let mut span_snippets = spans .primary_spans()
.iter() .iter()
.filter_map(|s| match tcx.sess.source_map().span_to_snippet(*s) { .filter_map(|span| tcx.sess.source_map().span_to_snippet(*span).ok())
Ok(s) => Some(format!("`{s}`")), .map(|s| format!("`{s}`"))
_ => None,
})
.collect::<Vec<String>>(); .collect::<Vec<String>>();
span_snippets.sort(); span_snippets.sort();
let msg = format!( let msg = format!(
"unused import{}{}", "unused import{}{}",
pluralize!(len), pluralize!(ms.primary_spans().len()),
if !span_snippets.is_empty() { if !span_snippets.is_empty() {
format!(": {}", span_snippets.join(", ")) format!(": {}", span_snippets.join(", "))
} else { } else {
@ -379,7 +378,7 @@ impl Resolver<'_, '_> {
let fix_msg = if fixes.len() == 1 && fixes[0].0 == unused.item_span { let fix_msg = if fixes.len() == 1 && fixes[0].0 == unused.item_span {
"remove the whole `use` item" "remove the whole `use` item"
} else if spans.len() > 1 { } else if ms.primary_spans().len() > 1 {
"remove the unused imports" "remove the unused imports"
} else { } else {
"remove the unused import" "remove the unused import"