[mesos] 04/04: Enabled more constructors for master `RegistryOperation`.

[mz...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mzhu pushed a commit to branch 1.7.x
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 22d07ce094a25c5cf3488f74cc4a70cc8d855f18
Author: Benjamin Bannier <be...@mesosphere.io>
AuthorDate: Thu Oct 18 09:33:39 2018 +0200

    Enabled more constructors for master `RegistryOperation`.
    
    Since this class was explicitly declaring a constructor, no other
    constructors such as copy or move constructors were generated.
    
    This patch removes the custom constructor so all default constructors
    are generated automatically. This e.g., enables move construction of
    `RegistryOperation` values.
    
    Review: https://reviews.apache.org/r/69042/
---
 src/master/registrar.hpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/master/registrar.hpp b/src/master/registrar.hpp
index 4f7c7ce..b5aa47f 100644
--- a/src/master/registrar.hpp
+++ b/src/master/registrar.hpp
@@ -45,9 +45,6 @@ class RegistrarProcess;
 class RegistryOperation : public process::Promise<bool>
 {
 public:
-  RegistryOperation() : success(false) {}
-  ~RegistryOperation() override {}
-
   // Attempts to invoke the operation on the registry object.
   // Aided by accumulator(s):
   //   slaveIDs - is the set of registered slaves.
@@ -70,7 +67,7 @@ protected:
   virtual Try<bool> perform(Registry* registry, hashset<SlaveID>* slaveIDs) = 0;
 
 private:
-  bool success;
+  bool success = false;
 };