On Tue, 6 May 2025 11:59:29 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> Chen Liang has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review remarks
>
> src/java.base/share/classes/jdk/internal/util/OperatingSystem.java line 88:
> 
>> 86: 
>> 87:     // The current OperatingSystem
>> 88:     private static @Stable OperatingSystem CURRENT_OS;
> 
> Could this be a `StableValue.supplier()`?

I would rather optimize MethodHandleAccessorFactory than defining a new class 
for this...

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25025#discussion_r2075326990

Reply via email to