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 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 href="http://cr.openjdk.java.net/%7Ebrutisso/7110190/webrev.01/" target="_blank">http://cr.openjdk.java.net/~<u></u>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()<u></u>.<br>
<br>
CR:<br>
7110190 GCCause::to_string missing case for _adaptive_size_policy<br>
<a href="http://monaco.us.oracle.com/detail.jsf?cr=7110190" target="_blank">http://monaco.us.oracle.com/<u></u>detail.jsf?cr=7110190</a><br><font color="#888888">
<br>
Bengt<br>
</font></blockquote></div><br>