Compilation broken in CVS

Jeremy Shaw jeremy.shaw at lindows.com
Fri Jan 9 13:29:46 CST 2004


Gah!

Correction, you should use it in addition to Marcus' patch.

Jeremy, airhead, Shaw.

At Fri, 09 Jan 2004 11:26:19 -0800,
Jeremy Shaw wrote:
> 
> 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