[clang-tools-extra] [clang-tidy] Fix modernize-use-integer-sign-comparison comparison (PR #121506)

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 11 01:09:25 PST 2025


================
@@ -126,6 +126,14 @@ class ModernizeModule : public ClangTidyModule {
         "modernize-use-uncaught-exceptions");
     CheckFactories.registerCheck<UseUsingCheck>("modernize-use-using");
   }
+
+  ClangTidyOptions getModuleOptions() override {
+    ClangTidyOptions Options;
+    Options.CheckOptions
+        ["modernize-use-integer-sign-comparison.ConsideringIntegerSize"] =
+        "true";
+    return Options;
+  }
----------------
PiotrZSL wrote:

remove, this is not need as this option is already set in constructor of a check
```suggestion
```

https://github.com/llvm/llvm-project/pull/121506


More information about the cfe-commits mailing list