[clang-tools-extra] 8d50a84 - [clang-tidy] Hotfix default parameter value in 'bugprone-easily-swappable-parameters'

via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 24 07:11:54 PDT 2021


Author: Whisperity
Date: 2021-08-24T16:10:19+02:00
New Revision: 8d50a847d410295a9fca466d38a261cca4c2302b

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

LOG: [clang-tidy] Hotfix default parameter value in 'bugprone-easily-swappable-parameters'

As identified by @RKSimon, there was a missing comma in the default
value for the "ignored parameter type suffixes" array, resulting in
bogus concatenation of two elements.

Added: 
    

Modified: 
    clang-tools-extra/clang-tidy/bugprone/EasilySwappableParametersCheck.cpp

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clang-tidy/bugprone/EasilySwappableParametersCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/EasilySwappableParametersCheck.cpp
index e4500706b4b2..16ede76af6ab 100644
--- a/clang-tools-extra/clang-tidy/bugprone/EasilySwappableParametersCheck.cpp
+++ b/clang-tools-extra/clang-tidy/bugprone/EasilySwappableParametersCheck.cpp
@@ -56,7 +56,7 @@ static const std::string DefaultIgnoredParameterTypeSuffixes =
                                    "reverse_const_iterator",
                                    "ConstReverseIterator",
                                    "Const_Reverse_Iterator",
-                                   "const_reverse_iterator"
+                                   "const_reverse_iterator",
                                    "Constreverseiterator",
                                    "constreverseiterator"});
 


        


More information about the cfe-commits mailing list