Compilation broken in CVS

Jeremy Shaw jeremy.shaw at lindows.com
Fri Jan 9 13:26:19 CST 2004


Ack!

I did test my patch before sending in it, but I am a doofus. I made
the changes in one copy of the source tree, but compiled and tested a
different copy of the source tree. So I didn't actually test the patch
at all!!

grr!

Here is a patch to fix things. You should use this patch instead of
Marcus' if you actually want to use the winearts driver because in
addition to the typo, I didn't realize at first that I needed to
assign the return value of HeapReAlloc() back to
wwo->sound_buffer. (But it's rather obvious once I realized it)

--- orig/dlls/winmm/winearts/audio.c
+++ mod/dlls/winmm/winearts/audio.c
@@ -719,7 +719,7 @@
     {
       if(wwo->sound_buffer)
       {
-	HeapReAlloc(GetProcessHeap(), 0, wwo->sound_buffer, toWrite);
+	wwo->sound_buffer = HeapReAlloc(GetProcessHeap(), 0, wwo->sound_buffer, toWrite);
 	wwo->buffer_size = toWrite;
       }
     }

At Fri, 09 Jan 2004 15:55:47 +0000,
Mike Hearn wrote:
> 
> Hi Paul,
> 
> Marcus submitted a patch for this. Should be fixed soon, it's just a silly
> typo. That tinderbox type thing is realy cool by the way! :)
> 
> 



More information about the wine-devel mailing list