This made sense, than you, Mandy.

Please review new version:
http://cr.openjdk.java.net/~shurailine/8158670/webrev.02/

Shura

> On Jul 2, 2016, at 3:26 PM, Mandy Chung <mandy.ch...@oracle.com> wrote:
> 
> 
>> 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
> 

Reply via email to