On Wed, 17 Dec 2025 23:51:42 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/AccessControlTest.java line 217:
> 
>> 215:             boolean sameClass = (c1 == c2);
>> 216:             Assertions.assertTrue(samePackage  || !sameTopLevel);
>> 217:             Assertions.assertTrue(sameTopLevel || !sameClass);
> 
> Suggestion:
> 
>             assertTrue(samePackage  || !sameTopLevel);
>             assertTrue(sameTopLevel || !sameClass);

Looks like you can do this cleanup in other places too.

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

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

Reply via email to