RFR: 8262889: Compiler implementation for Record Patterns [v2]
Jan Lahoda
jlahoda at openjdk.java.net
Fri May 6 17:43:07 UTC 2022
On Thu, 5 May 2022 18:11:54 GMT, Vicente Romero <vromero at openjdk.org> wrote:
>> Jan Lahoda has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Reflecting review feedback.
>
> src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Attr.java line 4217:
>
>> 4215: }
>> 4216: ListBuffer<BindingSymbol> outBindings = new ListBuffer<>();
>> 4217: List<Type> recordTypes = expectedRecordTypes;
>
> nit: probably a matter of style but why not reusing the already declared `expectedRecordTypes` declaring a new variable seems unnecessary
Please note the full `expectedRecordTypes` are used for error reporting, but the reference to `List` in `recordTypes` is overwritten in the loop (at the time of an error report, it may not longer point to the full expected types, and hence cannot be used for error reporting).
-------------
PR: https://git.openjdk.java.net/jdk/pull/8516
More information about the core-libs-dev
mailing list