RFR: 8333108: Update vmTestbase/nsk/share/DebugeeProcess.java to don't use finalization

Serguei Spitsyn sspitsyn at openjdk.org
Thu May 30 15:36:01 UTC 2024


On Tue, 28 May 2024 20:24:31 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> The 
> test/hotspot/jtreg/vmTestbase/nsk/share/jpda/DebugeeProcess.java
> uses cleanup() to kill debuggee process.
> 
> However, most tests kill the debuggee process explicitly. I verified that debuggee process is killed before test finishes. (Just by printing it's status.)
> 
> The fix adds a few checks debuggee.waitFor() int tests that didn't check debuggee process code.

Looks good.

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

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19437#pullrequestreview-2088643418


More information about the serviceability-dev mailing list