From e735abe33481afb51b30be5561d3c5b8a090384a Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 13 Dec 2011 21:45:26 +0100 Subject: 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. --- src/output/openal_output_plugin.c | 18 ++++-------------- 1 file changed, 4 insertions(+), 14 deletions(-) (limited to 'src/output') 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 = { -- cgit v1.2.3