RFR: 8310033: Clarify return value of Java Time compareTo methods [v2]

Pavel Rappo prappo at openjdk.org
Thu Jul 6 23:33:58 UTC 2023


On Sun, 18 Jun 2023 20:26:04 GMT, Stephen Colebourne <scolebourne at openjdk.org> wrote:

>> Roger Riggs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>> 
>>  - Use {@code xxx} to highlight the comparison against the arg.
>>    Update copyrights.
>>  - Merge branch 'master' into 8310033-time-compareto
>>  - Clarify for Duration, AbstractChronology, and Chronology
>>  - Correct javadoc of compareInstant
>>  - 8310033: Improve Instant.compareTo javadoc to mention before and after
>>    Refine timeline order to mention before and after
>>    Add javadoc @see tags to isBefore and isAfter methods
>
> As things stand, this PR makes things worse not better I'm afraid.

> @jodastephen Please review updated descriptions. Thanks

I'm not @jodastephen, obviously, but we both suggested doing something with punctuation/grammar as this reads funny/peculiar:

    @return the comparator value is less than zero

Maybe our suggestions got lost in this huge discussion, or you've decided to address them later.

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

PR Comment: https://git.openjdk.org/jdk/pull/14479#issuecomment-1624431201


More information about the core-libs-dev mailing list