JDK 14 RFR of JDK-8233096: Update javax.lang.model for switch expressions

[Jan Lahoda]

Looks good to me. Thanks for taking care of this!

Jan

On 13. 11. 19 4:50, Joe Darcy wrote:
> Hi Jon,
> 
> Re-refactored test to cut down on code duplication:
> 
>      http://cr.openjdk.java.net/~darcy/8233096.2/
> 
> Cheers,
> 
> -Joe
> 
> On 11/12/2019 5:43 PM, Jonathan Gibbons wrote:
>> Hi Joe,
>>
>> There seems to be a lot of repeated code in the test, perhaps because 
>> it has been slowly growing over the years.
>>
>> It might be worth considering using an enum to encapsulate the 
>> properties/predicate/name of "keyword" vs "name".
>>
>> It also seems like the check calls always come in pairs for true/false 
>> for the first argument. In conjunction with the previous suggestion, 
>> this could maybe be simplified as well.
>>
>> -- Jon
>>
>> On 11/12/19 3:37 PM, Joe Darcy wrote:
>>>
>>> Coming back to this after the switch expression changes were pushed, 
>>> I've updated the test to give a more informative output message on an 
>>> error:
>>>
>>> http://cr.openjdk.java.net/~darcy/8233096.1/
>>>