Very minor comments/tweaks.

On 8/18/2017 7:01 AM, Seán Coffey wrote:
Looking to backport 8170157 to jdk8u-dev. The 8170245 test bug also gets pulled in for this port since some tests need cleaning up to deal with unlimited crypto environment.

webrev : http://cr.openjdk.java.net/~coffeys/webrev.8170157.8u.01/webrev/index.html

Update copyright dates. Looks like the original work was done in December 2016, but this is our actual push.

JceSecurity.java
================
265:  Suggestion since this is JDK/JRE specific:

jre/lib/security
->
<java-home>/lib/security

Dynamic.java
============
Is the removal of the separate ECB case because the parameter will just be ignored? Why was this a failing test case?

Thanks.

Brad



Reply via email to