On Tue, 15 Oct 2024 03:18:09 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

> While implementing [JDK-8339517](https://bugs.openjdk.org/browse/JDK-8339517) 
> to eliminate native access warnings by passing "--enable-native-access" for 
> the three JavaFX modules with native code (javafx.graphics, javafx.media, and 
> javafx.web), it was found  that the Swing interop code in javafx.swing calls 
> a JNI method defined in one of the native graphics libraries (prism-common) 
> directly
> 
> This means that even after 
> [JDK-8339517](https://bugs.openjdk.org/browse/JDK-8339517) is fixed, we still 
> get native access warnings when running any test that uses SwingNode.
> 
> This fixes the native access warning by making javafx.graphics module call 
> the native JNI and Swing-interop calls the static utility method in 
> javafx.graphics, in this case defined in PlatformImpl
> 
> All test.javafx.embed.swing SwingNode tests are running ok without any native 
> warning

looks good on macOS (did not test on win/linux)

launching Tools -> JTextArea ... window from the monkey tester, the warning is 
silenced once `--enable-native-access=javafx.graphics` is added to the command 
line

left a minor comment

modules/javafx.swing/src/main/java/com/sun/javafx/embed/swing/newimpl/SwingNodeInteropN.java
 line 130:

> 128:     }
> 129: 
> 130: 

minor: extra newline?

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

Marked as reviewed by angorya (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/1600#pullrequestreview-2369789780
PR Review Comment: https://git.openjdk.org/jfx/pull/1600#discussion_r1801440624

Reply via email to