================ @@ -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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits