summaryrefslogtreecommitdiffstats
path: root/libcxx/src/experimental/filesystem/operations.cpp
diff options
context:
space:
mode:
authorJakub Kuderski <kubakuderski@gmail.com>2017-05-05 20:35:30 +0000
committerJakub Kuderski <kubakuderski@gmail.com>2017-05-05 20:35:30 +0000
commitf8d872c66e72277a4cb86b492c713b1f5771d2ce (patch)
treea9803a2f9dbcff9df0d31dc3d48eec5d9c8d7be8 /libcxx/src/experimental/filesystem/operations.cpp
parent807790a09b95595378f67229a41a300617285cb5 (diff)
downloadbcm5719-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
OpenPOWER on IntegriCloud