[llvm-branch-commits] [clang-tools-extra] 7161e4f - [clang-tidy] Hotfix default parameter value in 'bugprone-easily-swappable-parameters'

Tom Stellard via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Tue Aug 24 18:11:23 PDT 2021


Author: Whisperity
Date: 2021-08-24T18:10:58-07:00
New Revision: 7161e4f3345fda1b640a8250a4b34d23c74b0489

URL: https://github.com/llvm/llvm-project/commit/7161e4f3345fda1b640a8250a4b34d23c74b0489
DIFF: https://github.com/llvm/llvm-project/commit/7161e4f3345fda1b640a8250a4b34d23c74b0489.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.

(cherry picked from commit 8d50a847d410295a9fca466d38a261cca4c2302b)

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 e4500706b4b2e..16ede76af6ab8 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 llvm-branch-commits mailing list