http_pipeline does not have correct semantics under move.

jasonsandlin · Apr 14, 2016 · ba2adb3 · ba2adb3
1 parent baa8c9b
commit ba2adb3
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/Release/src/http/client/http_client.cpp b/Release/src/http/client/http_client.cpp
@@ -280,9 +280,11 @@ class http_pipeline
     http_pipeline(std::shared_ptr<details::_http_client_communicator> last) : m_last_stage(std::move(last))
     {}
 
-    // No copy or assignment.
-    http_pipeline & operator=(const http_pipeline &) = delete;
+    // pplx::extensibility::recursive_lock_t does not support move/copy, but does not delete the functions either.
     http_pipeline(const http_pipeline &) = delete;
+    http_pipeline(http_pipeline &&) = delete;
+    http_pipeline & operator=(const http_pipeline &) = delete;
+    http_pipeline & operator=(http_pipeline &&) = delete;
 
     /// <summary>
     /// Initiate an http request into the pipeline