RFR: 8328089: Automate javax/swing/JTable/4222153/bug4222153.java applet test [v3]

Abhishek Kumar abhiscxk at openjdk.org
Thu Mar 14 05:51:55 UTC 2024


On Thu, 14 Mar 2024 05:33:41 GMT, Tejesh R <tr at openjdk.org> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Whitespace error fix
>
> test/jdk/javax/swing/JTable/bug4222153.java line 66:
> 
>> 64:                 frame.setLocationRelativeTo(null);
>> 65:                 frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
>> 66:                 frame.setVisible(true);
> 
> I guess it is better to move frame initialization part into a separate method rather here.

Since the UI initialization was minimal, I kept it in main. Moved to separate method now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18275#discussion_r1524271401


More information about the client-libs-dev mailing list