On Mon, 23 Jun 2025 15:48:41 GMT, Alan Bateman <[email protected]> wrote:
>> Anton Artemov has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - 8359437: Addressed reviewers' comments
>> - 8359437: Addressed reviewers' comments
>
> test/jdk/java/lang/Thread/virtual/Parking.java line 388:
>
>> 386: @ParameterizedTest
>> 387: @ValueSource(booleans = { true, false })
>> 388: @DisabledIf("LockingMode#isLegacy")
>
> Would you mind checking if the import DisabledIf can be removed from these
> tests? I think we only used it to conditionally run when not legacy mode.
That imports became unused everywhere, so I removed them.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25847#discussion_r2163397316
