RFR: 8305083: Remove finalize() from test/hotspot/jtreg/vmTestbase/nsk/share/ and /jpda that are used in serviceability/dcmd/framework tests [v4]

[Coleen Phillimore]

On Fri, 5 May 2023 14:50:30 GMT, Afshin Zafari <duke at openjdk.org> wrote:

>> The `finalize()` method is removed from base classes/interfaces and are replaced by a Cleaner callback..
>
> Afshin Zafari has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains four commits:
> 
>  - Merge branch 'master' into _8305083
>  - 8305083: 8305083: Remove finalize() from test/hotspot/jtreg/vmTestbase/nsk/share/ and /jpda that are used in serviceability/dcmd/framework tests
>  - 8305083: Remove finalize() from test/hotspot/jtreg/vmTestbase/nsk/share/ and /jpda that are used in serviceability/dcmd/framework tests
>  - 8305083: Remove finalize() from test/hotspot/jtreg/vmTestbase/nsk/share/ and /jpda that are used in serviceability/dcmd/framework tests

test/hotspot/jtreg/vmTestbase/nsk/share/MainWrapper.java line 50:

> 48: 
> 49:         // Some tests use this property to understand if virtual threads are used
> 50:         System.setProperty("main.wrapper", wrapperName);

Should this line have been deleted?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13420#discussion_r1186423314