On Fri, 24 Mar 2023 00:27:42 GMT, Roger Riggs <rri...@openjdk.org> wrote:
>> Improvements to support OS specific customization for JDK internal use: >> - To select values and code; allowing elimination of unused code and values >> - Optionally evaluated by build processes, compilation, or archiving (i.e. >> CDS) >> - Simple API to replace adhoc comparisons with `os.name` >> - Clear and consistent use across build, runtime, and JDK modules >> >> The PR includes updates within java.base to use the new API. > > Roger Riggs has updated the pull request incrementally with one additional > commit since the last revision: > > Rename OperatingSystem enum values to uppercase @battleblow FYI src/java.base/share/classes/jdk/internal/util/OperatingSystem.java line 66: > 64: > 65: /** > 66: * The Linux Operating system. For the sake if completeness, Linux isn't an operating system, so this statement isn't correct. src/java.base/share/classes/jdk/internal/util/OperatingSystem.java line 81: > 79: */ > 80: AIX, > 81: ; So no Unknown value? ------------- PR Comment: https://git.openjdk.org/jdk/pull/12931#issuecomment-1483049583 PR Review Comment: https://git.openjdk.org/jdk/pull/12931#discussion_r1147785316 PR Review Comment: https://git.openjdk.org/jdk/pull/12931#discussion_r1147784804