Request for review (XXS): 7110190 GCCause::to_string missing case for _adaptive_size_policy

Bengt Rutisson bengt.rutisson at oracle.com
Thu Nov 10 23:28:01 UTC 2011


Thanks, Ramki!

John Cuthbertson also looked at it, so I'm all set to push it now.

Bengt

On 2011-11-11 00:24, Srinivas Ramakrishna wrote:
> Looks good to me; reviewed!
>
> -- ramki (OpenJDK id: ysr)
>
> On Wed, Nov 9, 2011 at 2:36 PM, Bengt Rutisson 
> <bengt.rutisson at oracle.com <mailto:bengt.rutisson at oracle.com>> wrote:
>
>
>     Hi all,
>
>     Could I have a couple of reviews for this very simple change?
>     http://cr.openjdk.java.net/~brutisso/7110190/webrev.01/
>     <http://cr.openjdk.java.net/%7Ebrutisso/7110190/webrev.01/>
>
>     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.
>
>     I choose the string "Ergonomics" to be consistent with
>     CMSExpansionCause::to_string().
>
>     CR:
>     7110190 GCCause::to_string missing case for _adaptive_size_policy
>     http://monaco.us.oracle.com/detail.jsf?cr=7110190
>
>     Bengt
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20111111/3fca37c2/attachment.htm>


More information about the hotspot-gc-dev mailing list