Review Request for 8027481: jdeps to handle classes with the same package name ...

Alan Bateman Alan.Bateman at oracle.com
Wed Oct 30 14:38:58 UTC 2013


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.



More information about the core-libs-dev mailing list