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

Joe Darcy joe.darcy at oracle.com
Wed Nov 13 03:50:59 UTC 2019


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/
>>


More information about the compiler-dev mailing list