On Sep 3, 2013, at 5:11 PM, Brian Burkhalter wrote: > On Sep 3, 2013, at 5:09 PM, Doug Lea wrote: > >> Only adding isProbablePrime seems to be an OK conservative >> move: no existing usages would be affected, but users would >> need to somehow be told that they could improve performance >> by changing their code to use the new method with >> ThreadLocalRandom.current() as argument. > > This is what I was thinking. Some verbiage update would be in order.
I have updated the webrev http://cr.openjdk.java.net/~bpb/7189139/ to add the two-parameter version of isProbablePrime() which was discussed. Naturally a CCC request would be needed in the event this were to go forward. Comments welcome. Thanks, Brian