Merge pull request #28 from mauropasse/mauro/add-events-executor-fixes

Fix linter issues
irobot-ros · Feb 10, 2022 · 490de16 · 490de16
2 parents 5d5c981 + 2e189cf
commit 490de16
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 6 deletions.
diff --git a/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_client_info.hpp b/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_client_info.hpp
@@ -130,7 +130,7 @@ class ClientListener : public eprosima::fastdds::dds::DataReaderListener
 
           std::unique_lock<std::mutex> lock_mutex(on_new_response_m_);
 
-          if(on_new_response_cb_) {
+          if (on_new_response_cb_) {
             on_new_response_cb_(user_data_, 1);
           } else {
             unread_count_++;

diff --git a/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_service_info.hpp b/rmw_fastrtps_shared_cpp/include/rmw_fastrtps_shared_cpp/custom_service_info.hpp
@@ -251,7 +251,7 @@ class ServiceListener : public eprosima::fastdds::dds::DataReaderListener
 
         std::unique_lock<std::mutex> lock_mutex(on_new_request_m_);
 
-        if(on_new_request_cb_) {
+        if (on_new_request_cb_) {
           on_new_request_cb_(user_data_, 1);
         } else {
           unread_count_++;

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,6 +16,7 @@
 #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
@@ -43,7 +43,7 @@ PubListener::on_offered_deadline_missed(
 
   std::unique_lock<std::mutex> lock_mutex(on_new_event_m_);
 
-  if(on_new_event_cb_){
+  if (on_new_event_cb_){
     on_new_event_cb_(user_data_, 1);
   } else {
     unread_events_count_++;
@@ -69,7 +69,7 @@ void PubListener::on_liveliness_lost(
 
   std::unique_lock<std::mutex> lock_mutex(on_new_event_m_);
 
-  if(on_new_event_cb_) {
+  if (on_new_event_cb_) {
     on_new_event_cb_(user_data_, 1);
   } else {
     unread_events_count_++;

diff --git a/rmw_fastrtps_shared_cpp/src/custom_subscriber_info.cpp b/rmw_fastrtps_shared_cpp/src/custom_subscriber_info.cpp
@@ -45,7 +45,7 @@ SubListener::on_requested_deadline_missed(
 
   std::unique_lock<std::mutex> lock_mutex(on_new_event_m_);
 
-  if(on_new_event_cb_) {
+  if (on_new_event_cb_) {
     on_new_event_cb_(user_data_, 1);
   } else {
     unread_events_count_++;
@@ -73,7 +73,7 @@ void SubListener::on_liveliness_changed(
 
   std::unique_lock<std::mutex> lock_mutex(on_new_event_m_);
 
-  if(on_new_event_cb_) {
+  if (on_new_event_cb_) {
     on_new_event_cb_(user_data_, 1);
   } else {
     unread_events_count_++;