Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | [clang-tidy] Fix broken test with redundant string init (msvc). | Etienne Bergeron | 2016-03-22 | 1 | -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 Bergeron | 2016-03-22 | 1 | -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 |