On Thu, 11 Mar 2021 23:17:24 GMT, Alex Menkov <[email protected]> wrote:
>> The fix updates the test to skip ThreadStartEvent/ThreadDeathEvent from
>> other threads
>
> Alex Menkov has updated the pull request incrementally with one additional
> commit since the last revision:
>
> reformatted comment
Alex,
The fix looks good.
One minor suggestion about the messages:
+ log2("Got ThreadStartEvent for wrong thread: " + event);
+ log2("Got ThreadDeathEvent for wrong thread: " + event);
More clear message would be something like:
"Thread is not expected, skipping XXX event: " + ...
Thanks,
Serguei
-------------
Marked as reviewed by sspitsyn (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/2927
