CRR: 6804436: G1: heap region indices should be size_t (S)

Tony Printezis tony.printezis at oracle.com
Wed Jun 1 22:18:25 UTC 2011


Stefan and John,

Thanks! All set with this one too.

Tony

On 06/01/2011 03:11 PM, Stefan Karlsson wrote:
> 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