RFR: 8145680: Remove unnecessary explicit initialization of volatile variables in java.base
Ulf Zibis
Ulf.Zibis at CoSoCo.de
Mon Dec 21 11:32:44 UTC 2015
Am 21.12.2015 um 11:31 schrieb Ulf:
> Am 21.12.2015 um 00:06 schrieb Claes Redestad:
>> Hi all,
>>
>> On 2015-12-20 20:43, Ulf wrote:
>>> Hi Claes,
>>>
>>> I had a very short look and found in j.l.Thread:
>>> 211 * Java thread status for tools, 0 indicate thread 'not yet started'
>>> I guess you meant:
>>> 211 * Java thread status for tools; 0 indicates: thread 'not yet started'.
>>>
>>> -Ulf
>>
>> /*
>> * Java thread status for tools, default indicates thread 'not yet started'
>> */
>>
>> OK?
>
> To my feeling, 0 is more clear than default.
> Often enough the default for a value is other than 0, so the user ends up to search for "What is
> the default of this value?".
> Also I think, the expression "thread 'not yet started'" should be separated in some way from the
> rest of the phrase, maybe just shift the "'" one word before.
Using both seems a good compromise, and remember punctuation please:
* Java thread status for tools(,|;) default 0 indicates: thread 'not yet started'.
-Ulf
More information about the core-libs-dev
mailing list