Always clear log consumers before exiting

Signed-off-by: tempate <[email protected]>
eProsima · Jul 3, 2024 · 6776265 · 6776265
1 parent 8dabbce
commit 6776265
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/fastddsspy_tool/src/cpp/main.cpp b/fastddsspy_tool/src/cpp/main.cpp
@@ -47,6 +47,10 @@
 #include "user_interface/ProcessReturnCode.hpp"
 #include "tool/Controller.hpp"
 
+
+int exit(const eprosima::spy::ProcessReturnCode& code);
+
+
 int main(
         int argc,
         char** argv)
@@ -60,15 +64,15 @@ int main(
 
     if (arg_parse_result == eprosima::spy::ProcessReturnCode::help_argument)
     {
-        return static_cast<int>(eprosima::spy::ProcessReturnCode::success);
+        return exit(eprosima::spy::ProcessReturnCode::success);
     }
     else if (arg_parse_result == eprosima::spy::ProcessReturnCode::version_argument)
     {
-        return static_cast<int>(eprosima::spy::ProcessReturnCode::success);
+        return exit(eprosima::spy::ProcessReturnCode::success);
     }
     else if (arg_parse_result != eprosima::spy::ProcessReturnCode::success)
     {
-        return static_cast<int>(arg_parse_result);
+        return exit(arg_parse_result);
     }
 
     // Check file is in args, else get the default file
@@ -236,21 +240,23 @@ int main(
                 "Error Loading Fast DDS Spy Configuration from file " << commandline_args.file_path <<
                 ". Error message:\n " <<
                 e.what());
-        return static_cast<int>(eprosima::spy::ProcessReturnCode::execution_failed);
+        return exit(eprosima::spy::ProcessReturnCode::execution_failed);
     }
     catch (const eprosima::utils::InitializationException& e)
     {
         logError(FASTDDSSPY_TOOL,
                 "Error Initializing Fast DDS Spy. Error message:\n " <<
                 e.what());
-        return static_cast<int>(eprosima::spy::ProcessReturnCode::execution_failed);
+        return exit(eprosima::spy::ProcessReturnCode::execution_failed);
     }
 
-    // Force print every log before closing
-    eprosima::utils::Log::Flush();
+    return exit(eprosima::spy::ProcessReturnCode::success);
+}
 
+int exit(const eprosima::spy::ProcessReturnCode& code)
+{
     // Delete the consumers before closing
     eprosima::utils::Log::ClearConsumers();
 
-    return static_cast<int>(eprosima::spy::ProcessReturnCode::success);
+    return static_cast<int>(code);
 }