Re: Review for 8006225: tools/jdeps/Basic.java fails with AssertionError

2013-02-14 Thread Alan Bateman
On 14/02/2013 01:52, Mandy Chung wrote: This fixes 8006225: tools/jdeps/Basic.java fails with AssertionError. Webrev: http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8006225/webrev.00/ jdeps maintains a static list of analyzed classes, each of which is expected to be from one archive.

Re: Review for 8006225: tools/jdeps/Basic.java fails with AssertionError

2013-02-14 Thread Mandy Chung
On 2/14/13 7:43 AM, Alan Bateman wrote: On 14/02/2013 01:52, Mandy Chung wrote: This fixes 8006225: tools/jdeps/Basic.java fails with AssertionError. Webrev: http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8006225/webrev.00/ jdeps maintains a static list of analyzed classes, each of which

Review for 8006225: tools/jdeps/Basic.java fails with AssertionError

2013-02-13 Thread Mandy Chung
This fixes 8006225: tools/jdeps/Basic.java fails with AssertionError. Webrev: http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8006225/webrev.00/ jdeps maintains a static list of analyzed classes, each of which is expected to be from one archive. test/tools/jdeps/Basic.java calls