msg: Add optimizing move

Variables declared in the function should be returned without using
`std::move`, but parameters to the function should be returned *with*
std::move. (If one wants them to move.)

(I know it's complicated. Sorry.)

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>

diff --git a/src/msg/async/frames_v2.h b/src/msg/async/frames_v2.h
index 9975b74aa9fcac1dd739224fd3326f19fd4b5aa2..3c92d8cf193ceadba55961e0daa22bd533b946b7 100644 (file)
--- a/src/msg/async/frames_v2.h
+++ b/src/msg/async/frames_v2.h
@@ -161,7 +161,7 @@ static ceph::bufferlist segment_onwire_bufferlist(ceph::bufferlist&& bl)
   if (padding_size) {
     bl.append_zero(padding_size);
   }
-  return bl;
+  return std::move(bl);
 }
 
 template <class T, uint16_t... SegmentAlignmentVs>