CI fixes

Signed-off-by: Mauro Passerino <[email protected]>
jefferyyjhsu · Aug 30, 2022 · c2ba785 · c2ba785
1 parent 67b20ae
commit c2ba785
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 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
@@ -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
@@ -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
@@ -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
@@ -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;