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

sangheon.kim sangheon.kim at oracle.com
Thu Oct 19 19:14:44 UTC 2017


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.

Thanks,
Sangheon


> Testing:
> local compilation, jprt
>
> Thanks,
>    Thomas
>




More information about the hotspot-gc-dev mailing list