Some simple fixes

Author: florianbecker

source/FileLogger.cpp

@@ -114,7 +114,7 @@ namespace vx {
       return;
     }
 
-    std::string output;
+    std::string output {};
     output.reserve( _message.size() + fileOverhead );
     output.append( timestamp() );
 
@@ -153,8 +153,7 @@ namespace vx {
     std::shared_lock<std::shared_mutex> lock( m_mutex );
 
     /* check if it should be closed and reopened */
-    auto now = std::chrono::system_clock::now();
-
+    const auto now = std::chrono::system_clock::now();
     if ( now - m_lastReopen > m_reopenInterval ) {
 
       m_lastReopen = now;

source/LoggerFactory.cpp

@@ -45,44 +45,8 @@ namespace vx {
     try {
 
       m_creators.try_emplace( "", []( const std::unordered_map<std::string, std::string> &_configuration ) { return std::make_unique<Logger>( _configuration ); } );
-    }
-    catch ( [[maybe_unused]] const std::bad_alloc &_exception ) {
-
-      /* _exception.what() */
-    }
-    catch ( [[maybe_unused]] const std::exception &_exception ) {
-
-      /* _exception.what() */
-    }
-
-    try {
-
       m_creators.try_emplace( "std", []( const std::unordered_map<std::string, std::string> &_configuration ) -> std::unique_ptr<Logger> { return std::make_unique<StdLogger>( _configuration ); } );
-    }
-    catch ( [[maybe_unused]] const std::bad_alloc &_exception ) {
-
-      /* _exception.what() */
-    }
-    catch ( [[maybe_unused]] const std::exception &_exception ) {
-
-      /* _exception.what() */
-    }
-
-    try {
-
       m_creators.try_emplace( "file", []( const std::unordered_map<std::string, std::string> &_configuration ) -> std::unique_ptr<Logger> { return std::make_unique<FileLogger>( _configuration ); } );
-    }
-    catch ( [[maybe_unused]] const std::bad_alloc &_exception ) {
-
-      /* _exception.what() */
-    }
-    catch ( [[maybe_unused]] const std::exception &_exception ) {
-
-      /* _exception.what() */
-    }
-
-    try {
-
       m_creators.try_emplace( "xml", []( const std::unordered_map<std::string, std::string> &_configuration ) -> std::unique_ptr<Logger> { return std::make_unique<XmlFileLogger>( _configuration ); } );
     }
     catch ( [[maybe_unused]] const std::bad_alloc &_exception ) {

source/StdLogger.cpp

@@ -60,7 +60,7 @@ namespace vx {
       return;
     }
 
-    std::string output;
+    std::string output {};
     output.reserve( _message.size() + overhead );
     output.append( timestamp() );
 

source/XmlFileLogger.cpp

@@ -56,7 +56,7 @@ namespace vx {
       return;
     }
 
-    std::string output;
+    std::string output {};
     output.reserve( _message.size() + xmlOverhead );
     output.append( "<entry>" );
     output.append( "<timestamp>" );