RFR: 8356080: Address post-integration comments for Stable Values [v2]
Per Minborg
pminborg at openjdk.org
Mon May 12 07:16:59 UTC 2025
On Fri, 9 May 2025 14:55:06 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Per Minborg has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 384 commits:
>>
>> - Fix an issue with toString on nested constructs
>> - Merge branch 'master' into jep502-followup
>> - Merge branch 'master' into jep502-followup
>> - Update src/java.base/share/classes/java/lang/StableValue.java
>>
>> Co-authored-by: Chen Liang <liach at openjdk.org>
>> - Simplify furhter
>> - Address comments in PR
>> - Merge master
>> - Remove unused method and add comment
>> - Address comments
>> - Merge branch 'master' into jep502-followup
>> - ... and 374 more: https://git.openjdk.org/jdk/compare/900b3ff7...3eebd504
>
> src/java.base/share/classes/java/util/ReverseOrderListView.java line 46:
>
>> 44: final List<E> base;
>> 45: @Stable
>> 46: final boolean modifiable;
>
> Unfortunately, stable on boolean only works for true - wish we can get trusted or strict final soon :)
We could change it to `Boolean`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25004#discussion_r2083985618
More information about the core-libs-dev
mailing list