RFR: 8351565: Implement JEP 502: Stable Values (Preview) [v56]
Jorn Vernee
jvernee at openjdk.org
Fri Apr 18 15:20:17 UTC 2025
On Thu, 17 Apr 2025 10:47:03 GMT, Viktor Klang <vklang at openjdk.org> wrote:
>> Per Minborg has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Address comments on original vs underlying
>
> src/java.base/share/classes/java/lang/StableValue.java line 422:
>
>> 420: * As objects can be set via stable values but never removed, this can be a source
>> 421: * of unintended memory leaks. A stable value's content is
>> 422: * {@linkplain java.lang.ref##reachability strongly reachable}. Clients are
>
> I wonder if "Clients" is the best word here. Perhaps "Developers"?
I think you can drop the 'clients are advised that' prefix here. (I think usually it's 'X are advised to Y', and you would use 'be advised that X' as a general statement.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2050610207
More information about the core-libs-dev
mailing list