RFR: 8310241: OffsetDateTime compareTo redundant computation [v2]

Roger Riggs rriggs at openjdk.org
Wed Jun 28 20:01:57 UTC 2023


On Wed, 28 Jun 2023 19:35:27 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Reopen to work on an improved fix
>>   Add a test for the specific condition being optimized, the test was missing in the original.
>
> test/jdk/java/time/tck/java/time/TCKOffsetDateTime.java line 1384:
> 
>> 1382:         assertEquals(b.compareTo(b) == 0, true);
>> 1383:         assertEquals(a.toInstant().compareTo(b.toInstant()) == 0, true);
>> 1384:         assertEquals(OffsetDateTime.timeLineOrder().compare(a,b) == 0, true);
> 
> Could be in a separate test case, as this method tests `compareTo` method?

This is one of a sequence of tests of compareTo.  The other (previous) tests include verifying the assertion about the result of the `timeLineOrder` comparator with the same data.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14618#discussion_r1245695820


More information about the core-libs-dev mailing list