RFR: 8370636: com/sun/jdi/TwoThreadsTest.java should wait for completion of all threads

Chris Plummer cjplummer at openjdk.org
Tue Oct 28 19:24:17 UTC 2025


On Fri, 24 Oct 2025 20:20:14 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.

test/jdk/com/sun/jdi/TwoThreadsTest.java line 58:

> 56:         t1.start();
> 57:         t2.start();
> 58:         // The threads might be virutal and daemon, so wait until completion.

"virutal" is misspelled.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27982#discussion_r2470780839


More information about the serviceability-dev mailing list