From: Stephen Seo <seo.disparate@gmail.com>
Date: Thu, 19 Dec 2019 02:39:34 +0000 (+0900)
Subject: Minor refactorings
X-Git-Tag: 1.0~101
X-Git-Url: https://git.seodisparate.com/stephenseo/server_config?a=commitdiff_plain;h=1af44db109f4fdf12a8225125936d73e782ae93e;p=UDPConnection

Minor refactorings

UDPC_update now locks the mutex to keep it thread-safe.
---

diff --git a/src/UDPConnection.cpp b/src/UDPConnection.cpp
index 6e29c15..79788c9 100644
--- a/src/UDPConnection.cpp
+++ b/src/UDPConnection.cpp
@@ -1784,9 +1784,10 @@ void UDPC::threadedUpdate(Context *ctx) {
     decltype(now) nextNow;
     while(ctx->threadRunning.load()) {
         now = std::chrono::steady_clock::now();
-        ctx->mutex.lock();
-        ctx->update_impl();
-        ctx->mutex.unlock();
+        {
+            std::lock_guard<std::mutex> lock(ctx->mutex);
+            ctx->update_impl();
+        }
         nextNow = std::chrono::steady_clock::now();
         std::this_thread::sleep_for(ctx->threadedSleepTime - (nextNow - now));
     }
@@ -2075,6 +2076,7 @@ void UDPC_update(UDPC_HContext ctx) {
         return;
     }
 
+    std::lock_guard<std::mutex> lock(c->mutex);
     c->update_impl();
 }