On Thu, 22 Aug 2024 05:24:24 GMT, Abhishek Kumar <abhis...@openjdk.org> wrote:

> This test should have failed with the 
> [changeset](https://github.com/openjdk/jdk/pull/20612) proposed for 
> [JDK-4795384](https://bugs.openjdk.org/browse/JDK-4795384) but it didn't 
> because GTK look and feel isn't supported in headless environment.
> 
> Fix is to run the test with headful environment on linux platform. 
> 
> Tested in CI pipeline and link attached to JBS.

Please, update the copyright year.

test/jdk/javax/swing/JFileChooser/8080628/bug8080628.java line 105:

> 103:         }
> 104:     }
> 105: 

I'd rather keep this blank line. Most classes have blank line between the 
closing brace of the last method and the closing brace of the class itself.

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

Marked as reviewed by aivanov (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20671#pullrequestreview-2256726122
PR Review Comment: https://git.openjdk.org/jdk/pull/20671#discussion_r1728627065

Reply via email to