> On Apr 13, 2016, at 10:29 AM, Alexandre (Shura) Iline > <[email protected]> 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
