CRR: 6804436: G1: heap region indices should be size_t (S)
Stefan Karlsson
stefan.karlsson at oracle.com
Wed Jun 1 13:11:19 UTC 2011
Looks good.
StefanK
On 06/01/2011 02:04 PM, Tony Printezis wrote:
> Hi,
>
> Could I please have a couple of code reviews for this simple change:
>
> http://cr.openjdk.java.net/~tonyp/6804436/webrev.0/
>
> Some of the int -> size_t changes were done as part of the
> HeapRegionSeq cleanup (see the separate code review request for
> 7042285 and 7045330 that I just sent out). This change does the rest
> (specifically: the changes related to the HeapRegion class).
>
> Note: I know that the webrev index says that some files have more than
> one change applied to them. This is incorrect and a side-effect of
> generating the webrev from a workspace with multiple patches stacked
> up. The diffs are actually correct (I checked).
>
> Tony
>
More information about the hotspot-gc-dev
mailing list