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