From: John L. <jla...@gm...> - 2006-06-08 14:39:51
|
On 6/8/06, klaas.holwerda <kla...@nl...> wrote: > Hi John, > > John Labenski wrote: > > On 6/7/06, klaas.holwerda <kla...@nl...> wrote: > > > >> Your idea to add precompiled headers like this does not work. Also > >> othere changed you said you made, made no difference. > >> > >> hook_cpp_binding_header_includes = "#include > >> \"general/include/a2dprec.h\"\n..#include \"wxbind/include/wxbind.h\"\n" > >> > >> I think the only thing that will be correct is to place the next at as > >> the first include file in *.cpp generated. > >> This is what the comment says, and how it worked before. > >> > > > > As I said, just reverting the change will break compilation on X > > Windows systems since the X windows headers very selfishly #define > > "Above" and "Below" to be numbers and you have to #undef them *after* > > including them. > > > After some searching i found those in wx_rules.lua. I first had no idea > what you were talking about :-) > But indeed how did the come up to use such defines?? > Should they also be placed in say wxluacan_rules.lua, in order to make > it work on Linux? > Sofar i did not see a problem on Linux RedHat, but on Suse there are. So > maybe this is what is the reason. > Lost my Suse install, so can't check it for the moment. You only need them if you have a function or something called Above(...) or Below(...). wxWidgets #undefs them in wx/layout.h since the layout constraints class has those functions. I've actually made the bindings #ifdef the layout constraints for < wxchkver26 since sizers have been around since 2.2 IIRC. So, no we don't need to add them elsewhere unless there is a compliation problem. -John |