Core/Logging: Minor internal refactor

* Avoid formatting to output with fprintf
* Use vector instead of unordered_map to store appenders in Logger

1 files changed, 4 insertions, 10 deletions

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);
 }