8174823: Module system implementation refresh (3/2017)
Alan Bateman
Alan.Bateman at oracle.com
Wed Mar 22 17:27:08 UTC 2017
On 22/03/2017 17:20, Remi Forax wrote:
> So ModuleTarget.osVersion is removed, i will have to update ASM accordingly.
ModuleTarget is JDK-specific now, I think JEP 261 will be the place to
document it.
>
> In jdk/internal/module/ModuleBootstrap.java,
> addExtraExportsOrOpens(bootLayer, extraOpens, true, builder);
> should be
> addExtraExportsOrOpens(bootLayer, extraOpens, ADD_OPENS, builder);
> there is no point to introduce a boolean here.
This hasn't changed in this patch but I agree it could be a bit cleaner.
> In java.lang.reflect.Method,
> the StringJoiner is created even if there are no parameters.
>
> AccessibleObject.toShortString() should have a javadoc.
>
The javadoc is brief but it's not a public method.
-Alan
More information about the hotspot-runtime-dev
mailing list