<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
Thanks, Ramki!<br>
<br>
John Cuthbertson also looked at it, so I'm all set to push it now.<br>
<br>
Bengt<br>
<br>
On 2011-11-11 00:24, Srinivas Ramakrishna wrote:
<blockquote
cite="mid:CABzyjykJ5+_qphz3iDCfamDJ6gt2GOehAu_RbQH-y8KN_wALoA@mail.gmail.com"
type="cite">Looks good to me; reviewed!<br>
<br>
-- ramki (OpenJDK id: ysr)<br>
<br>
<div class="gmail_quote">On Wed, Nov 9, 2011 at 2:36 PM, Bengt
Rutisson <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:bengt.rutisson@oracle.com">bengt.rutisson@oracle.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt
0.8ex; border-left: 1px solid rgb(204, 204, 204);
padding-left: 1ex;"><br>
Hi all,<br>
<br>
Could I have a couple of reviews for this very simple change?<br>
<a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Ebrutisso/7110190/webrev.01/"
target="_blank">http://cr.openjdk.java.net/~brutisso/7110190/webrev.01/</a><br>
<br>
GCCause::to_string() was missing a case for
_adaptive_size_policy. I added this case and also moved around
a couple of the existing case statements to make sure that the
statements are in the same order as they are defined in the
enum GCCause::Cause.<br>
<br>
I choose the string "Ergonomics" to be consistent with
CMSExpansionCause::to_string().<br>
<br>
CR:<br>
7110190 GCCause::to_string missing case for
_adaptive_size_policy<br>
<a moz-do-not-send="true"
href="http://monaco.us.oracle.com/detail.jsf?cr=7110190"
target="_blank">http://monaco.us.oracle.com/detail.jsf?cr=7110190</a><br>
<font color="#888888">
<br>
Bengt<br>
</font></blockquote>
</div>
<br>
</blockquote>
<br>
</body>
</html>