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