Improved performance by not locking when message is discarded

Author: doe300

.travis.yml

@@ -13,12 +13,12 @@ matrix:
       - COMPILER=clang++
       - CPPFLAGS="-I /usr/include/c++/v1"
       - CPPFLAGS="-stdlib=libc++"
-    - os: osx
-      compiler: clang
-      env: 
-      - COMPILER=clang++
-      - CPPFLAGS="-stdlib=libc++"
-      # GCC is linked to clang, so we don't need to compile twice for same compiler/platform
+#    - os: osx
+#      compiler: clang
+#      env: 
+#      - COMPILER=clang++
+#      - CPPFLAGS="-stdlib=libc++"
+#      # GCC is linked to clang, so we don't need to compile twice for same compiler/platform
 
 before_script:
   - echo $TRAVIS_OS_NAME

src/logger.cpp

@@ -67,10 +67,10 @@ void ConsoleLogger::logMessage(const Level level,
                                const std::wstring& local,
                                const std::chrono::system_clock::time_point timestamp)
 {
+	if (!logLevel(level)) {
+		return;
+	}
 	std::lock_guard<std::mutex> guard(writeLock);
-    if (!logLevel(level)) {
-        return;
-    }
     if (level == Level::ERROR || level == Level::SEVERE) {
         std::wcerr << toString(level) << " " << getCurrentTime() << ": " << local;
     }
@@ -93,10 +93,10 @@ void FileLogger::logMessage(const Level level,
                             const std::wstring& local,
                             const std::chrono::system_clock::time_point timestamp)
 {
-	std::lock_guard<std::mutex> guard(writeLock);
 	if (!logLevel(level)) {
 		return;
 	}
+	std::lock_guard<std::mutex> guard(writeLock);
 	fileStream << toString(level) << " " << getCurrentTime() << ": " << local;
 }
 
@@ -112,10 +112,10 @@ void ColoredLogger::logMessage(const Level level,
                                const std::wstring& local,
                                const std::chrono::system_clock::time_point timestamp)
 {
-	std::lock_guard<std::mutex> guard(writeLock);
     if (!logLevel(level)) {
         return;
     }
+    std::lock_guard<std::mutex> guard(writeLock);
     if (level == Level::ERROR || level == Level::SEVERE) {
         stream << "\033[31m" << toString(level) << " " << getCurrentTime() << ": " << local << "\033[39;49m";