Request for review: 8016302: Change type of the number of GC workers to unsigned int (2)
Stefan Karlsson
stefan.karlsson at oracle.com
Thu Apr 3 12:38:06 UTC 2014
On 2014-04-03 14:40, Thomas Schatzl wrote:
> Hi,
>
> On Thu, 2014-04-03 at 15:41 +0400, Vladimir Kempik wrote:
>> Hello
>>
>> I've used UINT32_FORMAT because:
>>
>> 1) previous format type for worker was INT32_FORMAT, when worker was int
>>
>> 2) there is no such thing as UINT_FORMAT, there is only UINTX_FORMAT
>>
>> Do you think UINTX_FORMAT is better than UINT32_FORMAT for this case ?
> UINTX_FORMAT or uintx is an unsigned data type that has 32 or 64 bit
> size depending on the processes' word size.
>
> I think UINT32_FORMAT is correct here, and used elsewhere for uint
> variables. Afaik uint is 32 bits on all platforms we use.
Why don't we just use %u?
thanks,
StefanK
>
> Thanks,
> Thomas
>
>
More information about the hotspot-gc-dev
mailing list