output/openal: use alGetSourcei(AL_BUFFER) to force-unqueue buffers

The implementation of cancel() did not work well: you cannot use
alSourceUnqueueBuffers() to unqueue queued buffers, and our function
openal_unqueue_buffers() left the OpenAL library in a rather undefined
state; nothing was supposed to be queued, but the "filled" variable
was not reset.

1 files changed, 4 insertions, 14 deletions

diff --git a/src/output/openal_output_plugin.c b/src/output/openal_output_plugin.c
index 016189ae3..6f2c049b8 100644
--- a/src/output/openal_output_plugin.c
+++ b/src/output/openal_output_plugin.c
@@ -109,19 +109,6 @@ openal_setup_context(struct openal_data *od,
 	return true;
 }
 
-static void
-openal_unqueue_buffers(struct openal_data *od)
-{
-	ALint num;
-	ALuint buffer;
-
-	alGetSourcei(od->source, AL_BUFFERS_QUEUED, &num);
-
-	while (num--) {
-		alSourceUnqueueBuffers(od->source, 1, &buffer);
-	}
-}
-
 static struct audio_output *
 openal_init(const struct config_param *param, GError **error_r)
 {
@@ -256,7 +243,10 @@ openal_cancel(struct audio_output *ao)
 	od->filled = 0;
 	alcMakeContextCurrent(od->context);
 	alSourceStop(od->source);
-	openal_unqueue_buffers(od);
+
+	/* force-unqueue all buffers */
+	alSourcei(od->source, AL_BUFFER, 0);
+	od->filled = 0;
 }
 
 const struct audio_output_plugin openal_output_plugin = {