[PATCH] D21303: [clang-tidy] Adds performance-returning-type check.

Eugene Zelenko via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 13 14:11:01 PDT 2016


Eugene.Zelenko added a subscriber: Eugene.Zelenko.

================
Comment at: docs/ReleaseNotes.rst:238
@@ +237,3 @@
+
+  This check finds places where we are returning object of a different type than
+  the function return type. In such places, we should use std::move, otherwise
----------------
One sentence should be enough. Please also remove "This check". "We" should be replaced with "function".

================
Comment at: docs/clang-tidy/checks/performance-returning-type.rst:7
@@ +6,3 @@
+This check finds places where we are returning object of a different type than
+the function return type. In such places, we should use std::move, otherwise
+the object will not be moved automatically.
----------------
Please highlight std::move with ``. Same below


http://reviews.llvm.org/D21303





More information about the cfe-commits mailing list