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