RFR: 8285306: Fix typos in java.desktop [v6]
Alexey Ivanov
aivanov at openjdk.java.net
Wed Jun 15 10:12:46 UTC 2022
On Wed, 15 Jun 2022 07:27:50 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:
>> Magnus Ihse Bursie has updated the pull request incrementally with 17 additional commits since the last revision:
>>
>> - Update src/java.desktop/windows/classes/sun/awt/windows/WPrinterJob.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/unix/classes/sun/awt/X11/XBaseMenuWindow.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/share/classes/sun/swing/SwingUtilities2.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/share/classes/sun/swing/SwingUtilities2.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/share/classes/sun/awt/image/ImagingLib.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/share/classes/sun/awt/image/ImagingLib.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/share/classes/javax/swing/tree/FixedHeightLayoutCache.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/share/classes/javax/swing/text/html/parser/TagStack.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - Update src/java.desktop/share/classes/javax/swing/text/html/CSS.java
>>
>> Co-authored-by: Alexey Ivanov <70774172+aivanov-jdk at users.noreply.github.com>
>> - ... and 7 more: https://git.openjdk.org/jdk/compare/40c585cd...98e635a5
>
> src/java.desktop/share/classes/javax/swing/JTree.java line 3320:
>
>> 3318: (parent, path.getPathComponent(counter));
>> 3319: parent = path.getPathComponent(counter);
>> 3320: if(indices[counter - 1] < 0)
>
> let's add space after `if`
Better not: if you look at the code around it, there's no space between a keyword and the opening parenthesis. So the code looks consistent. Changing this `if` will make the code inconsistent.
-------------
PR: https://git.openjdk.org/jdk/pull/8328
More information about the client-libs-dev
mailing list