On Tue, 9 May 2023 14:12:09 GMT, Lukasz Kostyra <lkost...@openjdk.org> wrote:

> Calls of `Runtime.getRuntime().exec()` were changed to `String[]` variant.
> 
> Only Windows and macOS parts were affected, tests work good on both platforms.
> 
> I looked through the code and didn't find any other cases of 
> `Runtime.getRuntime().exec(String)` being used, so that should cover all 
> possible deprecation warnings for this issue.

modules/javafx.graphics/src/main/java/com/sun/javafx/application/HostServicesDelegate.java
 line 138:

> 136:             try {
> 137:                 if (osName.startsWith("Mac OS")) {
> 138:                     Runtime.getRuntime().exec(new String[] {

not a review, but a general question:
do we have (do we want) an enum identifying the platform?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1130#discussion_r1188949404

Reply via email to