On Tue, 9 Feb 2021 20:27:52 GMT, Joe Darcy <da...@openjdk.org> wrote:

>> A follow-up of sorts to JDK-8257086, this change aims to improve the 
>> discussion of the relationship between Object.equals and compareTo and 
>> compare methods. The not-consistent-with-equals natural ordering of 
>> BigDecimal get more explication too. While updating Object, I added some 
>> uses of apiNote and implSpec, as appropriate. While a signum method did not 
>> exist when Comparable was added, it has existed for many years so I removed 
>> obsolete text that defined a "sgn" function to compute signum.
>> 
>> Once the changes are agreed to, I'll reflow paragraphs and update the 
>> copyright years.
>
> Joe Darcy has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix typos found in code review.

src/java.base/share/classes/java/lang/Object.java line 148:

> 146:      * relation, each equivalence class only has a single element.
> 147:      *
> 148:      * @apiNoted

`s/apiNoted/apiNote/`

src/java.base/share/classes/java/lang/Object.java line 198:

> 196:      * need to be modified.
> 197:      *
> 198:      * @implspec

`s/implspec/implSpec/`

src/java.base/share/classes/java/math/BigDecimal.java line 3062:

> 3060: 
> 3061:      * @apiNote
> 3062:      * Note: this class has a natural ordering that is inconsistent 
> with equals.

Is `Note: ` really needed?

-------------

PR: https://git.openjdk.java.net/jdk/pull/2471

Reply via email to