Yes, I'll do that. Thanks!
/Jesper
On 19/3/13 10:53 AM, Bengt Rutisson wrote:
Hi Jesper,
I think you can remove the YoungGCInfo class all together. Other than that it
looks good.
Bengt
On 3/18/13 10:04 PM, Jesper Wilhelmsson wrote:
Could I have a couple of reviews for this really small change?
Bug 8010227: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8010227
The young garbage collection tracing event has a boolean to indicate if a
promotion failed has occurred or not. This information is also available in
its own event so the variable is duplicating the information. It was
introduced for convenience to make it easy to see if a collection got
promotion failed or not when looking at the GC events. However, the existence
of this variable complicates the implementation of the evacuation failed event
for G1, so we have decided to remove it.
Webrev: http://cr.openjdk.java.net/~jwilhelm/8010227/webrev/
Thanks,
/Jesper
