RFR: 6983726: Reimplement MethodHandleProxies.asInterfaceInstance [v25]
Mandy Chung
mchung at openjdk.org
Mon Jul 17 17:32:30 UTC 2023
On Mon, 17 Jul 2023 17:21:50 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review changees
>
> test/jdk/java/lang/invoke/MethodHandleProxies/WrapperHiddenClassTest.java line 165:
>
>> 163: () -> "incorrect dynamic module name: " + implModule.getName());
>> 164:
>> 165: assertSame(implClass.getClassLoader(), implModule.getClassLoader(),
>
> I think this should check against `ifaceModule.getClassLoader()` instead, right? Since the dynamic module is defined in the interface' class loader.
Good catch! Yes, this should check against the interface's loader.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13197#discussion_r1265684091
More information about the core-libs-dev
mailing list