<i18n dev> RFR: 8285306: Fix typos in java.desktop [v6]
Magnus Ihse Bursie
ihse at openjdk.java.net
Fri May 13 16:56:54 UTC 2022
On Thu, 28 Apr 2022 21:25:02 GMT, Alexey Ivanov <aivanov 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.java.net/jdk/compare/40c585cd...98e635a5
>
> Nearly 500 files are too many. Smaller chunks would be easier to review.
>
> Some of the native code files could come from upstream libraries.
@aivanov-jdk While I approve of finding and fixing spelling mistakes, grammatical or semantic errors, the mistakes you posted suggestions for was not at all associated with the mechanical changes made by codespell that this PR included, and I assume you noticed them just because you read the text around the typo my PR fixed.
In hindsight, it would have been better if you've made these changes yourself, in a separate PR.
But since you had spent the effort of looking for these problems, and suggesting fixes, I have now accepted all of them. Due to how the Github UI works, this was quite tedious, so please refrain from submitting more than a handful suggestions next time you review a PR.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8328
More information about the i18n-dev
mailing list