On 5/10/19 5:04 AM, Weijun Wang wrote:
Please take a review at the CSR at

    https://bugs.openjdk.java.net/browse/JDK-8223682

Add an "@since 13" to the new constant.


The text is copied from 4.5.2 and 4.5.2.3 of 
https://www.w3.org/TR/xmldsig-core/.

One thing I am not sure is that I haven't include the definition of 
ECParameters in 4.5.2.3.1 which is quite long. While I added an @implNote 
saying ECParameters is not supported in this implementation, I understand 
@implNote is not really a part of the spec  and it's not the reason I omit the 
definition of ECParameters. If you believe it should be included, I'll add it, 
or add a link.

I would leave out the implNote completely. It doesn't seem the right place to put it, since this is an interface. What is the reason ECParameters is not supported?

--Sean


The code change is exactly the same as the specification, so no webrev.

Thanks,
Max

Reply via email to