Set size limit in constructor

Author: Mauro Passerino

rclcpp/include/rclcpp/experimental/buffers/bounded_events_queue.hpp

@@ -36,6 +36,12 @@ namespace buffers
 class BoundedEventsQueue : public EventsQueue
 {
 public:
+  RCLCPP_PUBLIC
+  explicit BoundedEventsQueue(size_t queue_size_limit)
+  {
+    queue_size_limit_ = queue_size_limit;
+  }
+
   RCLCPP_PUBLIC
   ~BoundedEventsQueue() = default;
 
@@ -121,21 +127,9 @@ class BoundedEventsQueue : public EventsQueue
     return local_queue;
   }
 
-  /**
-   * @brief sets the queue size limit
-   * @param limit The queue size limit
-   */
-  RCLCPP_PUBLIC
-  void
-  set_queue_size_limit(size_t queue_size_limit)
-  {
-    queue_size_limit_ = queue_size_limit;
-  }
-
 private:
   std::queue<rmw_listener_event_t> event_queue_;
-
-  size_t queue_size_limit_ = 1000;
+  size_t queue_size_limit_;
 };
 
 }  // namespace buffers

rclcpp/test/rclcpp/executors/test_events_queue.cpp

@@ -41,8 +41,7 @@ class TestEventsQueue : public ::testing::Test
 TEST_F(TestEventsQueue, BoundedQueue)
 {
   // Create BoundedEventsQueue and set limit to 10 events.
-  auto bounded_queue = std::make_unique<rclcpp::experimental::buffers::BoundedEventsQueue>();
-  bounded_queue->set_queue_size_limit(10);
+  auto bounded_queue = std::make_unique<rclcpp::experimental::buffers::BoundedEventsQueue>(10);
 
   // Create an events executor using the bounded queue
   EventsExecutor executor_sub(std::move(bounded_queue));