On Mon, 24 Apr 2023 15:16:18 GMT, Jan Lahoda <jlah...@openjdk.org> wrote:
>> This is the first draft of a patch for JEP 440 and JEP 441. Changes included:
>>
>> - the pattern matching for switch and record patterns features are made
>> final, together with updates to tests.
>> - parenthesized patterns are removed.
>> - qualified enum constants are supported for case labels.
>>
>> This change herein also includes removal record patterns in for each loop,
>> which may be split into a separate PR in the future.
>
> Jan Lahoda has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Reflecting review changes.
src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Flow.java line 3212:
> 3210: @Override
> 3211: public void visitCase(JCCase tree) {
> 3212: super.visitCase(tree); // Generated from
> nbfs://nbhost/SystemFileSystem/Templates/Classes/Code/OverriddenMethodBody
do we need this comment?
src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TransPatterns.java line
862:
> 860:
> 861: private LoadableConstant createEnumDesc(DiagnosticPosition pos,
> ClassSymbol enumClass, Name constant) {
> 862: //ConstantBootstrap.invoke(...,
> EnumDesc.of(ConstantBootstrap.invoke(..., ClassDesc.of(enumClass)), constant))
I think this comment can be removed or probably reformatted
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13074#discussion_r1175611723
PR Review Comment: https://git.openjdk.org/jdk/pull/13074#discussion_r1175615234
