RFR [10]: 8185362: Replace use of AtomicReferenceFieldUpdater from BufferedInputStream with Unsafe
Aleksey Shipilev
shade at redhat.com
Mon Aug 21 10:29:52 UTC 2017
On 08/21/2017 12:20 PM, Claes Redestad wrote:
> Webrev: http://cr.openjdk.java.net/~redestad/8185362/jdk.00/
*) Should be static *final*, otherwise you miss constant folding for Unsafe accesses:
66 private static long BUF_OFFSET = U.objectFieldOffset(BufferedInputStream.class, "buf");
*) While you are at it, maybe switch to proper Java style here, e.g. "volatile byte[] buf"?
73 protected volatile byte buf[];
Thanks,
-Aleksey
More information about the core-libs-dev
mailing list