On Mon, Apr 9, 2018 at 1:25 PM, Chris Hegarty <chris.hega...@oracle.com> wrote:
> > > On 9 Apr 2018, at 20:31, Alan Bateman <alan.bate...@oracle.com> wrote: > > > >> On 09/04/2018 18:54, Martin Buchholz wrote: > >> Another little cleanup to make Google's race detector happy. > >> > >> 8201327: Make Sensor deeply immutably thread safe > >> http://cr.openjdk.java.net/~martin/webrevs/jdk/Sensor-init/ < > http://cr.openjdk.java.net/%7Emartin/webrevs/jdk/Sensor-init/> > >> https://bugs.openjdk.java.net/browse/JDK-8201327 > >> > > This looks okay to me, > > +1 > > > except maybe the "initially false" and "initially 0" comments as they > are just documenting the default values and don't add anything. > > Yeah, I’ve done this before too, especially when removing default values ( > as if to prevent them from inadvertently creeping back in ). > If only there was standard concise wording for "intentionally using the field's default value, which is slightly more efficient and slightly more theoretically thread-safe, even though slightly less readable" ... // VM-initialized to false