On Fri, 21 Feb 2025 10:05:36 GMT, Maurizio Cimadamore <mcimadam...@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

Reply via email to