Properly add RWMutex

include/RWMutex.h

@@ -0,0 +1,20 @@
+// Author: lionkor
+#pragma once
+
+/*
+ * An RWMutex allows multiple simultaneous readlocks but only one writelock at a time,
+ * and write locks and read locks are mutually exclusive.
+ */
+
+#include <shared_mutex>
+
+// Use ReadLock(m) and WriteLock(m) to lock it.
+using RWMutex = std::shared_mutex;
+// Construct with an RWMutex as a non-const reference.
+// locks the mutex in lock_shared mode (for reading). Locking in a thread that already owns a lock
+// i.e. locking multiple times successively is UB. Construction may be blocking. Destruction is guaranteed to release the lock.
+using ReadLock = std::shared_lock<RWMutex>;
+// Construct with an RWMutex as a non-const reference.
+// locks the mutex for writing. Construction may be blocking. Destruction is guaranteed to release the lock.
+using WriteLock = std::unique_lock<RWMutex>;
+

src/Console.cpp

@@ -27,8 +27,9 @@ void HandleInput(const std::string& cmd) {
     std::cout << std::endl;
     if (cmd == "exit") {
         _Exit(0);
-    } else
+    } else {
         LuaConsole->Execute(cmd);
+    }
 }
 
 void ProcessOut() {

src/logger.cpp

@@ -7,14 +7,10 @@
 #include <chrono>
 #include <fstream>
 #include <mutex>
-#include <shared_mutex>
 #include <sstream>
 #include <thread>
 #include <unordered_map>
-
+#include "RWMutex.h"
-using RWMutex = std::shared_mutex;
-using ReadLock = std::shared_lock<RWMutex>;
-using WriteLock = std::unique_lock<RWMutex>;
 
 static RWMutex ThreadNameMapMutex;
 static std::unordered_map<std::thread::id, std::string> ThreadNameMap;
@@ -29,7 +25,6 @@ std::string ThreadName() {
         ss << std::this_thread::get_id();
         Name = ss.str();
     }
-    lock.unlock();
     return Name;
 }
 
@@ -38,7 +33,6 @@ void SetThreadName(const std::string& Name, bool overwrite) {
     if (overwrite || ThreadNameMap.find(std::this_thread::get_id()) == ThreadNameMap.end()) {
         ThreadNameMap[std::this_thread::get_id()] = Name;
     }
-    lock.unlock();
 }
 
 std::string getDate() {