fix: simplified condition variable use

removed redundant unlock() and lambda function in - Server::wait_running() - Server::wait_stopped() Signed-off-by: James Chapman <[email protected]>
EVerest · Jul 29, 2024 · f6bbee5 · f6bbee5
1 parent 077b12d
commit f6bbee5
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/lib/staging/tls/tls.cpp b/lib/staging/tls/tls.cpp
@@ -1198,14 +1198,16 @@ void Server::stop() {
 
 void Server::wait_running() {
     std::unique_lock lock(m_cv_mutex);
-    m_cv.wait(lock, [this] { return m_running; });
-    lock.unlock();
+    while (!m_running) {
+        m_cv.wait(lock);
+    }
 }
 
 void Server::wait_stopped() {
     std::unique_lock lock(m_cv_mutex);
-    m_cv.wait(lock, [this] { return !m_running; });
-    lock.unlock();
+    while (m_running) {
+        m_cv.wait(lock);
+    }
 }
 
 // ----------------------------------------------------------------------------