[clang-tools-extra] dcba401 - Fix 45129: Incorrect generated configuration modernize-make-shared.IncludeStyle

Nathan James via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 5 16:07:21 PST 2020


Author: Nathan James
Date: 2020-03-06T00:07:08Z
New Revision: dcba401a39d7f3c915fdea4a220380e07fe965a3

URL: https://github.com/llvm/llvm-project/commit/dcba401a39d7f3c915fdea4a220380e07fe965a3
DIFF: https://github.com/llvm/llvm-project/commit/dcba401a39d7f3c915fdea4a220380e07fe965a3.diff

LOG: Fix 45129: Incorrect generated configuration modernize-make-shared.IncludeStyle

Added: 
    

Modified: 
    clang-tools-extra/clang-tidy/modernize/MakeSmartPtrCheck.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clang-tidy/modernize/MakeSmartPtrCheck.cpp b/clang-tools-extra/clang-tidy/modernize/MakeSmartPtrCheck.cpp
index a41a5a2ca25d..b23ef6dd6b7d 100644
--- a/clang-tools-extra/clang-tidy/modernize/MakeSmartPtrCheck.cpp
+++ b/clang-tools-extra/clang-tidy/modernize/MakeSmartPtrCheck.cpp
@@ -54,7 +54,7 @@ MakeSmartPtrCheck::MakeSmartPtrCheck(StringRef Name,
       IgnoreMacros(Options.getLocalOrGlobal("IgnoreMacros", true)) {}
 
 void MakeSmartPtrCheck::storeOptions(ClangTidyOptions::OptionMap &Opts) {
-  Options.store(Opts, "IncludeStyle", IncludeStyle);
+  Options.store(Opts, "IncludeStyle", utils::IncludeSorter::toString(IncludeStyle));
   Options.store(Opts, "MakeSmartPtrFunctionHeader", MakeSmartPtrFunctionHeader);
   Options.store(Opts, "MakeSmartPtrFunction", MakeSmartPtrFunctionName);
   Options.store(Opts, "IgnoreMacros", IgnoreMacros);


        


More information about the cfe-commits mailing list