From b2549dd04078a3f31d43215b8a858d05a2de9565 Mon Sep 17 00:00:00 2001 From: Ashton Larkin <42042756+adlarkin@users.noreply.github.com> Date: Tue, 12 Apr 2022 16:01:52 -0600 Subject: [PATCH] Remove unused View::Clone method (#1440) Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com> --- include/ignition/gazebo/detail/BaseView.hh | 2 -- include/ignition/gazebo/detail/View.hh | 9 --------- 2 files changed, 11 deletions(-) diff --git a/include/ignition/gazebo/detail/BaseView.hh b/include/ignition/gazebo/detail/BaseView.hh index 7f08090b80..b426f01bb6 100644 --- a/include/ignition/gazebo/detail/BaseView.hh +++ b/include/ignition/gazebo/detail/BaseView.hh @@ -188,8 +188,6 @@ class IGNITION_GAZEBO_VISIBLE BaseView /// \sa ToAddEntities public: void ClearToAddEntities(); - public: virtual std::unique_ptr 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 diff --git a/include/ignition/gazebo/detail/View.hh b/include/ignition/gazebo/detail/View.hh index 23eccf7303..3972b3d079 100644 --- a/include/ignition/gazebo/detail/View.hh +++ b/include/ignition/gazebo/detail/View.hh @@ -106,8 +106,6 @@ class View : public BaseView /// \brief Documentation inherited public: void Reset() override; - public: std::unique_ptr 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 @@ -332,13 +330,6 @@ void View::Reset() this->invalidConstData.clear(); this->missingCompTracker.clear(); } - -////////////////////////////////////////////////// -template -std::unique_ptr View::Clone() const -{ - return std::make_unique>(*this); -} } // namespace detail } // namespace IGNITION_GAZEBO_VERSION_NAMESPACE } // namespace gazebo