On Wed, 16 Jun 2021 10:58:22 GMT, Stephen Colebourne <scolebou...@openjdk.org>
wrote:
>> Patrick Concannon 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 four additional
>> commits since the last revision:
>>
>> - 8268469: Removed excessive spacing; corrected misplaced comments
>> - Merge remote-tracking branch 'origin/master' into JDK-8268469
>> - Merge remote-tracking branch 'origin/master' into JDK-8268469
>> - 8268469: Update java.time to use switch expressions
>
> src/java.base/share/classes/java/time/LocalDate.java line 607:
>
>> 605: if (field instanceof ChronoField chronoField) {
>> 606: if (chronoField.isDateBased()) {
>> 607: int n = switch (chronoField) {
>
> This logic is harder to read than before, and it requires the CPU to perform
> an extra branch on `(n == -1)`. It should just be a `return switch ...`
Reverted logic as requested. See 24ae9e5
-------------
PR: https://git.openjdk.java.net/jdk/pull/4433
