On Mon, 10 Mar 2025 18:11:23 GMT, Per Minborg <pminb...@openjdk.org> wrote:
> Implement JEP 502.
>
> The PR passes tier1-tier3 tests.
src/java.base/share/classes/java/util/ImmutableCollections.java line 772:
> 770:
> 771: @jdk.internal.ValueBased
> 772: static final class StableList<E> extends AbstractImmutableList<E> {
Is there significant reuse gained by putting StableList in ImmutableCollection?
The back-and-forth between here and SV through SharedSecrets is a little
awkward.
src/java.base/share/classes/java/util/ImmutableCollections.java line 1462:
> 1460:
> 1461: static final class StableMap<K, V>
> 1462: extends AbstractImmutableMap<K, V> {
Same question about whether StableMap needs to go here. Though there’s more
stuff going on for maps than lists here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r1988220230
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r1988222063
