summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp
diff options
context:
space:
mode:
authorBenjamin Kramer <benny.kra@googlemail.com>2016-04-07 14:55:25 +0000
committerBenjamin Kramer <benny.kra@googlemail.com>2016-04-07 14:55:25 +0000
commita62e2232815e537952bd6df271763521a0472498 (patch)
treeea5a784158442dae656b208e3d436cbc624fa55b /clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp
parent53f7c0ea8e02e2a49f13e713ec6bdbfe067e8383 (diff)
downloadbcm5719-llvm-a62e2232815e537952bd6df271763521a0472498.tar.gz
bcm5719-llvm-a62e2232815e537952bd6df271763521a0472498.zip
[clang-tidy] Remove unnecessary getName() on Decls and Types feeding into a DiagnosticBuilder
Going through a string removes some of the smarts of the diagnosic printer and makes the code more complicated. This change has some cosmetic impact on the output but that's mostly minor. llvm-svn: 265680
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp')
-rw-r--r--clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp26
1 files changed, 13 insertions, 13 deletions
diff --git a/clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp b/clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp
index 9510616128c..b36e32301f5 100644
--- a/clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp
+++ b/clang-tools-extra/test/clang-tidy/performance-faster-string-find.cpp
@@ -33,12 +33,12 @@ void StringFind() {
std::string Str;
Str.find("a");
- // CHECK-MESSAGES: [[@LINE-1]]:12: warning: find called with a string literal consisting of a single character; consider using the more effective overload accepting a character [performance-faster-string-find]
+ // CHECK-MESSAGES: [[@LINE-1]]:12: warning: 'find' called with a string literal consisting of a single character; consider using the more effective overload accepting a character [performance-faster-string-find]
// CHECK-FIXES: Str.find('a');
// Works with the pos argument.
Str.find("a", 1);
- // CHECK-MESSAGES: [[@LINE-1]]:12: warning: find called with a string literal
+ // CHECK-MESSAGES: [[@LINE-1]]:12: warning: 'find' called with a string literal
// CHECK-FIXES: Str.find('a', 1);
// Doens't work with strings smaller or larger than 1 char.
@@ -50,35 +50,35 @@ void StringFind() {
// Other methods that can also be replaced
Str.rfind("a");
- // CHECK-MESSAGES: [[@LINE-1]]:13: warning: rfind called with a string literal
+ // CHECK-MESSAGES: [[@LINE-1]]:13: warning: 'rfind' called with a string literal
// CHECK-FIXES: Str.rfind('a');
Str.find_first_of("a");
- // CHECK-MESSAGES: [[@LINE-1]]:21: warning: find_first_of called with a string
+ // CHECK-MESSAGES: [[@LINE-1]]:21: warning: 'find_first_of' called with a string
// CHECK-FIXES: Str.find_first_of('a');
Str.find_first_not_of("a");
- // CHECK-MESSAGES: [[@LINE-1]]:25: warning: find_first_not_of called with a
+ // CHECK-MESSAGES: [[@LINE-1]]:25: warning: 'find_first_not_of' called with a
// CHECK-FIXES: Str.find_first_not_of('a');
Str.find_last_of("a");
- // CHECK-MESSAGES: [[@LINE-1]]:20: warning: find_last_of called with a string
+ // CHECK-MESSAGES: [[@LINE-1]]:20: warning: 'find_last_of' called with a string
// CHECK-FIXES: Str.find_last_of('a');
Str.find_last_not_of("a");
- // CHECK-MESSAGES: [[@LINE-1]]:24: warning: find_last_not_of called with a
+ // CHECK-MESSAGES: [[@LINE-1]]:24: warning: 'find_last_not_of' called with a
// CHECK-FIXES: Str.find_last_not_of('a');
// std::wstring should work.
std::wstring WStr;
WStr.find(L"n");
- // CHECK-MESSAGES: [[@LINE-1]]:13: warning: find called with a string literal
+ // CHECK-MESSAGES: [[@LINE-1]]:13: warning: 'find' called with a string literal
// CHECK-FIXES: Str.find(L'n');
// Even with unicode that fits in one wide char.
WStr.find(L"\x3A9");
- // CHECK-MESSAGES: [[@LINE-1]]:13: warning: find called with a string literal
+ // CHECK-MESSAGES: [[@LINE-1]]:13: warning: 'find' called with a string literal
// CHECK-FIXES: Str.find(L'\x3A9');
// Also with other types, but only if it was specified in the options.
llvm::StringRef sr;
sr.find("x");
- // CHECK-MESSAGES: [[@LINE-1]]:11: warning: find called with a string literal
+ // CHECK-MESSAGES: [[@LINE-1]]:11: warning: 'find' called with a string literal
// CHECK-FIXES: sr.find('x');
NotStringRef nsr;
nsr.find("x");
@@ -92,7 +92,7 @@ int FindTemplateDependant(T value) {
template <typename T>
int FindTemplateNotDependant(T pos) {
return std::string().find("A", pos);
- // CHECK-MESSAGES: [[@LINE-1]]:29: warning: find called with a string literal
+ // CHECK-MESSAGES: [[@LINE-1]]:29: warning: 'find' called with a string literal
// CHECK-FIXES: return std::string().find('A', pos);
}
@@ -105,6 +105,6 @@ int FindStr() {
int Macros() {
return STR_MACRO(std::string()) + POS_MACRO(1);
- // CHECK-MESSAGES: [[@LINE-1]]:10: warning: find called with a string literal
- // CHECK-MESSAGES: [[@LINE-2]]:37: warning: find called with a string literal
+ // CHECK-MESSAGES: [[@LINE-1]]:10: warning: 'find' called with a string literal
+ // CHECK-MESSAGES: [[@LINE-2]]:37: warning: 'find' called with a string literal
}
OpenPOWER on IntegriCloud