RFR (S): 8066102: Clean up HeapRegionRemSet files

Thomas Schatzl thomas.schatzl at oracle.com
Tue Dec 2 08:14:55 UTC 2014


Hi Kim,

On Mon, 2014-12-01 at 16:36 -0500, Kim Barrett wrote:
> On Dec 1, 2014, at 7:07 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
> >> Regarding Mikael's comment:
> >>> In heapRegionRemSet.hpp:
> >>> +  // One bits in the bitmaps indicate that the given region or card is live.
> >>> typo: "One bit in one of the bitmaps indicate that the.."
> >> 
> >> I think the existing comment is correct, but might be clearer if it
> >> said
> >> 
> >> "Set bits in the bitmaps ..."
> > 
> > Fixed.
> > 
> > http://cr.openjdk.java.net/~tschatzl/8066102/webrev.1 (this is a full
> > webrev again, really only fixed that comment).
> 
> I’m not seeing any change from the original webrev.

Fixed and reuploaded. Sorry.

Thanks,
  Thomas





More information about the hotspot-gc-dev mailing list