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

Reply via email to