[clang-tools-extra] [clang-tidy] use upper case letters for bool conversion suffix (PR #102831)
Julian Schmidt via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 14 03:49:18 PDT 2024
================
@@ -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
More information about the cfe-commits
mailing list