On Wed, 14 Feb 2024 20:01:01 GMT, Alexey Ivanov <[email protected]> wrote:

>> done.
>
> This is not thread-safe, the condition `if (mainFrame != null)` should also 
> be inside `invokeAndWait`.
> 
> Maybe create a method to use method reference: `invokeAndWait(me::dispose)`? 
> In this case, `mainFrame` could be made instance field.

Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17736#discussion_r1491387725

Reply via email to