Remove dummy sentinel

Signed-off-by: Michael Carroll <[email protected]>

Author: mjcarroll

rclcpp/include/rclcpp/event_handler.hpp

@@ -106,12 +106,6 @@ class EventHandlerBase : public Waitable
     Event,
   };
 
-  // TODO(wjwwood): is this ok? do events continue to stay ready until they are
-  // taken/checked?
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   RCLCPP_PUBLIC
   virtual ~EventHandlerBase();
 

rclcpp/include/rclcpp/executors/executor_notify_waitable.hpp

@@ -53,10 +53,6 @@ class ExecutorNotifyWaitable : public rclcpp::Waitable
   RCLCPP_PUBLIC
   ExecutorNotifyWaitable & operator=(ExecutorNotifyWaitable & other);
 
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   /// Add conditions to the wait set
   /**
    * \param[inout] wait_set structure that conditions will be added to

rclcpp/include/rclcpp/experimental/subscription_intra_process.hpp

@@ -101,10 +101,6 @@ class SubscriptionIntraProcess
 
   virtual ~SubscriptionIntraProcess() = default;
 
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   RCLCPP_PUBLIC
   void
   add_to_wait_set(rcl_wait_set_t * wait_set) override

rclcpp/include/rclcpp/waitable.hpp

@@ -35,11 +35,6 @@ class Waitable
   RCLCPP_PUBLIC
   virtual ~Waitable() = default;
 
-  RCLCPP_PUBLIC
-  virtual
-  void
-  dummy() = 0;
-
   /// Get the number of ready subscriptions
   /**
    * Returns a value of 0 by default.

rclcpp/test/rclcpp/executors/test_executors.cpp

@@ -336,12 +336,6 @@ class TestWaitable : public rclcpp::Waitable
 public:
   TestWaitable() = default;
 
-  // TODO(wjwwood): is this ok? do events continue to stay ready until they are
-  // taken/checked?
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   void
   add_to_wait_set(rcl_wait_set_t & wait_set) override
   {

rclcpp/test/rclcpp/executors/test_static_executor_entities_collector.cpp

@@ -230,12 +230,6 @@ TEST_F(TestStaticExecutorEntitiesCollector, add_remove_node_out_of_scope) {
 class TestWaitable : public rclcpp::Waitable
 {
 public:
-  // TODO(wjwwood): is this ok? do events continue to stay ready until they are
-  // taken/checked?
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   void add_to_wait_set(rcl_wait_set_t &) override {}
 
   bool is_ready(const rcl_wait_set_t &) override {return true;}

rclcpp/test/rclcpp/node_interfaces/test_node_waitables.cpp

@@ -28,12 +28,6 @@
 class TestWaitable : public rclcpp::Waitable
 {
 public:
-  // TODO(wjwwood): is this ok? do events continue to stay ready until they are
-  // taken/checked?
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   void add_to_wait_set(rcl_wait_set_t &) override {}
   bool is_ready(const rcl_wait_set_t &) override {return false;}
 

rclcpp/test/rclcpp/strategies/test_allocator_memory_strategy.cpp

@@ -39,11 +39,6 @@ static bool test_waitable_result = false;
 class TestWaitable : public rclcpp::Waitable
 {
 public:
-  // TODO(wjwwood): is this ok? double check that this test doesn't need more
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   void add_to_wait_set(rcl_wait_set_t &) override
 
   {

rclcpp/test/rclcpp/test_memory_strategy.cpp

@@ -35,12 +35,6 @@ typedef std::map<rclcpp::CallbackGroup::WeakPtr,
 class TestWaitable : public rclcpp::Waitable
 {
 public:
-  // TODO(wjwwood): is this ok? do events continue to stay ready until they are
-  // taken/checked?
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   void add_to_wait_set(rcl_wait_set_t &) override {}
   bool is_ready(const rcl_wait_set_t &) override {return true;}
 

rclcpp/test/rclcpp/wait_set_policies/test_dynamic_storage.cpp

@@ -51,12 +51,6 @@ class TestWaitable : public rclcpp::Waitable
   TestWaitable()
   : is_ready_(false) {}
 
-  // TODO(wjwwood): is this ok? do events continue to stay ready until they are
-  // taken/checked?
-  RCLCPP_PUBLIC
-  void
-  dummy() override {};
-
   void add_to_wait_set(rcl_wait_set_t &) override {}
 
   bool is_ready(const rcl_wait_set_t &) override {return is_ready_;}