On Tue, 18 Aug 2026 21:45:29 GMT, Chris Plummer <[email protected]> wrote:
>> test/hotspot/jtreg/vmTestbase/nsk/jdi/ThreadReference/stop/stop002.java line
>> 241:
>>
>>> 239: log.display("TEST #4: resuming thread.");
>>> 240: thrRef.resume();
>>> 241: // Force the debuggee out of the loop.
>>
>> Any particular reason why removing that second part of the comment? Seems
>> that's still true.
>
> Because it is now needed if there is an OpaqueFrameException on a virtual
> thread. If this happens, there is no failure and we continue on to Test 5,
> but the debuggee would still be stuck in the loop in Test 4.
Right, but I mean the extra sentence that mentions that this is not always
needed is still correct.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32403#discussion_r3817489137
