diff options
| author | Lioncash <mathew1800@gmail.com> | 2018-11-21 21:44:58 -0500 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2018-11-21 21:45:01 -0500 |
| commit | 756e773096c5a64cb5c0ff48104ceb36fa1935cb (patch) | |
| tree | 0f6cc34f9bf1213cfa8fd3faca64b8cd7d43b591 /src | |
| parent | 02602afd100ad7489de6a5da54ead3e7f22cbac7 (diff) | |
common/thread: Initialize class member variables where applicable
Simplifies the constructor interfaces for Barrier and Event classes.
Diffstat (limited to 'src')
| -rw-r--r-- | src/common/thread.h | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/common/thread.h b/src/common/thread.h index 816183bc8c..741dce4872 100644 --- a/src/common/thread.h +++ b/src/common/thread.h @@ -15,8 +15,6 @@ namespace Common { class Event { public: - Event() : is_set(false) {} - void Set() { std::lock_guard<std::mutex> lk(mutex); if (!is_set) { @@ -48,14 +46,14 @@ public: } private: - bool is_set; + bool is_set = false; std::condition_variable condvar; std::mutex mutex; }; class Barrier { public: - explicit Barrier(std::size_t count_) : count(count_), waiting(0), generation(0) {} + explicit Barrier(std::size_t count_) : count(count_) {} /// Blocks until all "count" threads have called Sync() void Sync() { @@ -76,8 +74,8 @@ private: std::condition_variable condvar; std::mutex mutex; const std::size_t count; - std::size_t waiting; - std::size_t generation; // Incremented once each time the barrier is used + std::size_t waiting = 0; + std::size_t generation = 0; // Incremented once each time the barrier is used }; void SetThreadAffinity(std::thread::native_handle_type thread, u32 mask); |