> On Apr 13, 2016, at 10:29 AM, Alexandre (Shura) Iline 
> <alexandre.il...@oracle.com> wrote:
> 
> Hi,
> 
> Could you be so kind to take a look on this fix:
> http://cr.openjdk.java.net/~shurailine/8154182/webrev.00/

Looks okay.  Some suggestion:

You only need to walk one root:
   root = fs.getPath("/modules”)

and instead of caching the available modules, you could
   .filter(x -> Layer.boot().findModule(x.getName(1).toString())).isPresent()

I can sponsor your patch - please send me the changeset.

Mandy

Reply via email to