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

2011-12-02 Thread Alan Bateman
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

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

2011-12-01 Thread Chris Hegarty
This is a follow up to an issue that came up during discussion of another fix. Essentially, ScriptEngineManager should use j.u.ServiceLoader rather than sun.misc.Service. http://cr.openjdk.java.net/~chegar/7116957/webrev.00/webrev/ -Chris. P.S. and fix up imports in HttpServerProvider