RFR: 8306471: Add virtual threads support to JDWP ThreadReference.Stop and JDI ThreadReference.stop() [v10]
Chris Plummer
cjplummer at openjdk.org
Fri May 12 03:54:48 UTC 2023
On Fri, 12 May 2023 03:30:34 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
>>
>> update linenumber for breakpoint. It changed with a recent commit that added a comment.
>
> test/hotspot/jtreg/vmTestbase/nsk/jdi/ThreadReference/stop/stop002.java line 154:
>
>> 152: log.complain("TEST #1 FAILED: expected IllegalArgumentException was not thrown");
>> 153: tot_res = Consts.TEST_FAILED;
>> 154: } catch (InvalidTypeException ee) {
>
> It is confusing that the log.complain at line 152 expects `IllegalArgumentException` but the really expected exception is `InvalidTypeException`. Is it intentional? If so then it is better to clarify it in the comment.
Yes, this looks like a pre-existing bug. I'll cleanup the message.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13548#discussion_r1191876941
More information about the serviceability-dev
mailing list