RFR: 8294960: Convert java.base/java.lang.invoke package to use the Classfile API to generate lambdas and method handles [v18]
Adam Sotona
asotona at openjdk.org
Mon Jun 17 13:11:17 UTC 2024
On Thu, 6 Jun 2024 12:17:14 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Adam Sotona has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - reverted static initialization of ConstantPoolBuilder and CP entries
>> - fixed naming conventions
>
> src/java.base/share/classes/java/lang/invoke/GenerateJLIClassesHelper.java line 564:
>
>> 562: clb.withFlags(ACC_PRIVATE | ACC_FINAL | ACC_SUPER)
>> 563: .withSuperclass(InvokerBytecodeGenerator.INVOKER_SUPER_DESC)
>> 564: .with(SourceFileAttribute.of(className.substring(className.lastIndexOf('/') + 1)));
>
> Maybe we can keep the classDesc from ofInternalName and use its displayName.
I don't see any benefits, both ways are sub-stringing.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17108#discussion_r1642789436
More information about the core-libs-dev
mailing list