20 Sep
2007
20 Sep
'07
10:12 p.m.
I just tried building therion 0.5.1. The build fails against wx 2.8.0 The wx issue is that wx_SAVE, etc have been renamed to wx_FD_SAVE. I patched this as I did for Aven by using the new names, but defining these in terms of the old ones for wx < 2.7. There are obviously other approaches, but this way means that when you stop supporting wx 2.6 you can just drop the compatibility defines. Also, there are some warnings for passing size_t when the printf specifier expects an int (size_t is 64 bits on x86-64). That's undefined behaviour I believe. Patch attached to fix both of these (and also a typo in a comment). Cheers, Olly