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! :)