From 4f0ec03c97a76feea5682112f84933e59328457b Mon Sep 17 00:00:00 2001 From: Lioncash Date: Wed, 21 Nov 2018 21:44:58 -0500 Subject: [PATCH] common/thread: Initialize class member variables where applicable Simplifies the constructor interfaces for Barrier and Event classes. --- src/common/thread.h | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/common/thread.h b/src/common/thread.h index ccfcd6af72..278376cac5 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 lk(mutex); if (!is_set) { @@ -57,14 +55,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() { @@ -85,8 +83,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);