From ae81caf9a10cbfcb6ef3c6b81b59e73180f73ea2 Mon Sep 17 00:00:00 2001
From: Relintai <relintai@protonmail.com>
Date: Fri, 4 Feb 2022 22:12:08 +0100
Subject: [PATCH] Replaced the mutex in FileCache with an RWLock.

---
 core/file_cache.cpp | 16 ++++++++++------
 core/file_cache.h   |  4 ++--
 2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/core/file_cache.cpp b/core/file_cache.cpp
index 14c0b66..b84e569 100644
--- a/core/file_cache.cpp
+++ b/core/file_cache.cpp
@@ -17,9 +17,13 @@ bool FileCache::wwwroot_has_file(const String &file_path) {
 }
 
 void FileCache::wwwroot_refresh_cache() {
+	_lock.write_lock();
+	
 	registered_files.clear();
 
 	wwwroot_evaluate_dir(wwwroot.c_str());
+
+	_lock.write_unlock();
 }
 
 void FileCache::wwwroot_evaluate_dir(const char *path, const bool should_exist) {
@@ -62,9 +66,9 @@ void FileCache::wwwroot_evaluate_dir(const char *path, const bool should_exist)
 bool FileCache::get_cached_body(const String &path, String *body) {
 	//TODO ERROR MACRO body == null
 
-	//this shouldn't need mutexes
-
+	_lock.read_lock();
 	CacheEntry *e = cache_map[path];
+	_lock.read_unlock();
 
 	if (!e) {
 		return false;
@@ -84,7 +88,7 @@ bool FileCache::get_cached_body(const String &path, String *body) {
 }
 
 void FileCache::set_cached_body(const String &path, const String &body) {
-	cache_mutex.lock();
+	_lock.write_lock();
 
 	CacheEntry *e = cache_map[path];
 
@@ -98,11 +102,11 @@ void FileCache::set_cached_body(const String &path, const String &body) {
 	e->timestamp = current_timestamp;
 	e->body = body;
 
-	cache_mutex.unlock();
+	_lock.write_unlock();
 }
 
 void FileCache::clear() {
-	cache_mutex.lock();
+	_lock.write_lock();
 
 	registered_files.clear();
 
@@ -116,7 +120,7 @@ void FileCache::clear() {
 	
 	cache_map.clear();
 
-	cache_mutex.unlock();
+	_lock.write_unlock();
 }
 
 FileCache::FileCache(bool singleton) {
diff --git a/core/file_cache.h b/core/file_cache.h
index f06ebc7..4c29e81 100644
--- a/core/file_cache.h
+++ b/core/file_cache.h
@@ -6,7 +6,7 @@
 #include <set>
 #include <map>
 #include <chrono>
-#include <mutex>
+#include "core/threading/rw_lock.h"
 
 class FileCache {
 public:
@@ -44,7 +44,7 @@ protected:
         }
     };
 
-    std::mutex cache_mutex;
+    RWLock _lock;
     std::map<String, CacheEntry *> cache_map;
 
 private: