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