Refs #21817: Add suggested changes & uncrustify

Signed-off-by: Carlosespicur <[email protected]>
eProsima · Nov 19, 2024 · ac5d2cb · ac5d2cb
1 parent 5152155
commit ac5d2cb
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 20 deletions.
diff --git a/src/cpp/database/database_queue.cpp b/src/cpp/database/database_queue.cpp
@@ -190,7 +190,7 @@ EntityId DatabaseEntityQueue::process_datareader(
             if (participant_id.first != info.domain_id)
             {
                 throw BadParameter("Participant " + to_string(participant_guid)
-                          +  " found in the database but it is not in the current domain");
+                              +  " found in the database but it is not in the current domain");
             }
         }
         catch (const Exception&)
@@ -265,7 +265,7 @@ EntityId DatabaseEntityQueue::process_datawriter(
             if (participant_id.first != info.domain_id)
             {
                 throw BadParameter("Participant " + to_string(participant_guid)
-                          +  " found in the database but it is not in the current domain");
+                              +  " found in the database but it is not in the current domain");
             }
         }
         catch (const Exception&)
@@ -328,7 +328,7 @@ EntityId DatabaseEntityQueue::process_endpoint_discovery(
         if (participant_id.first != info.domain_id)
         {
             throw BadParameter("Participant " + to_string(participant_guid)
-                        +  " found in the database but it is not in the current domain");
+                          +  " found in the database but it is not in the current domain");
         }
     }
     catch (const Exception&)
@@ -377,15 +377,7 @@ EntityId DatabaseEntityQueue::process_endpoint_discovery(
     // Create the endpoint
     EntityId endpoint_id;
     std::stringstream name;
-
-    if (info.kind() == EntityKind::DATAREADER)
-    {
-        name << info.topic_name << "_" << info.guid.entityId;
-    }
-    else
-    {
-        name << info.topic_name << "_" << info.guid.entityId;
-    }
+    name << info.topic_name << "_" << info.guid.entityId;
 
     // Endpoint AppId and metadata
     // TODO: get app data from info (parameters), not from participant

diff --git a/test/dds/communication/Communication.hpp b/test/dds/communication/Communication.hpp
@@ -80,9 +80,9 @@ class Communication
     eProsima_user_DllExport Communication(
             const Communication& x)
     {
-                    m_index = x.m_index;
+        m_index = x.m_index;
 
-                    m_message = x.m_message;
+        m_message = x.m_message;
 
     }
 
@@ -105,9 +105,9 @@ class Communication
             const Communication& x)
     {
 
-                    m_index = x.m_index;
+        m_index = x.m_index;
 
-                    m_message = x.m_message;
+        m_message = x.m_message;
 
         return *this;
     }
@@ -133,7 +133,7 @@ class Communication
             const Communication& x) const
     {
         return (m_index == x.m_index &&
-           m_message == x.m_message);
+               m_message == x.m_message);
     }
 
     /*!
@@ -174,7 +174,6 @@ class Communication
         return m_index;
     }
 
-
     /*!
      * @brief This function copies the value in member message
      * @param _message New value to be copied in member message
@@ -213,8 +212,6 @@ class Communication
         return m_message;
     }
 
-
-
 private:
 
     uint32_t m_index{0};