On 30/10/2013 02:06, Mandy Chung wrote:
I have added the test cases to verify dependency on javax.crypto and
also a class in a SE package (javax.activity in the test case). The
updated webrev:
http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8027481/webrev.01/
Mandy
I've looked through the updated webrev and the changes look okay to me
(I didn't see anything obviously wrong). It is bit annoying to have to
special case javax.crypto, I've often wondered if it really necessary to
continue to put these classes in jce.jar.
The -showLabel looks useful but I could imagine the output being very
cluttered (time will tell).
The new test classes have 2012 in the header, otherwise looks okay.
-Alan.
