On Sat, 16 Nov 2024 18:11:32 GMT, Chen Liang <li...@openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   prev is not needed
>
> test/jdk/jdk/internal/ref/Cleaner/CleanableListTest.java line 30:
> 
>> 28:  * @compile/module=java.base 
>> jdk/internal/ref/CleanableListTestHelper.java 
>> jdk/internal/ref/TestCleanable.java
>> 29:  * @modules java.base/jdk.internal.ref
>> 30:  * @run testng/othervm --add-exports 
>> java.base/jdk.internal.ref=ALL-UNNAMED CleanableListTest
> 
> I don't think you need this --add-exports flag; JTReg modules tag should take 
> care of it for you.

Indeed. I'll drop it in next update.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22043#discussion_r1846163165

Reply via email to