RFR: 8321299: runtime/logging/ClassLoadUnloadTest.java doesn't reliably trigger class unloading [v4]

[David Holmes]

On Tue, 12 Mar 2024 20:46:23 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

>> ClassLoadUnloadTest.java passes by accident because an unrelated LambdaForm is being unloaded. The test class, `test.Empty` should be loaded by a custom class loader and then subsequently unloaded, however this is not actually occurring due to the sub-process not receiving the class path. This patch modifies `ClassUnloadCommon`, so it may accept a string containing the classlist, and `ClassLoadUnloadTest.java` so it checks specifically for `test.Empty` to be unloaded. Verified with tier1-5 tests.
>
> Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Updated copyright

Thanks for the update! Much simpler/cleaner.

Did you get a chance to investigate the other test?

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18207#pullrequestreview-1933313098