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