Skip to content

Commit 55b5428

Browse files
authored
Simplify a conditional in collect_annotations
Also avoid excessive cloning
1 parent 9d73176 commit 55b5428

File tree

1 file changed

+11
-16
lines changed

1 file changed

+11
-16
lines changed

src/librustc_errors/emitter.rs

+11-16
Original file line numberDiff line numberDiff line change
@@ -1707,33 +1707,28 @@ impl FileWithAnnotatedLines {
17071707
hi.col_display += 1;
17081708
}
17091709

1710-
let ann_type = if lo.line != hi.line {
1710+
if lo.line != hi.line {
17111711
let ml = MultilineAnnotation {
17121712
depth: 1,
17131713
line_start: lo.line,
17141714
line_end: hi.line,
17151715
start_col: lo.col_display,
17161716
end_col: hi.col_display,
17171717
is_primary: span_label.is_primary,
1718-
label: span_label.label.clone(),
1718+
label: span_label.label,
17191719
overlaps_exactly: false,
17201720
};
1721-
multiline_annotations.push((lo.file.clone(), ml.clone()));
1722-
AnnotationType::Multiline(ml)
1721+
multiline_annotations.push((lo.file, ml));
17231722
} else {
1724-
AnnotationType::Singleline
1725-
};
1726-
let ann = Annotation {
1727-
start_col: lo.col_display,
1728-
end_col: hi.col_display,
1729-
is_primary: span_label.is_primary,
1730-
label: span_label.label.clone(),
1731-
annotation_type: ann_type,
1732-
};
1733-
1734-
if !ann.is_multiline() {
1723+
let ann = Annotation {
1724+
start_col: lo.col_display,
1725+
end_col: hi.col_display,
1726+
is_primary: span_label.is_primary,
1727+
label: span_label.label,
1728+
annotation_type: AnnotationType::Singleline,
1729+
};
17351730
add_annotation_to_file(&mut output, lo.file, lo.line, ann);
1736-
}
1731+
};
17371732
}
17381733
}
17391734

0 commit comments

Comments
 (0)