RFR [10]: 8185362: Replace use of AtomicReferenceFieldUpdater from BufferedInputStream with Unsafe
Xueming Shen
xueming.shen at oracle.com
Mon Aug 21 15:24:27 UTC 2017
+1
On 8/21/17, 8:09 AM, Claes Redestad wrote:
> Hi Martin,
>
> On 08/21/2017 04:33 PM, Martin Buchholz wrote:
>> This comment still seems applicable. Can we move it into the
>> docstring for buf?
>>
>> We use nullness
>> - * of buf[] as primary indicator that this stream is closed. (The
>> - * "in" field is also nulled out on close.)
>>
>
> this is a reasonable request, the catch is that the field is protected
> and thus visible
> in javadoc, so it needs to be updated as a CSR(?). Perhaps we can move
> it to a regular
> comment associated with the field to avoid this implementation detail
> showing up in
> javadocs?
>
> http://cr.openjdk.java.net/~redestad/8185362/jdk.02/
>
> Thanks!
>
> /Claes
More information about the core-libs-dev
mailing list