Just wanted to change this one, and noticed this > > +DEF_SANITIZER_BUILTIN(BUILT_IN_TSAN_VOLATILE_WRITE16, > > "__tsan_volatile_write16", > > + BT_FN_VOID_PTR, ATTR_NOTHROW_LEAF_LIST)
is precisely 80 characters. So if I read the style guide right, it's <= 80 chars (and not < 80 chars), right? > This last entry is already too long (line limit 80 chars), so should be > wrapped like: > DEF_SANITIZER_BUILTIN(BUILT_IN_TSAN_VOLATILE_WRITE16, > "__tsan_volatile_write16", BT_FN_VOID_PTR, > ATTR_NOTHROW_LEAF_LIST) > instead. So I think it can stay. Thanks, -- Marco
