diff options
author | Shauren <shauren.trinity@gmail.com> | 2015-07-15 16:51:12 +0200 |
---|---|---|
committer | Shauren <shauren.trinity@gmail.com> | 2015-07-15 16:51:12 +0200 |
commit | 08316552b5b176e8209d7e8f15c53053d049743d (patch) | |
tree | dc8d541cd44166f03abafbd7835e5a307f3893da | |
parent | 35ffea8d219d25cbdaa7e0ac4872d9b5b6fa2a98 (diff) |
Core/Threading: Clarified comment on ProducerConsumerQueue::WaitAndPop
-rw-r--r-- | src/server/shared/Threading/ProducerConsumerQueue.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/server/shared/Threading/ProducerConsumerQueue.h b/src/server/shared/Threading/ProducerConsumerQueue.h index 3158e3deb24..223f5109545 100644 --- a/src/server/shared/Threading/ProducerConsumerQueue.h +++ b/src/server/shared/Threading/ProducerConsumerQueue.h @@ -70,7 +70,8 @@ public: { std::unique_lock<std::mutex> lock(_queueLock); - // we could be using .wait(lock, predicate) overload here but some threading error analysis tools produce false positives + // we could be using .wait(lock, predicate) overload here but it is broken + // https://connect.microsoft.com/VisualStudio/feedback/details/1098841 while (_queue.empty() && !_shutdown) _condition.wait(lock); |