Vantha/0ad
1
1
Fork 0
forked from 0ad/0ad
Code Pull Requests Activity

Default-init atomics in TaskManager::Impl

Browse Source 
std::atomic<bool> does not value-initialise the boolean.
This caused windows tests to use way too much CPU and fail.

Differential Revision: https://code.wildfiregames.com/D4089
This was SVN commit r25687.
This commit is contained in:
wraitii 2021-06-05 09:48:41 +00:00
parent 3745940de8
commit f3efe47ff6
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
source/ps/TaskManager.cpp
View File

@ -141,8 +141,8 @@ protected:
// Back reference (keep this first).
TaskManager& m_TaskManager;
std::atomic<bool> m_HasWork;
std::atomic<bool> m_HasLowPriorityWork;
std::atomic<bool> m_HasWork = false;
std::atomic<bool> m_HasLowPriorityWork = false;
std::mutex m_GlobalMutex;
std::mutex m_GlobalLowPriorityMutex;
std::deque<QueueItem> m_GlobalQueue;