================
@@ -80,16 +80,19 @@ 
UseIntegerSignComparisonCheck::UseIntegerSignComparisonCheck(
     : ClangTidyCheck(Name, Context),
       IncludeInserter(Options.getLocalOrGlobal("IncludeStyle",
                                                utils::IncludeSorter::IS_LLVM),
-                      areDiagsSelfContained()) {}
+                      areDiagsSelfContained()),
+      ConsideringIntSize(Options.get("ConsideringIntegerSize", true)) {}
----------------
PiotrZSL wrote:

use more straightforward option name, like "CheckIntegerSize" or something 
similar.
I just dont like how this "Considering" sounds in context of an option, even 
"Consider" would sound better.

https://github.com/llvm/llvm-project/pull/121506
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to