Apply more suggestions

Signed-off-by: tempate <[email protected]>

Author: Tempate

ddspipe_core/include/ddspipe_core/efficiency/payload/PayloadPoolMediator.hpp

@@ -33,8 +33,8 @@ namespace ddspipe {
 namespace core {
 
 /**
- * DataWriters always allocate memory for a the payload when writing it to the \c PayloadPool. In the DdsPipe, however,
- * the \c payload is already in the \c PayloadPool (saved by the DataReader), so we can reuse it to avoid an extra copy.
+ * DataWriters always allocate memory for a payload when writing it to the \c PayloadPool. In the DdsPipe, however, the
+ * \c payload is already in the \c PayloadPool (saved by the Reader), so we can reuse it to avoid an extra copy.
  *
  * This class works as a mediator between the \c FastPayloadPool and the DataWriter's write functions, to avoid making
  * the extra copy.

ddspipe_participants/src/cpp/types/dds/TopicDataType.cpp

@@ -66,6 +66,8 @@ bool TopicDataType::serialize(
     }
     else
     {
+        logWarning(DDSPIPE_DDS_TYPESUPPORT, "Copying the payload between two different payload pools.");
+
         // The src and dst Payload Pools are different. The payload must be copied.
         target_payload->copy(&src_payload->payload);
     }