Change looks fine. One nit: the extra space at the beginning of line 24 looks strange.
Thanks Max On 11/30/2016 5:27 PM, Sibabrata Sahoo wrote:
Hi, Please review the patch for, JBS: https://bugs.openjdk.java.net/browse/JDK-8170247 Webrev: http://cr.openjdk.java.net/~ssahoo/8170247/webrev.00/ Description: The Test was failing to handle the expected failure for invalid parameters, when the SecureRandom parameter size is > algorithm running with unlimited policy. I have modified the condition to handle the expected failures correctly. The change is only applicable to the “if” condition(Line:200). The other 2 lines are minor changes related to statement performance and renaming a variable. Thanks, Siba