RFR (XS): 8073654: Marking statistics should use size_t
derek.white at oracle.com
Fri Mar 6 18:09:38 UTC 2015
Looks good to me too. Jesper already asked the only question I had.
- Derek (drwhite)
On 3/6/15 11:32 AM, Sangheon Kim wrote:
> Hi Jesper,
> Thanks for reviewing this.
> On 03/06/2015 02:26 AM, Jesper Wilhelmsson wrote:
>> Looks good.
>> Just curious, what motivated the first change, moving the variables
>> in concurrentMark.cpp?
> I just wanted to have same order as the declaration at the header file.
>> Also, your webrevs would be easier to review if you used the latest
>> version of webrev found here:
> Oh, I see. Thanks for the pointer.
>> Sangheon Kim skrev den 6/3/15 02:23:
>>> Hi all,
>>> Can I have reviews for this simple fix for changing variables for
>>> from 'int' to 'size_t'.
>>> This is to avoid overflows on large heaps.
More information about the hotspot-gc-dev