From 634cf07d44bb1374598a26ebbe1ab820ba5eae8b Mon Sep 17 00:00:00 2001 From: Tomasz Sowa Date: Sat, 10 Apr 2021 21:12:28 +0200 Subject: [PATCH] using Winix::Lock object instead of Lock() method in ThreadManager --- winixd/core/threadmanager.cpp | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/winixd/core/threadmanager.cpp b/winixd/core/threadmanager.cpp index 08abf47..bf7fbb4 100644 --- a/winixd/core/threadmanager.cpp +++ b/winixd/core/threadmanager.cpp @@ -130,8 +130,7 @@ void ThreadManager::Add(BaseThread & pbase, const std::wstring & thread_name) void ThreadManager::StartAll() { - synchro->Lock(); - + Winix::Lock lock(synchro); int id = 0; for(ThreadItem & item : thread_tab) @@ -140,7 +139,7 @@ void ThreadManager::StartAll() id += 1; } - synchro->Unlock(); + lock.Unlock(); were_started = true; } @@ -165,17 +164,16 @@ void ThreadManager::StopAll() if( !were_started ) return; - // WakeUpThread() should be used with Lock/Unlock - synchro->Lock(); - - for(ThreadItem & item : thread_tab) { - item.object->WakeUpThread(); + Winix::Lock lock(synchro); + + for(ThreadItem & item : thread_tab) + { + // WakeUpThread() should be used with Lock/Unlock + item.object->WakeUpThread(); + } } - synchro->Unlock(); - - int id = 0; for(ThreadItem & item : thread_tab)