On Thu, 26 Jun 2025 05:12:08 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> test/jdk/javax/swing/JTree/TestTreeRowSelection.java line 62:
>> 
>>> 60:                 frame.pack();
>>> 61:                 frame.setLocationRelativeTo(null);
>>> 62:                 frame.setVisible(true);
>> 
>> You may move UI implementation code to helper method.
>
> Not needed strictly..good to have linear flow..

I agree it's not compulsory but I feel it better from readability POV.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25966#discussion_r2170789469

Reply via email to