On 3/9/18 8:25 AM, Adam Petcher wrote:
New webrev: http://cr.openjdk.java.net/~apetcher/8171277/webrev.01/


I think somewhere there should be a sentence or two on the difference between XECKeys and ECKeys and when you would want to use each. This is important enough that I think some detail should be in the javadoc to help users distinguish them. Perhaps put it in the XEC class description. Maybe more details can go in the JCA security guide.

I added a sentence to the comments in XECKey, but I'm not sure how much we should say here. I don't know if we should get into mathematical details. Take a look at the new wording and let me know if I should add more information.

Looks good. I would probably also put a similar sentence in the XECPublicKey and XECPrivateKey classes just so it is not missed.

All other changes look good.

--Sean

Reply via email to