RFR: 8263470: Consolidate copies of getClassBytes in various tests

Thomas Stuefe stuefe at openjdk.java.net
Fri Mar 12 16:13:22 UTC 2021


On Fri, 12 Mar 2021 01:16:18 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> See issue for details. This also moves ClassUnloadCommon and ClassWithManyMethodsClassLoader to jdk/lib/test/classloader and puts it in a package with other class loader test utility functions. I left getClassBytes in ClassunloadCommon, because most tests that use it also need to import ClassUnloadCommon.
> 
> Tested with tier1-3.

Hi Coleen,

LGTM. We loose the "filename is" debug output, was that intentional?

Cheers, Thomas

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

Marked as reviewed by stuefe (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2954


More information about the hotspot-runtime-dev mailing list