On Wed, 2 Apr 2025 14:42:17 GMT, Viktor Klang <vkl...@openjdk.org> wrote:

>> Per Minborg has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Add info that Map#values and Map#entrySet are stable
>
> src/java.base/share/classes/java/lang/StableValue.java line 567:
> 
>> 565:      * <p>
>> 566:      * The provided {@code original} supplier is guaranteed to be 
>> successfully invoked
>> 567:      * at most once even in a multi-threaded environment. Competing 
>> threads invoking the
> 
> There's a bit of a mix of "at most once" and "at-most-once". I'm fine with 
> either but I prefer uniformity in spelling :)

If "at most once" is an adjective then it should be spelled with dashes, 
otherwise not. 

"The method can be invoked at most once and hence It is an invoke-at-most-once 
method"

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2026509600

Reply via email to