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, 14 insertions, 12 deletions

diff --git a/Alc/ALc.c b/Alc/ALc.c
index c829d8e7..bf683ba3 100644
--- a/Alc/ALc.c
+++ b/Alc/ALc.c
@@ -1692,22 +1692,24 @@ void ALCcontext_ProcessUpdates(ALCcontext *context)
 
         LockUIntMapRead(&context->SourceMap);
         V0(device->Backend,lock)();
-        for(pos = 0;pos < context->SourceMap.size;pos++)
+        for(pos = 0;pos < context->VoiceCount;pos++)
         {
-            ALsource *Source = context->SourceMap.values[pos];
-            ALenum new_state;
-
-            if(Source->OffsetType != AL_NONE && IsPlayingOrPaused(Source))
+            ALvoice *voice = context->Voices[pos];
+            ALsource *source = voice->Source;
+            if(source && source->OffsetType != AL_NONE)
             {
-                WriteLock(&Source->queue_lock);
-                ApplyOffset(Source);
-                WriteUnlock(&Source->queue_lock);
+                WriteLock(&source->queue_lock);
+                ApplyOffset(source, voice);
+                WriteUnlock(&source->queue_lock);
             }
-
-            new_state = Source->new_state;
-            Source->new_state = AL_NONE;
+        }
+        for(pos = 0;pos < context->SourceMap.size;pos++)
+        {
+            ALsource *source = context->SourceMap.values[pos];
+            ALenum new_state = source->new_state;
+            source->new_state = AL_NONE;
             if(new_state)
-                SetSourceState(Source, context, new_state);
+                SetSourceState(source, context, new_state);
         }
         V0(device->Backend,unlock)();
         UnlockUIntMapRead(&context->SourceMap);