aaron.ballman added inline comments. ================ Comment at: test/clang-tidy/misc-move-const-arg.cpp:75-76 @@ +74,4 @@ + +struct NonMoveable { + public: + NonMoveable(); ---------------- This type isn't non-moveable. For that, you need to explicitly delete the move constructor. Perhaps a better name is `NonMoveConstructible`?
Also, no need for the `public` access specifier. ================ Comment at: test/clang-tidy/misc-move-const-arg.cpp:116 @@ +115,3 @@ +struct Moveable { + public: + Moveable(); ---------------- No need for the `public` access specifier. http://reviews.llvm.org/D21223 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits