On Tue, 11 Mar 2025 01:20:16 GMT, Johannes Graham <d...@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.
This allows reuse of `AbstractImmutableList` with list iterators, sub lists and
more.
> 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.
Same argument as for `StableList`. This allows reuse of several classes
including `AbstractImmutableMap`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r1988605709
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r1988607732
