Allow effect slots to be updated asynchronously

Updates when the slot changes effect type is still sychronous, however, to
ensure a proper state for the Process method call. Fixing this would
essentially require all effects to work from the same state.

2 files changed, 20 insertions, 4 deletions

diff --git a/OpenAL32/Include/alAuxEffectSlot.h b/OpenAL32/Include/alAuxEffectSlot.h
index 09bc363e..eced4872 100644
--- a/OpenAL32/Include/alAuxEffectSlot.h
+++ b/OpenAL32/Include/alAuxEffectSlot.h
@@ -18,6 +18,7 @@ typedef struct ALeffectslot
     ALfloat Gain;
     ALboolean AuxSendAuto;
 
+    ALboolean     NeedsUpdate;
     ALeffectState *EffectState;
 
     ALfloat WetBuffer[BUFFERSIZE];
diff --git a/OpenAL32/alAuxEffectSlot.c b/OpenAL32/alAuxEffectSlot.c
index 09e29f1e..b4046c71 100644
--- a/OpenAL32/alAuxEffectSlot.c
+++ b/OpenAL32/alAuxEffectSlot.c
@@ -86,6 +86,7 @@ AL_API ALvoid AL_APIENTRY alGenAuxiliaryEffectSlots(ALsizei n, ALuint *effectslo
 
             slot->Gain = 1.0;
             slot->AuxSendAuto = AL_TRUE;
+            slot->NeedsUpdate = AL_FALSE;
             for(j = 0;j < BUFFERSIZE;j++)
                 slot->WetBuffer[j] = 0.0f;
             for(j = 0;j < 1;j++)
@@ -495,12 +496,26 @@ static ALvoid InitializeEffect(ALCcontext *Context, ALeffectslot *EffectSlot, AL
         if(EffectSlot->EffectState)
             ALEffect_Destroy(EffectSlot->EffectState);
         EffectSlot->EffectState = NewState;
+
+        if(!effect)
+            memset(&EffectSlot->effect, 0, sizeof(EffectSlot->effect));
+        else
+            memcpy(&EffectSlot->effect, effect, sizeof(*effect));
+        /* FIXME: This should be done asychronously, but since the EfefctState
+         * object was changed, it needs an update before its Process method can
+         * be called (coming changes may not guarantee an update when the
+         * NeedsUpdate flag is set). */
+        EffectSlot->NeedsUpdate = AL_FALSE;
+        ALEffect_Update(EffectSlot->EffectState, Context, &EffectSlot->effect);
     }
-    if(!effect)
-        memset(&EffectSlot->effect, 0, sizeof(EffectSlot->effect));
     else
-        memcpy(&EffectSlot->effect, effect, sizeof(*effect));
-    ALEffect_Update(EffectSlot->EffectState, Context, &EffectSlot->effect);
+    {
+        if(!effect)
+            memset(&EffectSlot->effect, 0, sizeof(EffectSlot->effect));
+        else
+            memcpy(&EffectSlot->effect, effect, sizeof(*effect));
+        EffectSlot->NeedsUpdate = AL_TRUE;
+    }
 }