use std::lock_guard instead of std::unique_lock (#3179)

wsehjk · web-flow · commit 2169a6f6ae52 · 2024-09-11T22:18:51.000+03:00
diff --git a/include/spdlog/details/mpmc_blocking_q.h b/include/spdlog/details/mpmc_blocking_q.h
@@ -148,19 +148,19 @@ class mpmc_blocking_queue {
 #endif
 
     size_t overrun_counter() {
-        std::unique_lock<std::mutex> lock(queue_mutex_);
+        std::lock_guard<std::mutex> lock(queue_mutex_);
         return q_.overrun_counter();
     }
 
     size_t discard_counter() { return discard_counter_.load(std::memory_order_relaxed); }
 
     size_t size() {
-        std::unique_lock<std::mutex> lock(queue_mutex_);
+        std::lock_guard<std::mutex> lock(queue_mutex_);
         return q_.size();
     }
 
     void reset_overrun_counter() {
-        std::unique_lock<std::mutex> lock(queue_mutex_);
+        std::lock_guard<std::mutex> lock(queue_mutex_);
         q_.reset_overrun_counter();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -148,19 +148,19 @@ class mpmc_blocking_queue {`
`148`	`148`	`#endif`
`149`	`149`
`150`	`150`	`size_t overrun_counter() {`
`151`		`- std::unique_lock<std::mutex> lock(queue_mutex_);`
	`151`	`+ std::lock_guard<std::mutex> lock(queue_mutex_);`
`152`	`152`	`return q_.overrun_counter();`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`size_t discard_counter() { return discard_counter_.load(std::memory_order_relaxed); }`
`156`	`156`
`157`	`157`	`size_t size() {`
`158`		`- std::unique_lock<std::mutex> lock(queue_mutex_);`
	`158`	`+ std::lock_guard<std::mutex> lock(queue_mutex_);`
`159`	`159`	`return q_.size();`
`160`	`160`	`}`
`161`	`161`
`162`	`162`	`void reset_overrun_counter() {`
`163`		`- std::unique_lock<std::mutex> lock(queue_mutex_);`
	`163`	`+ std::lock_guard<std::mutex> lock(queue_mutex_);`
`164`	`164`	`q_.reset_overrun_counter();`
`165`	`165`	`}`
`166`	`166`