Skip to content

Commit

Permalink
remove unused View::Clone method
Browse files Browse the repository at this point in the history
Signed-off-by: Ashton Larkin <[email protected]>
  • Loading branch information
adlarkin committed Apr 12, 2022
1 parent f9f2bbd commit 5d2dae3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
2 changes: 0 additions & 2 deletions include/ignition/gazebo/detail/BaseView.hh
Original file line number Diff line number Diff line change
Expand Up @@ -188,8 +188,6 @@ class IGNITION_GAZEBO_VISIBLE BaseView
/// \sa ToAddEntities
public: void ClearToAddEntities();

public: virtual std::unique_ptr<BaseView> Clone() const = 0;

// TODO(adlarkin) make this a std::unordered_set for better performance.
// We need to make sure nothing else depends on the ordered preserved by
// std::set first
Expand Down
9 changes: 0 additions & 9 deletions include/ignition/gazebo/detail/View.hh
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,6 @@ class View : public BaseView
/// \brief Documentation inherited
public: void Reset() override;

public: std::unique_ptr<BaseView> Clone() const override;

/// \brief A map of entities to their component data. Since tuples are defined
/// at compile time, we need separate containers that have tuples for both
/// non-const and const component pointers (calls to ECM::Each can have a
Expand Down Expand Up @@ -332,13 +330,6 @@ void View<ComponentTypeTs...>::Reset()
this->invalidConstData.clear();
this->missingCompTracker.clear();
}

//////////////////////////////////////////////////
template<typename ...ComponentTypeTs>
std::unique_ptr<BaseView> View<ComponentTypeTs...>::Clone() const
{
return std::make_unique<View<ComponentTypeTs...>>(*this);
}
} // namespace detail
} // namespace IGNITION_GAZEBO_VERSION_NAMESPACE
} // namespace gazebo
Expand Down

0 comments on commit 5d2dae3

Please sign in to comment.