<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix"><br>
Hi Jesper,<br>
<br>
I think you can remove the
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
YoungGCInfo class all together. Other than that it looks good.<br>
<br>
Bengt<br>
<br>
On 3/18/13 10:04 PM, Jesper Wilhelmsson wrote:<br>
</div>
<blockquote cite="mid:5147815E.20601@oracle.com" type="cite">Could I
have a couple of reviews for this really small change?
<br>
<br>
Bug 8010227:
<a class="moz-txt-link-freetext" href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8010227">http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8010227</a>
<br>
<br>
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.
<br>
<br>
Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~jwilhelm/8010227/webrev/">http://cr.openjdk.java.net/~jwilhelm/8010227/webrev/</a>
<br>
<br>
Thanks,
<br>
/Jesper
<br>
</blockquote>
<br>
</body>
</html>