output_api: moved the command check out of method pause()

Move the "while" loop which checks for commands to the caller
ao_pause().  This simplifies the pause() method, and lets us remove
audio_output_is_pending().

1 files changed, 3 insertions, 10 deletions

diff --git a/src/output/shout_plugin.c b/src/output/shout_plugin.c
index 3fc372250..d2a8db597 100644
--- a/src/output/shout_plugin.c
+++ b/src/output/shout_plugin.c
@@ -506,19 +506,12 @@ my_shout_play(void *data, const char *chunk, size_t size)
 	return true;
 }
 
-static void my_shout_pause(void *data)
+static bool
+my_shout_pause(void *data)
 {
-	struct shout_data *sd = (struct shout_data *)data;
 	static const char silence[1020];
-	int ret;
 
-	/* play silence until the player thread sends us a command */
-
-	while (sd->opened && !audio_output_is_pending(sd->audio_output)) {
-		ret = my_shout_play(data, silence, sizeof(silence));
-		if (ret != 0)
-			break;
-	}
+	return my_shout_play(data, silence, sizeof(silence));
 }
 
 static void my_shout_set_tag(void *data,