On Fri, 23 Aug 2024 09:21:02 GMT, Abhishek Kumar <abhis...@openjdk.org> wrote:
>> test/jdk/javax/swing/JFileChooser/8080628/bug8080628.java line 1: >> >>> 1: /* >> >> Could you also clean up the main method and remove the extra overhead in the >> SwingUtilities.invokeAndWait call? Also remember to update copyright years > > I think that is ok.. it's pretty simple where runnable interface run method > is overridden and is supposed to run the test. > copyright year updated. What do you mean? Replacing `new Runnable` with lambda? Does it add value? Sure the code becomes shorter… However, the test still does its job. And IDE still shows the compact version displaying the body of the `run` method only. Given that no other code is changed, except for adding additional output, I'd rather keep it as it is. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20671#discussion_r1728705048
