From 92d9797b8cb5b51cdb1cb7f40b12a87ad91eac2e Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@duempel.org>
Date: Tue, 26 Aug 2008 08:27:16 +0200
Subject: player: don't call STOP before CLOSE_AUDIO

playerWait() stops the player thread (twice!) and closes the output
device.  It should be well enough to just send CLOSE_AUDIO, without
STOP.

This requires a tiny change to the player thread code: make it break
when CLOSE_AUDIO is sent.
---
 src/player.c | 22 ++++++++--------------
 1 file changed, 8 insertions(+), 14 deletions(-)

(limited to 'src/player.c')

diff --git a/src/player.c b/src/player.c
index fae405ae3..5c3319491 100644
--- a/src/player.c
+++ b/src/player.c
@@ -25,14 +25,6 @@
 #include "os_compat.h"
 #include "main_notify.h"
 
-static void playerCloseAudio(void);
-
-void playerWait(void)
-{
-	playerStop();
-	playerCloseAudio();
-}
-
 static void set_current_song(Song *song)
 {
 	assert(song != NULL);
@@ -76,6 +68,14 @@ void playerStop(void)
 	playerQueueUnlock();
 }
 
+void playerWait(void)
+{
+	player_command(PLAYER_COMMAND_CLOSE_AUDIO);
+
+	pc.queueState = PLAYER_QUEUE_BLANK;
+	playerQueueUnlock();
+}
+
 void playerKill(void) /* deprecated */
 {
 	playerPause();
@@ -165,12 +165,6 @@ char *getPlayerErrorStr(void)
 	return *error ? error : NULL;
 }
 
-static void playerCloseAudio(void)
-{
-	playerStop();
-	player_command(PLAYER_COMMAND_CLOSE_AUDIO);
-}
-
 int queueSong(Song * song)
 {
 	if (pc.queueState == PLAYER_QUEUE_BLANK) {
-- 
cgit v1.2.3