[PATCH] D30547: [clang-tidy] Forwarding reference overload in constructors

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 3 17:12:37 PDT 2017


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





More information about the cfe-commits mailing list