carlosgalvezp added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/index.rst:347 + // No warnings are suppressed, due to double negation + Foo(bool param); // NOLINT(-google*) }; ---------------- salman-javed-nz wrote: > Would anyone do this on purpose, or is this a user error? I don't see any use case here, no. I just thought to document it to clarify the double negation that's happening, in case a user gets confused and doesn't see the warning being suppressed. Do you think it brings value or does more harm than good? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111208/new/ https://reviews.llvm.org/D111208 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
