RFR: jsr166 jdk9 integration wave 14

Martin Buchholz martinrb at google.com
Wed Feb 1 15:41:19 UTC 2017


On Tue, Jan 31, 2017 at 4:03 PM, Paul Sandoz <paul.sandoz at oracle.com> wrote:

>
> On Tue, Jan 31, 2017 at 12:32 PM, Paul Sandoz <paul.sandoz at oracle.com>
> wrote:
>
>>
>> 1057     static final VarHandle ITEM;
>> 1058     static final VarHandle NEXT;
>>
>> Suggest a comment as to why they are package private, due to access via a
>> nested class. (Same in LinkedTransferQueue).
>>
>>
> package private is almost always used for nestmates in j.u.c.; a comment
> doesn't seem worthwhile.
>
>
>
> Ok, may i suggest renaming to NODE_ITEM and NODE_NEXT?
>

We prefer the shorter ITEM and NEXT, especially for nestmates where no
ambiguity is present.  There's a stronger case for your own  THREAD_PROBE,
although we might have happily stuck with PROBE.


More information about the core-libs-dev mailing list