Quuxplusone added a comment.

I still think this entire patch is misguided; there's no reason to make the 
note for `const std::string s; std::move(s)` any longer than the note for `int 
i; std::move(i)` or `volatile std::string v; std::move(v)`. People should not 
be using moved-from objects, period; and those who want to use moved-from 
objects, should not enable this clang-tidy check.

However, I have no further comments //besides// philosophical opposition to the 
whole idea.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74692/new/

https://reviews.llvm.org/D74692



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

Reply via email to