xaudio2: Don't render less than a period of audio.

Sometimes the ALSA driver will report a very small amount of frames available after a period signal. There is a bug in OpenAL 1.15 which will crash the application if alcRenderSamplesSOFT is called with less than 4 frames. And anyway, we shouldn't incur all of this overhead just to render a couple of frames. So, just skip the rendering step if we have less than a period of space available in the driver. Signed-off-by: Andrew Eikum <aeikum@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

xaudio2: Don't render less than a period of audio.
998e5a04 · Andrew Eikum · Alexandre Julliard · c5b54b34 · 998e5a04
Commit 998e5a04 authored Oct 21, 2015 by Andrew Eikum Committed by Alexandre Julliard Oct 22, 2015
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

xaudio_dll.c dlls/xaudio2_7/xaudio_dll.c +5 -1

No files found.
--- a/dlls/xaudio2_7/xaudio_dll.c
+++ b/dlls/xaudio2_7/xaudio_dll.c
@@ -2723,7 +2723,11 @@ static void do_engine_tick(IXAudio2Impl *This)
    }

    nframes = This->period_frames * 3 - pad;
-    TRACE("going to render %u frames\n", nframes);
+
+    TRACE("frames available: %u\n", nframes);
+
+    if(nframes < This->period_frames)
+        return;

    if(!nframes)
        return;