On Wed, 23 Jun 2021 16:35:30 GMT, Naoto Sato wrote:
>> Hi Naoto, I decided to only introduce the`instanceof` pattern variable where
>> I thought it would add additional value to the code. In situations like this
>> one, I thought there wasn't much point as the cast variable is only used
>> onc
On Tue, 22 Jun 2021 17:50:05 GMT, Patrick Concannon
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 rega
On Wed, 23 Jun 2021 10:39:06 GMT, Patrick Concannon
wrote:
>> 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) {
>>
>>
On Tue, 22 Jun 2021 17:50:05 GMT, Patrick Concannon
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 rega
On Tue, 22 Jun 2021 17:27:58 GMT, Naoto Sato wrote:
>> 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:
>
