Thanks. Removed that unused import and pushed the changes. -Sundar
On 5/6/2016 6:22 PM, Hannes Wallnoefer wrote: > +1 > > Minor issue: there's an unused import of java.util.ArrayList in > Bootstrap.java > > Hannes > > Am 2016-05-05 um 06:39 schrieb Sundararajan Athijegannathan: >> http://mail.openjdk.java.net/pipermail/nashorn-dev/2016-March/006026.html >> >> >> Please review http://cr.openjdk.java.net/~sundar/8150731/webrev.02/ >> >> I've update the webrev to use provider clause in module-info.java >> instead of the older META-INF/services way of exposing a service. >> >> Also, I've split the test as suggested by Michael Haupt suggested >> earlier. >> >> Thanks, >> >> -Sundar >> >
