Added two new platform macros GDK and CAIRO used in GraphicsContext
Created attachment 7901 [details] Additional Cairo and Gdk platform def
Created attachment 7902 [details] Patch using CAIRO PLATFORM Note if the Win32 build is changed to define BUILDING_CAIRO__ the extra test for WIN32 can be removed
Created attachment 7903 [details] use new CAIRO Platform
Comment on attachment 7902 [details] Patch using CAIRO PLATFORM There should never be a need to say "PLATFORM(CAIRO) || PLATFORM(WIN32)", so this is not correct. It should just be PLATFORM(CAIRO) and we should make sure it's set when building for Win32.
Comment on attachment 7901 [details] Additional Cairo and Gdk platform def This is wrong. We need to set KXMLCORE_PLATFORM_CAIRO for Win32 builds too, if it's going to exist. Also, graphics platform settings should go in a different section, not mixed in with MAC and WIN.
Comment on attachment 7903 [details] use new CAIRO Platform Looks good, but can't land this until we set CAIRO correctly (I review-'d the other related patches).
Comment on attachment 7903 [details] use new CAIRO Platform Oops, sorry, I take that back. The platform should be WIN_OS for this Win32 stuff. There is no PLATFORM(WIN32) and PLATFORM(WIN) is about the low level operating system rather than the high level operating environment (HDC and such).
Actually, PLATFORM(WIN) is the high level and PLATFORM(WIN_OS) is the low level. PLATFORM(WIN) would be right for these patches.
(In reply to comment #8) > Actually, PLATFORM(WIN) is the high level and PLATFORM(WIN_OS) is the low > level. PLATFORM(WIN) would be right for these patches. > If you can change the patch to match I'd appreciate it also as you note it needs to be tested under Windows which I'm not set up to do.
This was done long ago.