Thanks Lois an Aleksey for the quick review!
- Ioi
On 11/27/18 8:47 AM, Lois Foltan wrote:
Looks good.
Lois
On 11/27/2018 11:42 AM, Ioi Lam wrote:
https://bugs.openjdk.java.net/browse/JDK-8213275
http://cr.openjdk.java.net/~iklam/jdk12/8213275-ReplaceCriticalClasses-missing-PostVMInitHook.v01/
Please review this simple fix. The jdk.internal.vm.PostVMInitHook
class is not in the openjdk so it should be used in a test in the
openjdk repo.
It isn't important to test for this particular class. When I wrote
the test, I was just looking for a few classes that are loaded
immediately after JVMTI has exited JVMTI_PHASE_PRIMORDIAL, and
PostVMInitHook happened to be one of those classes.
Since we already test a few other classes such as java/util/Locale,
the fix is simply to remove PostVMInitHook from the test.
Thanks
- Ioi
