[PATCH] winealsa: Silence the buffer warning more often

Maarten Lankhorst m.b.lankhorst at gmail.com
Fri Jul 6 11:12:56 CDT 2007


only print a FIXME once if psize == 1024 (dmix default), but always a
warn in other cases.
-------------- next part --------------
>From b0c20cd73c90588ad3851c22592c1e1b3b4aafab Mon Sep 17 00:00:00 2001
From: Maarten Lankhorst <m.b.lankhorst at gmail.com>
Date: Thu, 5 Jul 2007 15:34:11 +0200
Subject: [PATCH] winealsa: Silence the buffer warning more often

---
 dlls/winealsa.drv/dsoutput.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/winealsa.drv/dsoutput.c b/dlls/winealsa.drv/dsoutput.c
index c5ad141..7990a0f 100644
--- a/dlls/winealsa.drv/dsoutput.c
+++ b/dlls/winealsa.drv/dsoutput.c
@@ -388,8 +388,10 @@ static HRESULT SetFormat(IDsDriverBufferImpl *This, LPWAVEFORMATEX pwfx, BOOL fo
      */
     if (psize >= 512)
     {
-        if (psize > 512 && ++warnonce == 1)
-            FIXME("Your alsa dmix period size is excessively high, unfortunately this is alsa default, try decreasing it to 512 or 256 (but double the amount of periods) if possible\n");
+        if (psize == 1024 && ++warnonce == 1)
+            FIXME("Your alsa dmix period size is 1024, try decreasing it to 512 if possible\n");
+        else if (psize > 512)
+            WARN("Your alsa period size is excessively high (%lu)\n", psize);
         This->mmap_commitahead = 2 * psize + psize/2;
         This->mmap_writeahead = 2 * psize;
     }
-- 
1.4.4.2



More information about the wine-patches mailing list