RFR: 8357272: Add overrides for all default methods to wrapper collections

[Jaikiran Pai]

On Wed, 28 May 2025 03:03:45 GMT, Stuart Marks <smarks at 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 1052:

> 1050:      * are intended to override but are declared improperly will be detected through testing.
> 1051:      * (The @Override annotation still occurs in several places in this file. These will be
> 1052:      * removed over time.)

Nit - The last sentence, I think, isn't necessary.

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

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