Thanks for the review. I have updated the MessageDigestSpi2 class accordingly:
http://cr.openjdk.java.net/~valeriep/8165275/webrev.01/src/java.base/share/classes/sun/security/util/MessageDigestSpi2.java.html

Valerie

On 10/10/2016 3:15 AM, Xuelei Fan wrote:
As this API is only used for legacy SSL 3.0, alternatively we may deprecate and remove the support of MessageDigestSpi2 in the future. I may add a note to MessageDigestSpi2, like:

   This is for SSL 3.0 implementation only, please don't
   use it for other purpose.

Otherwise, looks fine to me.

Thanks,
Xuelei

On 10/5/2016 6:07 AM, Valerie Peng wrote:
Hi Xuelei,

Could you please help reviewing this JSSE-related code refactoring?

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

Thanks,
Valerie

Reply via email to