Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/DefaultTimer.h

Issue 29402565: Issue 5082 - fix possible race condition in DefaultTimer (Closed)
Patch Set: Created April 4, 2017, 10:50 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/DefaultTimer.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/DefaultTimer.h
diff --git a/src/DefaultTimer.h b/src/DefaultTimer.h
index 0f7e7d9a067ca9fd3a59cc4b5dba471a6b8563f9..a9b8d36ad8145e6e1304a95ac5edb9ce2102ac06 100644
--- a/src/DefaultTimer.h
+++ b/src/DefaultTimer.h
@@ -22,7 +22,6 @@
#include <mutex>
#include <condition_variable>
#include <queue>
-#include <atomic>
#include <thread>
namespace AdblockPlus
@@ -53,7 +52,7 @@ namespace AdblockPlus
std::mutex mutex;
std::condition_variable conditionVariable;
TimerUnits timers;
- std::atomic<bool> shouldThreadStop;
+ bool shouldThreadStop;
std::thread m_thread;
};
}
« no previous file with comments | « no previous file | src/DefaultTimer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld