Looks good. ================ Comment at: clang-tidy/misc/BoolPointerImplicitConversion.h:22 @@ +21,3 @@ +/// if (p) { +/// // Never dereferenced. +/// } ---------------- Maybe "// Never used in a pointer-specific way."? http://reviews.llvm.org/D4458