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