On Fri, 6 Mar 2026 18:04:55 GMT, Phil Race <[email protected]> wrote:

>> src/java.desktop/share/classes/javax/swing/JOptionPane.java line 1659:
>> 
>>> 1657:     }
>>> 1658: 
>>> 1659:     private static volatile Frame sharedRootFrame;
>> 
>> Any reason for keeping it volatile? I guess similar change in other swing 
>> classes for AppContext removal is not made volatile for ex. 
>> https://github.com/openjdk/jdk/commit/0722ae926ff1327c47a922b1ca0b493a0d06526e
>
> Being a little careful.
> 
> The one you reference is actually even more strict, not less, since it is 
> used wrapped in synchronized {..}

ok

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30028#discussion_r2903183502

Reply via email to