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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits