Review for 8006225: tools/jdeps/Basic.java fails with AssertionError
Alan.Bateman at oracle.com
Thu Feb 14 15:43:36 UTC 2013
On 14/02/2013 01:52, Mandy Chung wrote:
> This fixes 8006225: tools/jdeps/Basic.java fails with AssertionError.
> jdeps maintains a static list of analyzed classes, each of which is
> expected to be from one archive. test/tools/jdeps/Basic.java calls
> com.sun.tools.jdeps.Main multiple times with different classpath and
> thus the same class came from different path in a different invocation
> of jdeps. Fix jdeps to keep the data in instance variables. I
> refactored the analysis from JdepsTask into a new Analyzer class to
> ease further enhancement.
This looks okay to me, I guess it's only going to be a test that runs it
more than once in the same VM that would have noticed this, someone
using the tool from the command-line would not.
More information about the core-libs-dev