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