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