Refs 20739: Rev 3 changes

Signed-off-by: Mario Dominguez <[email protected]>
eProsima · Apr 5, 2024 · 82a8b06 · 82a8b06
1 parent 97abb94
commit 82a8b06
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/cpp/fastdds/subscriber/SubscriberImpl.hpp b/src/cpp/fastdds/subscriber/SubscriberImpl.hpp
@@ -21,18 +21,18 @@
 #define _FASTDDS_SUBSCRIBERIMPL_HPP_
 #ifndef DOXYGEN_SHOULD_SKIP_THIS_PUBLIC
 
-#include <fastrtps/attributes/SubscriberAttributes.h>
+#include <map>
+#include <mutex>
 
 #include <fastdds/dds/core/status/StatusMask.hpp>
 #include <fastdds/dds/subscriber/DataReaderListener.hpp>
 #include <fastdds/dds/subscriber/qos/DataReaderQos.hpp>
 #include <fastdds/dds/subscriber/qos/SubscriberQos.hpp>
+#include <fastrtps/attributes/SubscriberAttributes.h>
 #include <fastrtps/types/TypesBase.h>
 
 #include <statistics/rtps/monitor-service/interfaces/IStatusQueryable.hpp>
 
-#include <map>
-#include <mutex>
 
 using eprosima::fastrtps::types::ReturnCode_t;
 

diff --git a/src/cpp/statistics/rtps/monitor-service/MonitorService.cpp b/src/cpp/statistics/rtps/monitor-service/MonitorService.cpp
@@ -291,7 +291,7 @@ bool MonitorService::write_status(
                     case INCONSISTENT_TOPIC:
                     {
                         EPROSIMA_LOG_ERROR(MONITOR_SERVICE, "Inconsistent topic status not supported yet");
-                        (void)local_entity_guid;
+                        static_cast<void>(local_entity_guid);
                         break;
                     }
                     case LIVELINESS_LOST:
@@ -321,7 +321,7 @@ bool MonitorService::write_status(
                     default:
                     {
                         EPROSIMA_LOG_ERROR(MONITOR_SERVICE, "Referring to an unknown status");
-                        (void)local_entity_guid;
+                        static_cast<void>(local_entity_guid);
                         break;
                     }
                 }
@@ -333,6 +333,11 @@ bool MonitorService::write_status(
                     status_data.value(data);
                     add_change(status_data, false);
                 }
+                else
+                {
+                    EPROSIMA_LOG_ERROR(MONITOR_SERVICE, "Could not retrieve the status data for " << i << " of " <<
+                            local_entity_guid);
+                }
             }
         }
     }