chore: Minor cleanups

Signed-off-by: Janosch Machowinski <J.Machowinski@cellumation.com>

Author: Janosch Machowinski

rclcpp/include/rclcpp/executors/events_cbg_executor/events_cbg_executor.hpp

@@ -52,7 +52,6 @@ class EventsCBGExecutor : public rclcpp::Executor
    * \param options common options for all executors
    * \param number_of_threads number of threads to have in the thread pool,
    *   the default 0 will use the number of cpu cores found (minimum of 2)
-   * \param yield_before_execute if true std::this_thread::yield() is called
    * \param timeout maximum time to wait
    */
   RCLCPP_PUBLIC
@@ -246,8 +245,6 @@ class EventsCBGExecutor : public rclcpp::Executor
 
   std::unique_ptr<cbg_executor::CBGScheduler> scheduler;
 
-  std::thread rcl_polling_thread;
-
   struct CallbackGroupData
   {
     CallbackGroup::WeakPtr callback_group;

rclcpp/src/rclcpp/executors/events_cbg_executor/timer_manager.hpp

@@ -491,7 +491,8 @@ class TimerQueue
 
 class TimerManager
 {
-  std::array<TimerQueue, 3> timer_queues;
+  static constexpr size_t NUM_TYPES_OF_TIMERS = 3;
+  std::array<TimerQueue, NUM_TYPES_OF_TIMERS> timer_queues;
 
 public:
   explicit TimerManager(const rclcpp::Context::SharedPtr & context)