================ @@ -36,6 +36,7 @@ class ImplicitBoolConversionCheck : public ClangTidyCheck { const bool AllowIntegerConditions; const bool AllowPointerConditions; + const bool UseUpperCaseSuffix; ---------------- 5chmidti wrote:
Yes, the comment is in regard to the option name. My rationale is, that the current option name is about an upper-case suffix, but suffix of what? Adding the literal will make things clearer. https://github.com/llvm/llvm-project/pull/102831 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits