Move the current buffer queue entry and play position to the voice

This has a couple behavioral changes. First and biggest is that querying
AL_BUFFERS_PROCESSED from a source will always return all buffers processed
when in an AL_STOPPED state. Previously all buffers would be set as processed
when first becoming stopped, but newly queued buffers would *not* be indicated
as processed. That old behavior was not compliant with the spec, which
unequivocally states "On a source in the AL_STOPPED state, all buffers are
processed."

Secondly, querying AL_BUFFER on an AL_STREAMING source will now always return
0. Previously it would return the current "active" buffer in the queue, but
there's no basis for that in the spec.

1 files changed, 3 insertions, 6 deletions

diff --git a/Alc/ALu.c b/Alc/ALu.c
index 3b83711f..b4938152 100644
--- a/Alc/ALu.c
+++ b/Alc/ALu.c
@@ -1594,16 +1594,13 @@ void aluHandleDisconnect(ALCdevice *device)
         voice_end = voice + Context->VoiceCount;
         while(voice != voice_end)
         {
-            ALenum playing = AL_PLAYING;
             ALsource *source = (*voice)->Source;
             (*voice)->Source = NULL;
 
-            if(source &&
-               ATOMIC_COMPARE_EXCHANGE_STRONG_SEQ(ALenum, &source->state, &playing, AL_STOPPED))
+            if(source)
             {
-                ATOMIC_STORE(&source->current_buffer, NULL, almemory_order_relaxed);
-                ATOMIC_STORE(&source->position, 0, almemory_order_relaxed);
-                ATOMIC_STORE(&source->position_fraction, 0, almemory_order_release);
+                ALenum playing = AL_PLAYING;
+                ATOMIC_COMPARE_EXCHANGE_STRONG_SEQ(ALenum, &source->state, &playing, AL_STOPPED);
             }
 
             voice++;