fix TraceFileFactory (#16)

Signed-off-by: Timm Ruppert <[email protected]>
Lichtblick-Suite · Dec 19, 2024 · 0117e7b · 0117e7b
1 parent 8941fbe
commit 0117e7b
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 15 deletions.
diff --git a/include/osi-utilities/tracefile/Reader.h b/include/osi-utilities/tracefile/Reader.h
@@ -110,7 +110,7 @@ class TraceFileReader {
 /**
  * @brief Factory class for creating trace file readers based on file extensions
  */
-class TraceFileFactory {
+class TraceFileReaderFactory {
    public:
     /**
      * @brief Creates a reader instance based on the file extension

diff --git a/src/tracefile/reader/Reader.cpp b/src/tracefile/reader/Reader.cpp
@@ -9,18 +9,19 @@
 #include "osi-utilities/tracefile/reader/TXTHTraceFileReader.h"
 #include "osi-utilities/tracefile/reader/MCAPTraceFileReader.h"
 
-class TraceFileFactory {
-public:
-    static std::unique_ptr<osi3::TraceFileReader> createReader(const std::filesystem::path& path) {
-        if (path.extension().string() == ".osi") {
-            return std::make_unique<osi3::SingleChannelBinaryTraceFileReader>();
-        }
-        if (path.extension().string() == ".mcap") {
-            return std::make_unique<osi3::MCAPTraceFileReader>();
-        }
-        if (path.extension().string() == ".txth") {
-            return std::make_unique<osi3::TXTHTraceFileReader>();
-        }
-        throw std::invalid_argument("Unsupported format: " + path.extension().string());
+namespace osi3 {
+
+std::unique_ptr<osi3::TraceFileReader> TraceFileReaderFactory::createReader(const std::filesystem::path& path) {
+    if (path.extension().string() == ".osi") {
+        return std::make_unique<osi3::SingleChannelBinaryTraceFileReader>();
+    }
+    if (path.extension().string() == ".mcap") {
+        return std::make_unique<osi3::MCAPTraceFileReader>();
     }
-};
+    if (path.extension().string() == ".txth") {
+        return std::make_unique<osi3::TXTHTraceFileReader>();
+    }
+    throw std::invalid_argument("Unsupported format: " + path.extension().string());
+}
+
+}