[PATCH] D22208: [clang-tidy] Fixes to modernize-use-emplace
Piotr Padlewski via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 26 11:58:10 PDT 2016
Prazek added inline comments.
================
Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:115
@@ -95,1 +114,3 @@
+ auto CtorCallSourceRange = CharSourceRange::getTokenRange(
+ InnerCtorCall->getExprLoc(), CallParensRange.getBegin());
----------------
alexfh wrote:
> This doesn't seem to be an issue, since expression `v.push_back(obj.member())` won't trigger this check: it expects that the argument of the `push_back` call is a `cxxConstructExpr` or a `cxxFunctionalCastExpr`.
what about the implicit conversion? What if obj.member would return object that is different from the one that v stores, but it is convertible to it?
https://reviews.llvm.org/D22208
More information about the cfe-commits
mailing list