alexfh added inline comments. ================ Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:115 @@ -95,1 +114,3 @@ + auto CtorCallSourceRange = CharSourceRange::getTokenRange( + InnerCtorCall->getExprLoc(), CallParensRange.getBegin()); ---------------- Prazek wrote: > 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? Sounds almost like a recipe for a test case ;) Have you tried to construct it this way?
https://reviews.llvm.org/D22208 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits