RFR: 8304912: Use OperatingSystem enum in java.desktop module [v2]

Andrey Turbanov aturbanov at openjdk.org
Tue Apr 4 21:49:07 UTC 2023


On Tue, 4 Apr 2023 00:23:07 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Update classes in the java.desktop module to use the jdk.internal.util.OperatingSystem enum instead of the `os.name` system property to select OS specific behaviors.
>
> Roger Riggs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
> 
>  - Merge branch 'master' into 8304912-use-os-desktop
>  - Use OperatingSystem.isAix
>  - Remove unused os.arch system property
>  - Apply OperatingSystem enum instead of os.name to java.desktop module

src/java.desktop/share/classes/sun/font/FontUtilities.java line 71:

> 69:                 isLinux = OperatingSystem.isLinux();
> 70: 
> 71:                 isMacOSX = OperatingSystem.isMacOS(); // TODO: MacOSX

I think this `TODO` is not actual anymore

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13213#discussion_r1157800085



More information about the client-libs-dev mailing list