GP-5522: Order LifeSet.spans()

This commit is contained in:
Dan 2025-03-26 19:55:39 +00:00
parent 83098ce2b5
commit 97e6205802

View file

@ -411,7 +411,7 @@ public interface Span<N, S extends Span<N, S>> extends Comparable<S> {
* *
* @return the set of spans * @return the set of spans
*/ */
Set<S> spans(); NavigableSet<S> spans();
/** /**
* Get the values in this map * Get the values in this map
@ -563,7 +563,7 @@ public interface Span<N, S extends Span<N, S>> extends Comparable<S> {
* *
* @return the iterable * @return the iterable
*/ */
Iterable<S> spans(); NavigableSet<S> spans();
/** /**
* Get a span which encloses all spans in the set * Get a span which encloses all spans in the set
@ -862,9 +862,12 @@ public interface Span<N, S extends Span<N, S>> extends Comparable<S> {
} }
@Override @Override
public Set<S> spans() { public NavigableSet<S> spans() {
// TODO: Make this a view? // TODO: Make this a view?
return spanTree.values().stream().map(e -> e.getKey()).collect(Collectors.toSet()); return spanTree.values()
.stream()
.map(e -> e.getKey())
.collect(Collectors.toCollection(TreeSet::new));
} }
@Override @Override
@ -1027,7 +1030,7 @@ public interface Span<N, S extends Span<N, S>> extends Comparable<S> {
} }
@Override @Override
public Iterable<S> spans() { public NavigableSet<S> spans() {
return map.spans(); return map.spans();
} }