[lworld] RFR: 8333757: [lworld] sync javac code generation with the latest spec [v3]

Vicente Romero vromero at openjdk.org
Tue Jun 11 13:56:28 UTC 2024


On Tue, 11 Jun 2024 00:11:13 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Vicente Romero has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   do not include arrays in the LoadableDescriptors attribute in InnerClassLambdaMetafactory, plus some refacts
>
> src/java.base/share/classes/java/lang/classfile/attribute/LoadableDescriptorsAttribute.java line 68:
> 
>> 66:      * @param loadableDescriptors the loadable descriptors
>> 67:      */
>> 68:     static LoadableDescriptorsAttribute of(Utf8Entry... loadableDescriptors) {
> 
> I recommend another set of factory:
> 
> static LoadableDescriptorsAttribute ofSymbols(ClassDesc... loadableDescriptors);
> static LoadableDescriptorsAttribute ofSymbols(List<ClassDesc> loadableDescriptors);

I don't see the benefit tbh

-------------

PR Review Comment: https://git.openjdk.org/valhalla/pull/1118#discussion_r1634935661



More information about the valhalla-dev mailing list