Thanks for the comment, I have removed extra check.

http://cr.openjdk.java.net/~rhalade/8151834/webrev.01/

Thanks,
Rajan

On 4/14/16 12:51 PM, Wang Weijun wrote:
Looks good.

Btw, since there is already the @requires tag, I think lines 52-56 are no 
longer needed.

Thanks
Max

在 2016年4月15日,03:44,Rajan Halade <rajan.hal...@oracle.com> 写道:

Hi Max,

Please review this patch to fix SmallPrimeExponentP test. It now uses fixed 
seed value to generate keypair. I did several rounds of testing on different 
machines to make sure that primes 63/65 are generated in fixed number of 
iterations. Other updates to test are done to include detailed summary and to 
log debug information.

Bug: https://bugs.openjdk.java.net/browse/JDK-8151834
Webrev: http://cr.openjdk.java.net/~rhalade/8151834/webrev.00/

Thank you for your help with the fix to have constant seed.

Thanks,
Rajan

Reply via email to