Review request for 6612680 (Remove classloader dependency on jkernel)
Alan Bateman
Alan.Bateman at Sun.COM
Tue Oct 6 08:56:54 UTC 2009
Mandy Chung wrote:
> Alan, Rémi,
>
> Thanks for the review. The revised webrev is at:
> http://cr.openjdk.java.net/~mchung/6612680/jdk-webrev.01/
> http://cr.openjdk.java.net/~mchung/6612680/hotspot-webrev.01/
The changes you pushed to jdk7/tl/jdk look fine - thanks for doing the
clean-ups (and for the explanation for the make/sun/jkernel/Makefile
change). One other thing I forgot to mention is that
System.initializeSystemClasses can be restored - as I recall, that was
hacked to workaround issues encountered with the download manager
running before the system classes were completely initialized.
Minor comment on thread.cpp is that it might be neater if
set_jkernel_boot_classloader_hook were:
if (k != NULL) {
JavaValue result(T_VOID);
JavaCalls::call_static( ... );
}
Otherwise, good job!
-Alan.
More information about the core-libs-dev
mailing list