RFR: 8332252: Clean up vmTestbase/vm/share [v3]

Stefan Karlsson stefank at openjdk.org
Mon Jun 17 09:01:15 UTC 2024


On Sat, 15 Jun 2024 16:28:05 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> The vmTestbase/vm/share is a shared test library for vmTestbase tests. This library contains a lot of code that is used by only by small number of tests or not used at all. There are no plans to actively develop new tests in vmTestsbase and improve this shared library. 
>> The final goal of this and the following PRs is to reduce the maintenance cost of vmTestbase by eliminating this library.
>> 
>> Also, this PR moves test-specific code into corresponding test directories to increase code locality. This allows later easier move tests from vmTestbase.
>> 
>> The few remaining classes include 
>> InMemoryJavaCompiler.java
>> that is very similar to same class from the standard testlibrary and could be merge with it and
>> ProcessUtils.java
>> which is used by
>> test/hotspot/jtreg/runtime/Thread/TestBreakSignalThreadDump.java
>> and thus should be moved into the standard testlibrary.
>> The stack and options might be merged in nsk/share test library.
>
> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
> 
>   removed unused import

Some of the moves/renames messed up the sort-order of the imports. Could you take a pass over the patch and clean that up?

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

PR Review: https://git.openjdk.org/jdk/pull/19727#pullrequestreview-2122317400


More information about the hotspot-dev mailing list