On Wed, 2 Apr 2025 18:33:22 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
>
> test/jdk/java/lang/StableValue/StableFunctionTest.java line 136:
>
>> 134: assertTrue(cached.toString().contains(Value.THIRTEEN +
>> "=.unset"));
>> 135: assertTrue(cached.toString().contains(Value.FORTY_TWO +
>> "=.unset"), cached.toString());
>> 136: assertTrue(cached.toString().endsWith("}"));
>
> Not possible to test full toString() representation?
The order in which mappings appear is unspecified, unfortunately.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2026610510
