On 2015-09-16 14:51, Alan Bateman wrote:
On 16/09/2015 13:28, Magnus Ihse Bursie wrote:
On 2015-09-11 18:00, Attila Szegedi wrote:
Please review the revised changes for JDK-8135251 "Use
Unsafe.defineAnonymousClass for loading Nashorn script code”
<https://bugs.openjdk.java.net/browse/JDK-8135251>
The revision incorporates the following changes:
- the feature can be disabled by setting the
"nashorn.disableVmAnonymousClasses system property”
- jdk9 root-level modules.xml explicitly exports sun.misc to
jdk.scripting.nashorn
- the changes to the CodeInstaller and Compiler API have since been
separately committed, so they are no longer part of this issue,
making its surface area smaller.
Webrevs are found at:
http://cr.openjdk.java.net/~attila/8135251/webrev.jdk9
<http://cr.openjdk.java.net/~attila/8135251/webrev.jdk9>
http://cr.openjdk.java.net/~attila/8135251/webrev.jdk9-nashorn
<http://cr.openjdk.java.net/~attila/8135251/webrev.jdk9-nashorn>
The fist webrev is for jdk9/modules.xml; the change simply adds an
export of sun.misc to jdk.scripting.nashorn; it is also the reason
why I included build-dev on this review.
Actually, I think it's a better idea to direct module.xml related
changes to jigsaw-dev. (Cc:ing them).
The changes to modules.xml looks fine. Also just to say that
modules.xml has been removed in the jigsaw/jake forest as it was just
a temporary document to maintain the module graph until the module
system came along.
Does this mean that updates to modules.xml between now and the
integration-to-come of jigsaw/jake do not anymore need specific approval?
/Magnus
-Alan.
