RFR: JDK-8307807 Replace use of System.getProperty("os.name") with PlatformUtil calls [v6]
Andy Goryachev
angorya at openjdk.org
Fri May 12 14:58:56 UTC 2023
On Fri, 12 May 2023 14:45:43 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Carl Döbbelin has updated the pull request incrementally with one additional commit since the last revision:
>>
>> made launcherimpl one line
>
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/Platform.java line 67:
>
>> 65: }
>> 66:
>> 67: if (PlatformUtil.isMac()) {
>
> this change might be ok, but it is not strictly equivalent.
> before, MAC path would be selected if os.name lowercase starts from either "mac" or "darwin", after the change, only starts with "Mac".
> Do we want to support "darwin"?
I'd still prefer to use PlatformUtil, since this is a cleaner solution that the variety of methods used before:
osNameLowerCase.startsWith("wind"
"ios".equals(System.getProperty("os.name")
osName.startsWith("Windows"
osName.startsWith("Mac OS"
("os.name").toLowerCase().indexOf("win")
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1132#discussion_r1192483282
More information about the openjfx-dev
mailing list