RFR: 8371128: NullPointerException occurs due to double cleanup of SwingNode [v2]

Andy Goryachev angorya at openjdk.org
Tue Nov 4 17:33:58 UTC 2025


On Tue, 4 Nov 2025 17:23:40 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Test message
>
> modules/javafx.swing/src/main/java/javafx/embed/swing/SwingNode.java line 283:
> 
>> 281:                     SwingNodeHelper.runOnEDT(() -> {
>> 282:                         if (lwFrame != null) {
>> 283:                             swNodeIOP.overrideNativeWindowHandle(lwFrame, 0L,
> 
> question: is it possible to have `lwFrame` set to `null` between L282 and L283?

seems like being set in the EDT always

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1960#discussion_r2491449372


More information about the openjfx-dev mailing list