tamer: diagnose::report::Section::maybe_squash_into: Remove syslabel TODO

Previously, when adjacent duplicate spans were both resolved, if one failed,
the other certainly would, which would result in duplicate labels each
squash.  Elided spans do not have syslabels, and so this is no longer a
concern.

DEV-12151
main
Mike Gerwitz 2022-04-29 13:07:51 -04:00
parent 2ae6df38e7
commit 3e04217741
1 changed files with 2 additions and 3 deletions

View File

@ -310,9 +310,8 @@ impl<'s, 'd> Section<'d> {
) -> Option<Self> {
match extend {
Some(extend_sec) if self.span == extend_sec.span => {
// TODO: At the time of writing this will cause duplication of
// system labels,
// which is not desirable.
// Note that system labels shouldn't exist for elided spans
// and so they should not be duplicated when squashing.
extend_sec.body.extend(
self.body
.into_iter()