From: shelarcy <she...@gm...> - 2008-04-10 17:48:56
|
Hi Jeremy, On Thu, 10 Apr 2008 23:12:50 +0900, Jeremy O'Donoghue <je...@o-...> wrote: > Attached is a (zipped) preliminary set of patches for wxWidgets 2.8.7 > support. Some notes to help. > > - wxWidgets 2.8.7 was compiled from source using Visual Studio 2008 I tested your patch on PowerPC Mac OS X. And I found below part causes compile error. > [Preliminary wxWidgets 2.8.7 support > jer...@gm...**20080410112559] { > (snip) > hunk ./wxc/src/eljvalidator.cpp 136 > (snip) > +#ifdef wxUSE_UNICODE > + items_copy[i] = _wcsdup(items.Item(i).GetData()); > +#else > + items_copy[i] = strdup(items.Item(i).GetData()); > +#endif _wcsdup is Visual Studio (2005 or higher)'s specific function. And wxchar has portable wxStrdup function. So, I think we should use wxStrdup function instead. > - Compiled on top of both 0.10.3 rc1 (tagged) and the released version > (head). Tested > against most of the samples/wx directory using rc1 (Windows only). I checked those are also working on PowerPC Mac OS X platform. Some sample cause problem. But anyway we have working code now. That is sounds great! > The general approach I've taken, as agreed with Eric when I discussed > this patch, is to shoot for wxcore API which is as close as possible to > wxWidgets 2.8. This breaks source compatibility in places, but has some > advantages: I don't check that yet. I'll check that this and/or next week. Best Regards, -- shelarcy <shelarcy hotmail.co.jp> http://page.freett.com/shelarcy/ |