alexfh added inline comments.

================
Comment at: clang-tidy/modernize/MakeSmartPtrCheck.cpp:32-33
@@ -31,1 +31,4 @@
 
+  // Calling make_smart_ptr from within a member function of a type with a
+  // private or protected constructor would be ill-formed.
+  auto CanCallCtor = unless(has(ignoringImpCasts(cxxConstructExpr(
----------------
malcolm.parsons wrote:
> The private constructor could also be called from a friend class.
It might only be relevant, if `std::make_(shared|unique)` is declared as a 
friend of the class in question, which is unlikely to be a common case and I 
don't think we need to focus on this now.


https://reviews.llvm.org/D23343



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to