RFR: 8321812: Update GC tests to use execute[Limited]TestJava [v4]

[Stefan Karlsson]

On Wed, 3 Jan 2024 08:29:48 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

>> Stefan Karlsson has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
>> 
>>  - Remove unused pb
>>  - Merge remote-tracking branch 'upstream/master' into 8321812_use_executeTestJava_in_gc_tests
>>  - Merge branch 'rename_executeTestJvm' into 8321812_use_executeTestJava_in_gc_tests
>>  - Merge remote-tracking branch 'upstream/master' into rename_executeTestJvm
>>  - Merge remote-tracking branch 'upstream/master' into rename_executeTestJvm
>>  - Whitespace fixes
>>    
>>    Co-authored-by: Thomas Schatzl <59967451+tschatzl at users.noreply.github.com>
>>  - 8321812: Update GC tests to use execute[Limited]TestJava
>>  - Test cleanup
>>  - Fix impl and add test
>>  - 8321713: Harmonize executeTestJvm with create[Limited]TestJavaProcessBuilder
>
> test/hotspot/jtreg/gc/g1/TestSharedArchiveWithPreTouch.java line 62:
> 
>> 60:         dump_args.addAll(Arrays.asList(new String[] { "-Xshare:dump", "-Xlog:cds" }));
>> 61: 
>> 62:         pb = ProcessTools.createLimitedTestJavaProcessBuilder(dump_args);
> 
> -        ProcessBuilder pb;
> ``` 
> Above no longer used.

Removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17067#discussion_r1440207292