[clang-tools-extra] [clang-tidy] support string::contains (PR #110351)
Tommy Chen via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 4 21:32:25 PDT 2024
================
@@ -453,3 +490,15 @@ void testOperandPermutations(std::map<int, int>& Map) {
// CHECK-MESSAGES: :[[@LINE-1]]:{{[0-9]+}}: warning: use 'contains' to check for membership [readability-container-contains]
// CHECK-FIXES: if (!Map.contains(0)) {};
}
+
+void testStringNops(std::string Str, std::string SubStr) {
+ if (Str.find("test") == std::string::npos) {};
+ // CHECK-MESSAGES: :[[@LINE-1]]:{{[0-9]+}}: warning: use 'contains' to check for membership [readability-container-contains]
+ // CHECK-FIXES: if (!Str.contains("test")) {};
+
+ if (Str.find('c') != std::string::npos) {};
+ // CHECK-MESSAGES: :[[@LINE-1]]:{{[0-9]+}}: warning: use 'contains' to check for membership [readability-container-contains]
+ // CHECK-FIXES: if (Str.contains('c')) {};
+
+ if (Str.find(SubStr) != std::string::npos) {};
+}
----------------
dl8sd11 wrote:
Done.
https://github.com/llvm/llvm-project/pull/110351
More information about the cfe-commits
mailing list