aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
Aside from some minor nits about newlines, LGTM! ================ Comment at: docs/clang-tidy/checks/misc-forwarding-reference-overload.rst:51-52 + + + ---------------- You can remove the empty lines at the end. ================ Comment at: test/clang-tidy/misc-forwarding-reference-overload.cpp:140 +}; \ No newline at end of file ---------------- Please add a newline at the end of the file. Repository: rL LLVM https://reviews.llvm.org/D30547 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits