On Wed, 2 Apr 2025 18:33:39 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 147:
>
>> 145: cached.apply(Value.FORTY_TWO);
>> 146: String toString = cached.toString();
>> 147: assertTrue(toString.contains("(this StableFunction)"),
>> toString);
>
> Not possible to test full toString() representation?
The order is unspecified but the test could be improved to cover the key
associated as well. I will add that.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2026615845
