Looks fine to me except a minor comment:

    throw new IllegalArgumentException(
        "Illegal strength: " + strength);

I may add more info in the exception message, for example "Illegal
security strength".

Xuelei

On 6/12/2016 9:45 AM, Wang Weijun wrote:
> Please take a review at
> 
>   http://cr.openjdk.java.net/~weijun/8158534/webrev.00
> 
> Thanks
> Max
> 

Reply via email to