diff options
author | Jakub Kuderski <kubakuderski@gmail.com> | 2017-05-05 20:35:30 +0000 |
---|---|---|
committer | Jakub Kuderski <kubakuderski@gmail.com> | 2017-05-05 20:35:30 +0000 |
commit | f8d872c66e72277a4cb86b492c713b1f5771d2ce (patch) | |
tree | a9803a2f9dbcff9df0d31dc3d48eec5d9c8d7be8 /libcxx/src/experimental/filesystem/operations.cpp | |
parent | 807790a09b95595378f67229a41a300617285cb5 (diff) | |
download | bcm5719-llvm-f8d872c66e72277a4cb86b492c713b1f5771d2ce.tar.gz bcm5719-llvm-f8d872c66e72277a4cb86b492c713b1f5771d2ce.zip |
[clang-tidy] Fix PR32896: detect initializer lists in modernize-use-empalce
Summary:
This patch fixes [[ https://bugs.llvm.org/show_bug.cgi?id=32896 | PR32896 ]].
The problem was that modernize-use-emplace incorrectly removed changed push_back into emplace_back, removing explicit constructor call with initializer list parameter, resulting in compiler error after applying fixits.
modernize-use-emplace used to check if matched constructor had InitListExpr, but didn't check against CXXStdInitializerListExpr.
Eg.
```
std::vector<std::vector<int>> v;
v.push_back(std::vector<int>({1})); // --> v.emplace_back({1});
```
Reviewers: Prazek, alexfh, aaron.ballman
Reviewed By: Prazek, alexfh, aaron.ballman
Subscribers: xazax.hun, cfe-commits
Tags: #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D32767
llvm-svn: 302281
Diffstat (limited to 'libcxx/src/experimental/filesystem/operations.cpp')
0 files changed, 0 insertions, 0 deletions