From bc3c7a76ffd0d79dbd349ac79ef56fa00c77494f Mon Sep 17 00:00:00 2001
From: Mauro Passerino <mpasserino@irobot.com>
Date: Thu, 30 Jun 2022 14:00:27 +0100
Subject: [PATCH] CI fixes

Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
---
 .../include/rmw_fastrtps_shared_cpp/custom_event_info.hpp     | 4 +++-
 .../rmw_fastrtps_shared_cpp/custom_subscriber_info.hpp        | 1 -
 rmw_fastrtps_shared_cpp/src/custom_publisher_info.cpp         | 3 +--
 rmw_fastrtps_shared_cpp/src/custom_subscriber_info.cpp        | 3 +--
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_event_info.hpp b/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_event_info.hpp
index 8ea7abcf2..5124b464c 100644
--- a/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_event_info.hpp
+++ b/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_event_info.hpp
@@ -15,8 +15,10 @@
 #ifndef RMW_FASTRTPS_SHARED_CPP__CUSTOM_EVENT_INFO_HPP_
 #define RMW_FASTRTPS_SHARED_CPP__CUSTOM_EVENT_INFO_HPP_
 
+#include <algorithm>
 #include <atomic>
 #include <condition_variable>
+#include <limits>
 #include <list>
 #include <memory>
 #include <mutex>
@@ -106,7 +108,7 @@ class EventTypeCallback
 public:
   EventTypeCallback() = default;
 
-  EventTypeCallback(size_t depth)
+  explicit EventTypeCallback(size_t depth)
   {
     history_depth_ = (depth > 0) ? depth : std::numeric_limits<size_t>::max();
   }
diff --git a/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_subscriber_info.hpp b/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_subscriber_info.hpp
index 9acb9d061..1db3fa57f 100644
--- a/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_subscriber_info.hpp
+++ b/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_subscriber_info.hpp
@@ -16,7 +16,6 @@
 #define RMW_FASTRTPS_SHARED_CPP__CUSTOM_SUBSCRIBER_INFO_HPP_
 
 #include <atomic>
-#include <algorithm>
 #include <condition_variable>
 #include <limits>
 #include <memory>
diff --git a/rmw_fastrtps_shared_cpp/src/custom_publisher_info.cpp b/rmw_fastrtps_shared_cpp/src/custom_publisher_info.cpp
index df5bcc28c..2bb61e6a4 100644
--- a/rmw_fastrtps_shared_cpp/src/custom_publisher_info.cpp
+++ b/rmw_fastrtps_shared_cpp/src/custom_publisher_info.cpp
@@ -111,8 +111,7 @@ rmw_ret_t PubListener::set_on_new_event_callback(
   const void * user_data,
   rmw_event_callback_t callback)
 {
-  switch (event_type)
-  {
+  switch (event_type) {
     case RMW_EVENT_LIVELINESS_LOST:
       on_liveliness_lost_.set_callback(user_data, callback);
       break;
diff --git a/rmw_fastrtps_shared_cpp/src/custom_subscriber_info.cpp b/rmw_fastrtps_shared_cpp/src/custom_subscriber_info.cpp
index f660906a8..138ad2c44 100644
--- a/rmw_fastrtps_shared_cpp/src/custom_subscriber_info.cpp
+++ b/rmw_fastrtps_shared_cpp/src/custom_subscriber_info.cpp
@@ -140,8 +140,7 @@ rmw_ret_t SubListener::set_on_new_event_callback(
   const void * user_data,
   rmw_event_callback_t callback)
 {
-  switch (event_type)
-  {
+  switch (event_type) {
     case RMW_EVENT_MESSAGE_LOST:
       on_sample_lost_.set_callback(user_data, callback);
       break;