Revert "add line feed for RCUTILS_SAFE_FWRITE_TO_STDERR (#608)" (#612)

This reverts commit 7202ceb. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
ros2 · Jun 17, 2022 · 1017f15 · 1017f15
1 parent 7202ceb
commit 1017f15
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/rmw_fastrtps_cpp/src/rmw_node.cpp b/rmw_fastrtps_cpp/src/rmw_node.cpp
@@ -68,7 +68,7 @@ rmw_create_node(
     if (RMW_RET_OK != rmw_fastrtps_shared_cpp::decrement_context_impl_ref_count(context)) {
       RCUTILS_SAFE_FWRITE_TO_STDERR(
         "'decrement_context_impl_ref_count' failed while being executed due to '"
-        RCUTILS_STRINGIFY(__function__) "' failing.\n");
+        RCUTILS_STRINGIFY(__function__) "' failing");
     }
   }
   return node;

diff --git a/rmw_fastrtps_dynamic_cpp/src/rmw_node.cpp b/rmw_fastrtps_dynamic_cpp/src/rmw_node.cpp
@@ -68,7 +68,7 @@ rmw_create_node(
     if (RMW_RET_OK != rmw_fastrtps_shared_cpp::decrement_context_impl_ref_count(context)) {
       RCUTILS_SAFE_FWRITE_TO_STDERR(
         "'decrement_context_impl_ref_count' failed while being executed due to '"
-        RCUTILS_STRINGIFY(__function__) "' failing.\n");
+        RCUTILS_STRINGIFY(__function__) "' failing");
     }
   }
   return node;

diff --git a/rmw_fastrtps_shared_cpp/src/listener_thread.cpp b/rmw_fastrtps_shared_cpp/src/listener_thread.cpp
@@ -69,7 +69,7 @@ rmw_fastrtps_shared_cpp::run_listener_thread(rmw_context_t * context)
     {
       RCUTILS_SAFE_FWRITE_TO_STDERR(
         RCUTILS_STRINGIFY(__FILE__) ":" RCUTILS_STRINGIFY(__function__) ":"
-        RCUTILS_STRINGIFY(__LINE__) ": Failed to destroy guard condition.\n");
+        RCUTILS_STRINGIFY(__LINE__) ": Failed to destroy guard condition");
     }
   }
   return RMW_RET_ERROR;

diff --git a/rmw_fastrtps_shared_cpp/src/participant.cpp b/rmw_fastrtps_shared_cpp/src/participant.cpp
@@ -316,7 +316,7 @@ rmw_fastrtps_shared_cpp::destroy_participant(CustomParticipantInfo * participant
     }
     ret = participant_info->participant_->delete_publisher(participant_info->publisher_);
     if (ReturnCode_t::RETCODE_OK != ret) {
-      RCUTILS_SAFE_FWRITE_TO_STDERR("Failed to delete dds publisher from participant.\n");
+      RCUTILS_SAFE_FWRITE_TO_STDERR("Failed to delete dds publisher from participant");
     }
   }
 
@@ -330,7 +330,7 @@ rmw_fastrtps_shared_cpp::destroy_participant(CustomParticipantInfo * participant
     }
     ret = participant_info->participant_->delete_subscriber(participant_info->subscriber_);
     if (ReturnCode_t::RETCODE_OK != ret) {
-      RCUTILS_SAFE_FWRITE_TO_STDERR("Failed to delete dds subscriber from participant.\n");
+      RCUTILS_SAFE_FWRITE_TO_STDERR("Failed to delete dds subscriber from participant");
     }
   }
 
@@ -346,7 +346,7 @@ rmw_fastrtps_shared_cpp::destroy_participant(CustomParticipantInfo * participant
     participant_info->participant_);
 
   if (ReturnCode_t::RETCODE_OK != ret) {
-    RCUTILS_SAFE_FWRITE_TO_STDERR("Failed to delete participant.\n");
+    RCUTILS_SAFE_FWRITE_TO_STDERR("Failed to delete participant");
   }
 
   // Delete Listener