On Tue, 19 Sep 2023 09:08:16 GMT, Tejesh R <[email protected]> wrote:

>> These are the tests being converted:
>> 
>> javax/swing/JTree/4696499/bug4696499.java
>> javax/swing/JTree/5039542/bug5039542.java
>> javax/swing/JViewport/4546474/bug4546474.java
>> javax/swing/JViewport/4677611/bug4677611.java
>> javax/swing/KeyboardManager/4345798/bug4345798.java
>
> test/jdk/javax/swing/JTree/bug5039542.java line 37:
> 
>> 35: 
>> 36:     public static void main(String[] args) throws Exception {
>> 37:         SwingUtilities.invokeAndWait(() -> {
> 
> `SwingUtilities.invokeAndWait` isn't required right?

I brought up the question if these headless tests should/shouldn't be on the 
EDT if it doesn't need it in last weeks meeting. The answer I received is to 
use it anyway as it sets a good example for tests with these components.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15802#discussion_r1330422628

Reply via email to