RFR: 8350118: Simplify the layout access VarHandle [v2]
Jorn Vernee
jvernee at openjdk.org
Wed Feb 26 17:02:00 UTC 2025
On Fri, 21 Feb 2025 10:05:36 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review remarks, dates, some more simplifications
>
> src/java.base/share/classes/java/lang/invoke/VarForm.java line 150:
>
>> 148: String methodName = value.methodName();
>> 149: MethodType type = methodType_table[value.at.ordinal()].insertParameterTypes(0, VarHandle.class);
>> 150: assert !UNSAFE.shouldBeInitialized(implClass) : implClass;
>
> Thanks - took a long time to figure this one out -- hopefully it will save time in the future ;-)
TBH, I thought that resolving the member name would implicitly initialize the class
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23720#discussion_r1971985978
More information about the build-dev
mailing list