On Fri, 22 Mar 2024 08:54:07 GMT, Tejesh R <[email protected]> wrote:

>> Clean up and opensource five tests.
>
> test/jdk/javax/swing/InputVerifier/bug4774166.java line 229:
> 
>> 227:     public void cleanupGUI() {
>> 228:         if (testframe != null) {
>> 229:             testframe.setVisible(false);
> 
> `testframe.setVisible(false);` is required here? Any advantage for setting 
> the visibility to false ? anyhow its been disposed.....

Not harmful, but not needed either.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18443#discussion_r1536710591

Reply via email to