summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/test/clang-tidy/readability-redundant-string-init-msvc.cpp
Commit message (Collapse)AuthorAgeFilesLines
* [clang-tidy] Fix broken test with redundant string init (msvc).Etienne Bergeron2016-03-221-61/+0
| | | | | | | | | | | | | | | | Summary: 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. Reviewers: alexfh, rnk Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D18365 llvm-svn: 264080
* [clang-tidy] Fix redundant-string-init check with msvc 14 headers.Etienne Bergeron2016-03-221-0/+122
Summary: The string constructors are not defined using optional parameters and are not recognized by the redundant-string-init checker. The following patch fixes the redundant-string-init checker for the Visual Studio 14 headers file. The matcher now accept both variant (with 1 and 2 parameters). Also added new unittests. Similar issue than: [[ http://reviews.llvm.org/D18285 | review ]] In the xstring.h header, the constructors are defined this way: ``` basic_string(const _Myt& _Right) [...] basic_string(const _Myt& _Right, const _Alloc& _Al) [...] ``` Reviewers: alexfh, hokein Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D18293 llvm-svn: 264069
OpenPOWER on IntegriCloud