Remove unnecesary error log msg when opening a SHM port. (eProsima#1095)

hzy001 · Mar 25, 2020 · 37bf0dd · 37bf0dd
1 parent 4ec3610
commit 37bf0dd
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/cpp/rtps/transport/shared_mem/SharedMemGlobal.hpp b/src/cpp/rtps/transport/shared_mem/SharedMemGlobal.hpp
@@ -420,6 +420,7 @@ class SharedMemGlobal
             uint32_t healthy_check_timeout_ms,
             Port::OpenMode open_mode = Port::OpenMode::ReadShared)
     {
+        std::string err_reason;
         std::shared_ptr<Port> port;
 
         auto port_segment_name = domain_name_ + "_port" + std::to_string(port_id);
@@ -464,10 +465,12 @@ class SharedMemGlobal
                 if ( (port_node->is_opened_read_exclusive && open_mode != Port::OpenMode::Write) ||
                         (port_node->is_opened_for_reading && open_mode == Port::OpenMode::ReadExclusive))
                 {
-                    logError(RTPS_TRANSPORT_SHM, THREADID << "Couln't open Port "
-                                                         << port_node->port_id << " (" << port_node->uuid.to_string() <<
-                                        ") for reading because is exclusive");
+                    std::stringstream ss;
+
+                    ss << port_node->port_id << " (" << port_node->uuid.to_string() <<
+                                        ") because is already opened ReadExclusive";
 
+                    err_reason = ss.str();
                     port.reset();
                 }
                 else
@@ -540,7 +543,7 @@ class SharedMemGlobal
 
         if (port == nullptr)
         {
-            throw std::runtime_error("Coulnd't open port ");
+            throw std::runtime_error("Coulnd't open port " + err_reason);
         }
 
         return port;