On Thu, 27 Oct 2022 03:41:34 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> Change Thread.resume() to error reporting and add some time to complete
>> thread.
>
> Leonid Mesnik has updated the pull request incrementally with one additional
> commit since the last revision:
>
> All dispose tests fixed.
test/hotspot/jtreg/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002a.java
line 130:
> 128: instruction = pipe.readln();
> 129: if (instruction.equals("check_done")) {
> 130: if (Utils.isAlive(test_thread)) {
We also have `jdk.test.lib.Utils` being referenced as `Utils` in the main test.
Maybe `JDIUtils` would be better.
Also, `isAlive` is a bit generic sounding. It would be nice if the name
conveyed the relationship to waiting after a dispose. Maybe
`isAliveAfterVMDispose`. At least then the user knows there's something special
about the "is alive" check being made.
-------------
PR: https://git.openjdk.org/jdk/pull/10877
