On Tue, 20 Jan 2026 23:00:02 GMT, Chen Liang <[email protected]> wrote:

>> 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.

Huh yeah. Maybe I put this in the wrong place. Nvm.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/28879#discussion_r2710661444

Reply via email to