On Wed, 2 Apr 2025 13:22:44 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> Implement JEP 502.
>> 
>> The PR passes tier1-tier3 tests.
>
> 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 162:

> 160:  * A <em>stable int function</em> is a function that takes an {@code 
> int} parameter and
> 161:  * uses it to compute a result that is then cached by the backing stable 
> value storage
> 162:  * for that parameter value. A stable int function is created via the

Suggestion:

 * for that parameter value. A stable IntFunction is created via the

src/java.base/share/classes/java/lang/StableValue.java line 172:

> 170:  * public final class SqrtUtil {
> 171:  *
> 172:  *      private SqrtUtil(){}

Suggestion:

 *      private SqrtUtil() {}

src/java.base/share/classes/java/lang/StableValue.java line 242:

> 240:  *      private static final int CACHED_SIZE = 10;
> 241:  *
> 242:  *     private SqrtUtil(){}

Suggestion:

 *     private SqrtUtil() {}

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2024922015
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2024923703
PR Review Comment: https://git.openjdk.org/jdk/pull/23972#discussion_r2024926886

Reply via email to