> On Jul 1, 2016, at 6:20 PM, Alexandre (Shura) Iline > <alexandre.il...@oracle.com> wrote: > > Please review the new version of the fix. > http://cr.openjdk.java.net/~shurailine/8158670/webrev.01/
This looks much better. Small comment: you can use Layer::findModule and also Optional::ifPresent, like this: boot.findModule("jdk.crypto.ucryptoā€¯) .ifPresent(m -> expected.add("com.oracle.security.ucrypto.UcryptoProviderā€¯)); Mandy