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
