RFR (XXS): 8189665: Change id parameter of G1ConcurrentMark::task() to uint

Stefan Johansson stefan.johansson at oracle.com
Fri Oct 20 08:09:14 UTC 2017



On 2017-10-19 21:14, sangheon.kim wrote:
> Hi Thomas,
>
> On 10/19/2017 07:29 AM, Thomas Schatzl wrote:
>> Hi all,
>>
>>    can I get reviews for this cleanup change that changes the type of
>> the "id" parameter of the mentioned method from int to uint?
>>
>> It only ever gets passed worker_id's (or 0's) which have been uints for
>> a long time now.
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8189665
>> Webrev:
>> http://cr.openjdk.java.net/~tschatzl/8189665/webrev/
> Looks good to me.
>
+1

Cheers,
StefanJ
> Thanks,
> Sangheon
>
>
>> Testing:
>> local compilation, jprt
>>
>> Thanks,
>>    Thomas
>>
>




More information about the hotspot-gc-dev mailing list