[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