On Mon, 16 Jan 2023 11:33:39 GMT, Alexey Ivanov <[email protected]> wrote:

>> In Aqua look and feel, `getDefaultButton()` of `JFileChooser` returns null. 
>> The fix overrides the method from `FileChooserUI` to return the 
>> `ApproveButton` of `FileChooser` which is similar to other L&F. 
>> The test is run for multiple iterations in CI machine.
>
> test/jdk/javax/swing/JFileChooser/AquaDefaultButtonTest.java line 48:
> 
>> 46:                 if (defaultApproveButton == null) {
>> 47:                     throw new RuntimeException("getDefaultButton() 
>> method returns null for Aqua L&F!");
>> 48:                 }
> 
> If you choose shorter names for variables, the line would fit into 80-column 
> limit. Now the two lines are nearly 100 characters long, though not a big 
> issue.

Could you please the copyright year in `AquaFileChooserUI.java`?

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

PR: https://git.openjdk.org/jdk/pull/12008

Reply via email to