Hi Claes, the patch is fine for me with the minor nitpick that the static final containing Unsafe should be called UNSAFE and not just U.
do you know why BufferedInputStream is loaded in first place during the startup of the VM ? regards, Rémi ----- Mail original ----- > De: "Claes Redestad" <[email protected]> > À: "core-libs-dev Libs" <[email protected]> > Envoyé: Dimanche 3 Avril 2016 02:51:35 > Objet: RFR: 8153334: Replace BufferedInputStreams use of > AtomicReferenceFieldUpdater with Unsafe > > Hi, > > BufferedInputStream is loaded early, and uses > AtomicReferenceFieldUpdater to provide > CAS functionality to allow for closing streams asynchronously. Using > Unsafe directly instead > does the exact same thing in the end, but avoids loading a few (4) > classes and thus brings > us a small startup improvement. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8153334 > Webrev: http://cr.openjdk.java.net/~redestad/8153334/webrev.01/ > > Thanks! > > /Claes >
