Remove a redundant comparison in java.time `OffsetDateTime.compareTo()`. If the `compareInstant` utility method returns 0 (equal), it compares the `LocalDateTime`. However, `compareInstant` has already done that comparison; if it found equal, the `compareTo` method unnecessarily does it again. The code is refactored in `compareInstant` to do the comparison of `LocalDateTime` exactly once, if it is needed.
This case is covered by existing tests in test/jdk/java/time/tck/java/time/TCKOffsetDateTime.java ------------- Commit messages: - 8310241: OffsetDateTime compareTo redundant computation Changes: https://git.openjdk.org/jdk/pull/14618/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14618&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8310241 Stats: 19 lines in 1 file changed: 10 ins; 5 del; 4 mod Patch: https://git.openjdk.org/jdk/pull/14618.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/14618/head:pull/14618 PR: https://git.openjdk.org/jdk/pull/14618
