default move constructors

Author: facontidavide

include/behaviortree_cpp/bt_factory.h

@@ -109,8 +109,8 @@ class Tree
   Tree(const Tree&) = delete;
   Tree& operator=(const Tree&) = delete;
 
-  Tree(Tree&& other);
-  Tree& operator=(Tree&& other);
+  Tree(Tree&& other) = default;
+  Tree& operator=(Tree&& other) = default;
 
   void initialize();
 
@@ -215,8 +215,8 @@ class BehaviorTreeFactory
   BehaviorTreeFactory(const BehaviorTreeFactory& other) = delete;
   BehaviorTreeFactory& operator=(const BehaviorTreeFactory& other) = delete;
 
-  BehaviorTreeFactory(BehaviorTreeFactory&& other) noexcept;
-  BehaviorTreeFactory& operator=(BehaviorTreeFactory&& other) noexcept;
+  BehaviorTreeFactory(BehaviorTreeFactory&& other) noexcept = default;
+  BehaviorTreeFactory& operator=(BehaviorTreeFactory&& other) noexcept = default;
 
   /// Remove a registered ID.
   bool unregisterBuilder(const std::string& ID);

src/bt_factory.cpp

@@ -107,17 +107,6 @@ BehaviorTreeFactory::BehaviorTreeFactory() : _p(new PImpl)
   _p->scripting_enums = std::make_shared<std::unordered_map<std::string, int>>();
 }
 
-BehaviorTreeFactory::BehaviorTreeFactory(BehaviorTreeFactory&& other) noexcept
-{
-  this->_p = std::move(other._p);
-}
-
-BehaviorTreeFactory& BehaviorTreeFactory::operator=(BehaviorTreeFactory&& other) noexcept
-{
-  this->_p = std::move(other._p);
-  return *this;
-}
-
 BehaviorTreeFactory::~BehaviorTreeFactory()
 {}
 
@@ -531,22 +520,9 @@ BehaviorTreeFactory::substitutionRules() const
   return _p->substitution_rules;
 }
 
-Tree& Tree::operator=(Tree&& other)
-{
-  subtrees = std::move(other.subtrees);
-  manifests = std::move(other.manifests);
-  wake_up_ = other.wake_up_;
-  return *this;
-}
-
 Tree::Tree()
 {}
 
-Tree::Tree(Tree&& other)
-{
-  (*this) = std::move(other);
-}
-
 void Tree::initialize()
 {
   wake_up_ = std::make_shared<WakeUpSignal>();