On Fri, 21 Apr 2023 22:07:12 GMT, Phil Race <[email protected]> wrote:
>> test/jdk/java/awt/dnd/NextDropActionTest/NextDropActionTest.java line 159:
>>
>>> 157: }
>>> 158: } finally {
>>> 159: if (frame != null) {
>>
>> frame is created with in EDT but disposing without EDT ?
>
> There's no harm in doing that but I'm not sure its necessary for the
> dispose().
it is fine to call dispose here on the main, but the frame possibly can be null
here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13591#discussion_r1174217484
