RFR (M): 8208498: Put archive regions into a first-class HeapRegionSet

Thomas Schatzl thomas.schatzl at oracle.com
Tue Aug 21 12:06:04 UTC 2018


Hi,

On Mon, 2018-08-20 at 16:58 +0200, Thomas Schatzl wrote:
> Hi,
> 
> 
[...]
> > Anyway, that's complex enough to be its own RFE, so I'd be happy if
> > you just put the For* constants in an enum. :)
> 
> I agree. I have some patches for cleaning this up already,
> effectively removing this new enum. However it just adds some noise
> for this particular change.
> 

  as you might have noticed, I sent out the review request for
"8209700: Remove HeapRegionSetBase::RegionSetKind for a more flexible
approach" in a separate thread.

Thanks,
  Thomas



More information about the hotspot-gc-dev mailing list