On Wed, 17 Dec 2025 23:58:36 GMT, Jorn Vernee <[email protected]> wrote:
>> Chen Liang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Some omissions
>
> test/jdk/java/lang/invoke/DefineClassTest.java line 211:
>
>> 209:
>> 210: Class<?> clazz =
>> lookup.defineClass(generateClass("java.lang.Foo"));
>> 211: assertEquals("java.lang.Foo", clazz.getName());
>
> Looks like the parameter order was already reversed here.
No? I think my new order is right that the "actual" should be the value from
reflection.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28879#discussion_r2710368951
