On Tue, 22 Jun 2021 16:07:12 GMT, Patrick Concannon <pconcan...@openjdk.org> wrote:
>> Hi, >> >> Could someone please review the second half of my update for the `java.time` >> package to make use of switch expressions? >> >> This PR was split into two parts due to the large number of files affected. >> >> Kind regards, >> >> Patrick > > Patrick Concannon has updated the pull request incrementally with one > additional commit since the last revision: > > 8269124: Added missing brace; fixed build issue src/java.base/share/classes/java/time/Instant.java line 562: > 560: public int get(TemporalField field) { > 561: if (field instanceof ChronoField) { > 562: return switch ((ChronoField) field) { Not really comment on the change itself, but Is this a leftover from the `instanceof` pattern variable exercise, or will we have another round for that? I see other locations which could be leftovers. ------------- PR: https://git.openjdk.java.net/jdk/pull/4552