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

Mandy Chung mandy.chung at oracle.com
Wed Oct 30 15:11:06 UTC 2013


On 10/30/2013 3:54 AM, Daniel Fuchs wrote:
> On 10/30/13 3:06 AM, 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/
>
> Looks good. What's the purpose of the split() method in Basic.java?
> It looks as if it's not used. Is it a trap for reviewers? ;-) 

Good catch.  Will remove it before the push. I initially was thinking to 
have that method to do the work to move the files.

Thanks
Mandy



More information about the core-libs-dev mailing list