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