RFR: 8144077: Add getter for G1CollectorPolicy::_collectionSetChooser
Erik Helin
erik.helin at oracle.com
Thu Nov 26 12:59:53 UTC 2015
On 2015-11-26, Mikael Gerdin wrote:
> On 2015-11-26 10:08, Thomas Schatzl wrote:
> >Hi,
> >
> >On Wed, 2015-11-25 at 19:56 +0100, Erik Helin wrote:
> >>Hi all,
> >>
> >>this small patch just introduces a getter for
> >>G1CollectorPolicy::_collectionSetChooser in order to make the code in
> >>G1CollectorPolicy slightly less verbose.
> >>
> >>Bug:
> >>https://bugs.openjdk.java.net/browse/JDK-8144077
> >>
> >>Webrev:
> >>http://cr.openjdk.java.net/~ehelin/8144077/webrev.00/
> >>
> >>Testing:
> >>- GC test suite with full verification.
> >
> > could you also rename _collectionSetChooser using the standard naming
> >conventions while you are there? I would be fine with _cset_chooser.
>
> Just FTR, that's ok with me as well.
Mikael, Thomas,
thanks for the reviews! I will do the renaming before I push.
Thanks,
Erik
> /Mikael
>
> >
> >Looks good otherwise.
> >
> >Thanks,
> > Thomas
> >
>
More information about the hotspot-gc-dev
mailing list