re: [PATCH] oleaut32: Fix valgrind warnings in vatype.c test

2008-06-05 Thread Jon Griffiths
Hi Dan, The last part of your patch, having to do with SysAllocStringByteLen(), seems correct to me, offhand. Maybe if you clean up the comment a bit more, sprinkle sugar on it, and resend, it'll make it in this time. I'll polish it and fire in again next week, there doesn't seem much point

re: [PATCH] oleaut32: Fix valgrind warnings in vatype.c test

2008-06-04 Thread Dan Kegel
Hi Jon, I guess http://www.winehq.org/pipermail/wine-patches/2008-May/055352.html must have been too scary to commit. I independently sent in a fix for the easy part, and it was committed as http://winehq.org/pipermail/wine-cvs/2008-June/043999.html The last part of your patch, having to do with