[future] Minor: std::move() callback in future for the convenience operator case. (pytorch#37861)

Jeremy Lilley · facebook-github-bot · commit b6d494d6dab8 · 2020-05-12T13:35:41.000-07:00
Summary: Pull Request resolved: pytorch#37861 ghstack-source-id: 103511066 Test Plan: buck test mode/dev-nosan caffe2/test/... Differential Revision: D21409650 fbshipit-source-id: 6c501963f73590512e426f6806f4530aad618b1a
diff --git a/torch/csrc/utils/future.h b/torch/csrc/utils/future.h
@@ -130,7 +130,7 @@ class TORCH_API Future final {
   }
 
   void addCallback(std::function<void(const Future<T>& future)> cb) {
-    addCallback([this, cb]() { cb(*this); });
+    addCallback([this, cb = std::move(cb)]() { cb(*this); });
   }
 
  private:

Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ class TORCH_API Future final {`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`void addCallback(std::function<void(const Future<T>& future)> cb) {`
`133`		`- addCallback([this, cb]() { cb(*this); });`
	`133`	`+ addCallback([this, cb = std::move(cb)]() { cb(*this); });`
`134`	`134`	`}`
`135`	`135`
`136`	`136`	`private:`