RFR: 8285485: Fix typos in corelibs
Magnus Ihse Bursie
ihse at openjdk.java.net
Fri May 13 11:29:46 UTC 2022
On Wed, 27 Apr 2022 16:50:30 GMT, Pavel Rappo <prappo at openjdk.org> wrote:
>> I ran `codespell` on modules owned by core-libs, and accepted those changes where it indeed discovered real typos.
>>
>> I will update copyright years using a script before pushing (otherwise like every second change would be a copyright update, making reviewing much harder).
>>
>> The long term goal here is to make tooling support for running `codespell`. The trouble with automating this is of course all false positives. But before even trying to solve that issue, all true positives must be fixed. Hence this PR.
>
> src/java.se/share/data/jdwp/jdwp.spec line 47:
>
>> 45: "Returns reference types for all the classes loaded by the target VM "
>> 46: "which match the given signature. "
>> 47: "Multiple reference types will be returned if two or more class "
>
> This file's name scares me. The fact that there's no "serviceability" label on this PR adds to this feeling. I would ask around if I were you.
I'll revert all changes in this file. I don't want it to hold up the entire PR. We can revisit it later, with serviceability.
> src/java.sql.rowset/share/classes/com/sun/rowset/JoinRowSetImpl.java line 636:
>
>> 634: // to be INNER JOIN'ED to form a new rowset
>> 635: // Compare table1.MatchColumn1.value1 == { table2.MatchColumn2.value1
>> 636: // ... up to table2.MatchColumn2.valueN }
>
> Curious: it is not some established string representation, is it?
You mean writing "upto" instead of "up to"? No, it's just incorrect, and a somewhat common typo.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8364
More information about the nio-dev
mailing list