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

Tony Printezis tony.printezis at oracle.com
Wed Jun 1 12:04:45 UTC 2011


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