On Wed, 2 Apr 2025 09:21:17 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> 
wrote:

>> Per Minborg has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - Add lazy toSting for StableMap::values
>>  - Make toString for reversed and sublist lazy
>
> src/java.base/share/classes/java/lang/StableValue.java line 378:
> 
>> 376:      * @param value to set
>> 377:      */
>> 378:     boolean trySet(T value);
> 
> This can probably throw `IllegalStateException` too right -- e.g. if used 
> inside a supplier passed to `orElseSet` ?

Nice catch.

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

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

Reply via email to