RFR: 8370636: com/sun/jdi/TwoThreadsTest.java should wait for completion of all threads [v2]
Chris Plummer
cjplummer at openjdk.org
Tue Oct 28 20:15:50 UTC 2025
On Tue, 28 Oct 2025 19:48:59 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:
>> Test
>> com/sun/jdi/TwoThreadsTest.java
>> start new threads using DebuggeeWrapper.newThread()
>> and exit immediately.
>>
>> It is needed to jon() threads so test really works for virtual threads.
>>
>> I looked on other threads that use DebuggeeWrapper.newThread(). Seems most of them already updated to join spawned threads.
>>
>> Tested by running test on all platfrom with and wihtout virtual threads.
>
> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
>
> fixed spelling
Marked as reviewed by cjplummer (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/27982#pullrequestreview-3390615792
More information about the serviceability-dev
mailing list