Hi Mandy,

Thanks for your review!

Calvin

On 4/27/20 2:39 PM, Mandy Chung wrote:

On 4/27/20 12:21 PM, Calvin Cheung wrote:
JBS: https://bugs.openjdk.java.net/browse/JDK-8241815
webrev: http://cr.openjdk.java.net/~ccheung/jdk15/8241815/webrev.00/

This change is to avoid java up call of definePackage() while loading a shared app/platform class originated from the module image since the packages from module image are already created duringĀ  VM bootstrap in Module::define_module().

(including core-libs-dev since the change removes two methods in jdk/internal/loader/ClassLoaders.java)


java.lang.Package for named modules are automatically defined and this patch looks okay to me.

Mandy

Reply via email to