[clang-tools-extra] [clang-tidy][NFC] use local config in test cases (PR #120004)

Carlos Galvez via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 15 11:16:40 PST 2024


================
@@ -26,7 +26,7 @@ class InconsistentDeclarationParameterNameCheck : public ClangTidyCheck {
                                             ClangTidyContext *Context)
       : ClangTidyCheck(Name, Context),
         IgnoreMacros(Options.getLocalOrGlobal("IgnoreMacros", true)),
-        Strict(Options.getLocalOrGlobal("Strict", false)) {}
+        Strict(Options.get("Strict", false)) {}
----------------
carlosgalvezp wrote:

We can look into it in a separate patch. Renaming is just as problematic as removing it: we break it for users who use `Strict`, so we once again need a grace period where we support both options and then switch to the new one. Personally I don't think it's worth the trouble so I'd just leave it like it is.

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


More information about the cfe-commits mailing list