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


Reply via email to