On Wed, 28 May 2025 03:03:45 GMT, Stuart Marks <sma...@openjdk.org> wrote:

> This mostly just adds overrides to the wrapper implementations in 
> java.util.Collections. However, in order to satisfy the test, some overrides 
> are also added to ReverseOrderSortedMapView, which is used by the SortedMap 
> wrapper keySet, values, and entrySet views. These overrides don't do anything 
> except to call super, so they're arguably dead code. However, they enable all 
> combinations of sequenced map views to be tested, which is valuable.

src/java.base/share/classes/java/util/Collections.java line 1407:

> 1405:         public E removeFirst()             {throw new 
> UnsupportedOperationException();}
> 1406:         public E removeLast()              {throw new 
> UnsupportedOperationException();}
> 1407: 

Is this `UnmodifiableSortedSet` missing an override for the `Spliterator<E> 
spliterator()` default method that's on `SortedSet`?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25478#discussion_r2189950431

Reply via email to