You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by mp...@apache.org on 2017/12/05 19:54:12 UTC

[4/4] mesos git commit: Made `Event` move-only in libprocess.

Made `Event` move-only in libprocess.

Review: https://reviews.apache.org/r/64347/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/6839897c
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/6839897c
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/6839897c

Branch: refs/heads/master
Commit: 6839897c5464fce6b8cbd253d959a7e2efd72987
Parents: c9e6a03
Author: Dmitry Zhuk <dz...@twopensource.com>
Authored: Tue Dec 5 11:21:27 2017 -0800
Committer: Michael Park <mp...@apache.org>
Committed: Tue Dec 5 11:21:27 2017 -0800

----------------------------------------------------------------------
 3rdparty/libprocess/include/process/event.hpp | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/6839897c/3rdparty/libprocess/include/process/event.hpp
----------------------------------------------------------------------
diff --git a/3rdparty/libprocess/include/process/event.hpp b/3rdparty/libprocess/include/process/event.hpp
index 76bcdb8..2e93428 100644
--- a/3rdparty/libprocess/include/process/event.hpp
+++ b/3rdparty/libprocess/include/process/event.hpp
@@ -121,8 +121,7 @@ struct MessageEvent : Event
     : message{std::move(name), from, to, std::move(data)} {}
 
   MessageEvent(MessageEvent&& that) = default;
-  // TODO(dzhuk) Make it movable only, when move is properly supported in defer.
-  MessageEvent(const MessageEvent& that) = default;
+  MessageEvent(const MessageEvent& that) = delete;
   MessageEvent& operator=(MessageEvent&&) = default;
   MessageEvent& operator=(const MessageEvent&) = delete;
 
@@ -218,8 +217,7 @@ struct ExitedEvent : Event
     : pid(_pid) {}
 
   ExitedEvent(ExitedEvent&&) = default;
-  // TODO(dzhuk) Make it movable only, when move is properly supported in defer.
-  ExitedEvent(const ExitedEvent&) = default;
+  ExitedEvent(const ExitedEvent&) = delete;
   ExitedEvent& operator=(ExitedEvent&&) = default;
   ExitedEvent& operator=(const ExitedEvent&) = delete;