RFR (S): 8u40 Backport of 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data

Bengt Rutisson bengt.rutisson at oracle.com
Tue Aug 19 08:33:06 UTC 2014


Hi Thomas,

I only checked the heapRegionSeq.hpp file and that looks ok to me.

Reviewed.

Bengt


On 2014-08-19 10:20, Thomas Schatzl wrote:
> Hi all,
>
>    can I have reviews for the backport of 8054818?
>
> The file heapRegionSeq.hpp did not apply because of an unfixed typo.
> I.e. the current file reads "Empty contructor", while the patch expects
> "Empty constructor" in some comment.
>
> New webrev:
> http://cr.openjdk.java.net/~tschatzl/8054818/webrev.8u40/
>
> Original change:
> http://cr.openjdk.java.net/~tschatzl/8054818/webrev.1/
>
> Testing:
> jprt
>
> Thanks,
>    Thomas
>
>




More information about the hotspot-gc-dev mailing list