RFR: 8279795: Fix typo in BasicFileChooserUI: Constucts -> Constructs
Alexey Ivanov
aivanov at openjdk.java.net
Tue Jan 18 17:14:33 UTC 2022
On Wed, 12 Jan 2022 02:40:22 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
>> Fix the typo in the constructor for BasicFileChooserUI.DoubleClickListener: ‘Constucts’ → ‘Constructs’.
>> Also fixed the typo in the parameter of the constructor: ‘the lsit’ → ‘the list’.
>>
>> I also organised the imports which replaced all wildcard imports with specific class imports.
>
> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicFileChooserUI.java line 73:
>
>> 71: import javax.swing.plaf.FileChooserUI;
>> 72: import javax.swing.plaf.UIResource;
>> 73:
>
> I see that you added a line between different top packages, I use the same style, but then it will make sense to do the same for java -> javax above.
Hm… Usually, there's no blank line between java- and javax- packages. Or I may have missed that it's the common practice.
In fact, I was thinking about adding a blank line in this case because the list of classes imported from either package is large.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7030
More information about the client-libs-dev
mailing list