From 81bec6954fa6640e2942d836a856e6bc9d421bce Mon Sep 17 00:00:00 2001
From: Shauren <shauren.trinity@gmail.com>
Date: Tue, 16 Jul 2024 16:31:40 +0200
Subject: Core/Logging: Minor internal refactor

* Avoid formatting to output with fprintf
* Use vector instead of unordered_map to store appenders in Logger
---
 src/common/Logging/Logger.cpp | 14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

(limited to 'src/common/Logging/Logger.cpp')

diff --git a/src/common/Logging/Logger.cpp b/src/common/Logging/Logger.cpp
index 63653770ade..571e4d6b7ae 100644
--- a/src/common/Logging/Logger.cpp
+++ b/src/common/Logging/Logger.cpp
@@ -31,14 +31,9 @@ LogLevel Logger::getLogLevel() const
     return level;
 }
 
-void Logger::addAppender(uint8 id, Appender* appender)
+void Logger::addAppender(Appender* appender)
 {
-    appenders[id] = appender;
-}
-
-void Logger::delAppender(uint8 id)
-{
-    appenders.erase(id);
+    appenders.push_back(appender);
 }
 
 void Logger::setLogLevel(LogLevel _level)
@@ -54,7 +49,6 @@ void Logger::write(LogMessage* message) const
         return;
     }
 
-    for (std::pair<uint8 const, Appender*> const& appender : appenders)
-        if (appender.second)
-            appender.second->write(message);
+    for (Appender* appender : appenders)
+        appender->write(message);
 }
-- 
cgit v1.2.3