RFR 8068938: javax.script package description should specify use of ServiceLoader

[Alan Bateman]

On 11/01/2016 05:03, Sundararajan Athijegannathan wrote:
> Please review http://cr.openjdk.java.net/~sundar/8068938/webrev.00/ 
> for https://bugs.openjdk.java.net/browse/JDK-8068938
>
> This is a doc-only change to mention that ScriptEngineFactory 
> instances are loaded using java.util.ServiceLoader API and avoid 
> wording on META-INF/services [ which is just one impl. mechanism used 
> by ServiceLoader ]. This will allow the ScriptEngineFactory 
> implementations to be deployed as modules without the service 
> configuration file.
The original wording has "current ClassLoader" which isn't very clear, 
does it mean the caller's class loader or the TCCL? The updated wording 
removes this but leaves it ambiguous as to how ScriptEngineFactory 
providers are located. Can it reference someone where in the 
javax.script javadoc where this is more clearly specified?

<code>ScriptEngineFactories</code>, might be better as {@code 
ScriptEngineFactory} instances.

I would be good to keep the line length somewhat consistent with the 
rest of the class description. Having have one >100 char line will also 
be annoying for future side-by-side diffs.

I assume this can go into jdk9/dev, no need for it to go into jigsaw/jake.

-Alan.