Add missing virtual dtors

Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
ros2 · Jun 1, 2020 · a5c8c42 · a5c8c42
1 parent 2857753
commit a5c8c42
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 0 deletions.
diff --git a/rclcpp/include/rclcpp/rate.hpp b/rclcpp/include/rclcpp/rate.hpp
@@ -31,6 +31,7 @@ class RateBase
 public:
   RCLCPP_SMART_PTR_DEFINITIONS_NOT_COPYABLE(RateBase)
 
+  virtual ~RateBase() {}
   virtual bool sleep() = 0;
   virtual bool is_steady() const = 0;
   virtual void reset() = 0;

diff --git a/rclcpp/include/rclcpp/timer.hpp b/rclcpp/include/rclcpp/timer.hpp
@@ -62,6 +62,7 @@ class TimerBase
 
   /// TimerBase destructor
   RCLCPP_PUBLIC
+  virtual
   ~TimerBase();
 
   /// Cancel the timer.

diff --git a/rclcpp_lifecycle/include/rclcpp_lifecycle/lifecycle_publisher.hpp b/rclcpp_lifecycle/include/rclcpp_lifecycle/lifecycle_publisher.hpp
@@ -36,6 +36,7 @@ namespace rclcpp_lifecycle
 class LifecyclePublisherInterface
 {
 public:
+  virtual ~LifecyclePublisherInterface() {};
   virtual void on_activate() = 0;
   virtual void on_deactivate() = 0;
   virtual bool is_activated() = 0;

diff --git a/rclcpp_lifecycle/include/rclcpp_lifecycle/node_interfaces/lifecycle_node_interface.hpp b/rclcpp_lifecycle/include/rclcpp_lifecycle/node_interfaces/lifecycle_node_interface.hpp
@@ -100,6 +100,10 @@ class LifecycleNodeInterface
   RCLCPP_LIFECYCLE_PUBLIC
   virtual CallbackReturn
   on_error(const State & previous_state);
+
+  RCLCPP_LIFECYCLE_PUBLIC
+  virtual
+  ~LifecycleNodeInterface() {}
 };
 
 }  // namespace node_interfaces