Implement user assign operator to avoid implicit

Addresses #332 Signed-off-by: Michael Carroll <michael@openrobotics.org>
gazebosim · Mar 15, 2022 · ddf028b · ddf028b
1 parent ba10632
commit ddf028b
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/include/ignition/physics/SpecifyData.hh b/include/ignition/physics/SpecifyData.hh
@@ -57,6 +57,9 @@ namespace ignition
  /// the newly copied MapData.
  public: ExpectData(const ExpectData &_other);
 
+ /// \brief Assignment operator.
+ public: ExpectData& operator=(const ExpectData &_other);
+
  /// TODO(anyone) Implement move constructor and assignment operator. Due
  /// to the multiple inheritence used to implement SpcifyData, care must be
  /// taken when implementing move constructs to avoid calling a move

diff --git a/include/ignition/physics/detail/SpecifyData.hh b/include/ignition/physics/detail/SpecifyData.hh
@@ -47,6 +47,16 @@ namespace ignition
  // Do nothing
  }
 
+ /////////////////////////////////////////////////
+ template <typename Expected>
+ ExpectData<Expected>& ExpectData<Expected>::operator=(
+ const ExpectData<Expected> &_other)
+ {
+ // Call base class assignment operator
+ this->CompositeData::operator=(_other);
+ return *this;
+ }
+
  /////////////////////////////////////////////////
  template <typename Expected>
  template <typename Data>