7116957: javax.script.ScriptEngineManager should use java.util.ServiceLoader to lookup service providers

Alan Bateman Alan.Bateman at oracle.com
Fri Dec 2 13:54:18 UTC 2011

On 02/12/2011 11:30, Chris Hegarty wrote:
> Alan,
> Would you mind taking a quick look at the updated webrev?
> Trivially, I fixed a few style issues in HttpServerProvider (a nit of 
> mine) and updated the commented code in FtpClientProvider (in case it 
> ever gets enabled ). Let's remove any possibility of sun.misc.Service 
> every reappearing ;-)
> http://cr.openjdk.java.net/~chegar/7116957/webrev.01/webrev/
> -Chris.
Loos fine to me. In FtpClientProvider.java I would probably have just 
removed this code.


More information about the core-libs-dev mailing list