RFR: JDK-8308028 Replace more uses of System.getProperty("os.name") with PlatformUtil calls [v2]

Nir Lisker nlisker at openjdk.org
Sat May 13 16:50:52 UTC 2023


On Sat, 13 May 2023 16:42:33 GMT, Marius Hanl <mhanl at openjdk.org> wrote:

>> Carl Döbbelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   added isEmbedded
>
> modules/javafx.media/src/main/java/com/sun/media/jfxmediaimpl/HostUtils.java line 60:
> 
>> 58: 
>> 59:     public static boolean isWindows() {
>> 60:         return osName.startsWith("windows");
> 
> `osName` is unused now

`osArch` is also unused. It's just a local variable.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1134#discussion_r1193012839


More information about the openjfx-dev mailing list