From 19e4672a5431aadfd6cc40a576f53a0027af360f Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@musicpd.org>
Date: Fri, 17 May 2019 11:17:16 +0200
Subject: [PATCH] input/buffering: use notify_all() instead of notify_one()

More preparations to support multiple readers.
---
 src/input/BufferingInputStream.cxx | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/input/BufferingInputStream.cxx b/src/input/BufferingInputStream.cxx
index 9542e48b2..32ac0749c 100644
--- a/src/input/BufferingInputStream.cxx
+++ b/src/input/BufferingInputStream.cxx
@@ -39,7 +39,7 @@ BufferingInputStream::~BufferingInputStream() noexcept
 	{
 		const std::lock_guard<Mutex> lock(mutex);
 		stop = true;
-		wake_cond.notify_one();
+		wake_cond.notify_all();
 	}
 
 	thread.Join();
@@ -73,7 +73,7 @@ BufferingInputStream::Seek(std::unique_lock<Mutex> &lock, size_t new_offset)
 
 	seek_offset = new_offset;
 	seek = true;
-	wake_cond.notify_one();
+	wake_cond.notify_all();
 
 	client_cond.wait(lock, [this]{ return !seek; });
 
@@ -105,7 +105,7 @@ BufferingInputStream::Read(std::unique_lock<Mutex> &lock, void *ptr, size_t s)
 
 			if (!IsAvailable()) {
 				/* wake up the sleeping thread */
-				wake_cond.notify_one();
+				wake_cond.notify_all();
 			}
 
 			return nbytes;
@@ -150,7 +150,7 @@ BufferingInputStream::RunThread() noexcept
 			}
 
 			seek = false;
-			client_cond.notify_one();
+			client_cond.notify_all();
 		} else if (offset != input->GetOffset() && !IsAvailable()) {
 			/* a past Seek() call was a no-op because data
 			   was already available at that position, but
@@ -172,7 +172,7 @@ BufferingInputStream::RunThread() noexcept
 				   own InputStream interface) is in
 				   "read" mode */
 				read_error = std::current_exception();
-				client_cond.notify_one();
+				client_cond.notify_all();
 				OnBufferAvailable();
 				break;
 			}
@@ -191,7 +191,7 @@ BufferingInputStream::RunThread() noexcept
 				input->Seek(lock, new_offset);
 			} catch (...) {
 				read_error = std::current_exception();
-				client_cond.notify_one();
+				client_cond.notify_all();
 				OnBufferAvailable();
 				break;
 			}
@@ -215,7 +215,7 @@ BufferingInputStream::RunThread() noexcept
 						input->Seek(lock, new_offset);
 					} catch (...) {
 						read_error = std::current_exception();
-						client_cond.notify_one();
+						client_cond.notify_all();
 						OnBufferAvailable();
 						break;
 					}
@@ -230,7 +230,7 @@ BufferingInputStream::RunThread() noexcept
 						input->Seek(lock, offset);
 					} catch (...) {
 						read_error = std::current_exception();
-						client_cond.notify_one();
+						client_cond.notify_all();
 						OnBufferAvailable();
 						break;
 					}
@@ -246,12 +246,12 @@ BufferingInputStream::RunThread() noexcept
 					      read_offset + nbytes);
 			} catch (...) {
 				read_error = std::current_exception();
-				client_cond.notify_one();
+				client_cond.notify_all();
 				OnBufferAvailable();
 				break;
 			}
 
-			client_cond.notify_one();
+			client_cond.notify_all();
 			OnBufferAvailable();
 		} else
 			wake_cond.wait(lock);