[PATCH] D18365: [clang-tidy] Fix broken test with redundant string init (msvc).
Etienne Bergeron via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 22 11:21:52 PDT 2016
etienneb created this revision.
etienneb added reviewers: rnk, alexfh.
etienneb added a subscriber: cfe-commits.
There is a silly bug that got introduced after fixing incorrect paths with this patch:
http://reviews.llvm.org/D18293
The tests was present twice in the file.
http://reviews.llvm.org/D18365
Files:
test/clang-tidy/readability-redundant-string-init-msvc.cpp
Index: test/clang-tidy/readability-redundant-string-init-msvc.cpp
===================================================================
--- test/clang-tidy/readability-redundant-string-init-msvc.cpp
+++ test/clang-tidy/readability-redundant-string-init-msvc.cpp
@@ -59,64 +59,3 @@
std::wstring y(LR"(y)");
std::wstring z;
}
-// RUN: %check_clang_tidy %s readability-redundant-string-init %t
-
-namespace std {
-template <typename T>
-class allocator {};
-template <typename T>
-class char_traits {};
-template <typename C, typename T = std::char_traits<C>, typename A = std::allocator<C>>
-struct basic_string {
- basic_string();
- basic_string(const basic_string&);
- // MSVC headers define two constructors instead of using optional arguments.
- basic_string(const C *);
- basic_string(const C *, const A &);
- ~basic_string();
-};
-typedef basic_string<char> string;
-typedef basic_string<wchar_t> wstring;
-}
-
-void f() {
- std::string a = "";
- // CHECK-MESSAGES: [[@LINE-1]]:15: warning: redundant string initialization [readability-redundant-string-init]
- // CHECK-FIXES: std::string a;
- std::string b("");
- // CHECK-MESSAGES: [[@LINE-1]]:15: warning: redundant string initialization
- // CHECK-FIXES: std::string b;
- std::string c = R"()";
- // CHECK-MESSAGES: [[@LINE-1]]:15: warning: redundant string initialization
- // CHECK-FIXES: std::string c;
- std::string d(R"()");
- // CHECK-MESSAGES: [[@LINE-1]]:15: warning: redundant string initialization
- // CHECK-FIXES: std::string d;
-
- std::string u = "u";
- std::string w("w");
- std::string x = R"(x)";
- std::string y(R"(y)");
- std::string z;
-}
-
-void g() {
- std::wstring a = L"";
- // CHECK-MESSAGES: [[@LINE-1]]:16: warning: redundant string initialization
- // CHECK-FIXES: std::wstring a;
- std::wstring b(L"");
- // CHECK-MESSAGES: [[@LINE-1]]:16: warning: redundant string initialization
- // CHECK-FIXES: std::wstring b;
- std::wstring c = LR"()";
- // CHECK-MESSAGES: [[@LINE-1]]:16: warning: redundant string initialization
- // CHECK-FIXES: std::wstring c;
- std::wstring d(LR"()");
- // CHECK-MESSAGES: [[@LINE-1]]:16: warning: redundant string initialization
- // CHECK-FIXES: std::wstring d;
-
- std::wstring u = L"u";
- std::wstring w(L"w");
- std::wstring x = LR"(x)";
- std::wstring y(LR"(y)");
- std::wstring z;
-}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18365.51310.patch
Type: text/x-patch
Size: 2351 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160322/cdea70f2/attachment-0001.bin>
More information about the cfe-commits
mailing list