Please review fix to StackOverflowError in jigsaw tracing

Alan Bateman Alan.Bateman at oracle.com
Mon May 24 05:55:48 PDT 2010


Mandy Chung wrote:
> :
> Thanks for the comment.  I revise the fix:
>    http://cr.openjdk.java.net/~mchung/6951456/webrev.01/
This looks better.  A couple of comments:

Is "inited" used now? It doesn't appear to be and so can be removed.

Cache.print does more than the method name suggests. Maybe Cache.drain 
or printAndClear?

Otherwise I think it's okay.

-Alan.



More information about the jigsaw-dev mailing list