Shouldn't this just be removed?

[Self_Destructo]

In armagetronad\src\defs.h I get an error: src/defs.h:105: error: new declaration `bool isblank(int)'. This is no surprise since it was expected (http://wiki.armagetronad.net/index.php? ... leshooting). But, the code seems completely dumb because we just simply comment it out and move on. Therefore, if it's only for WIN32 and the function doesn't exist in Windows it should be removed, right?

Code: Select all

#ifdef WIN32
// this function does not seem to exist in windows...
inline bool isblank(int x){ return ((x == ' ') || (x == '\t')); }
#endif

[Z-Man]

To quote the wiki, "you may have...". What compilation environment are you using? I've never had an issue with this line with all the versions of Visual Studio (6 and 2003 Express) and Code::Blocks (latest RC) I tested, and in fact it would not compile without this line (at least with VS6 where the line originates from), so it can't be removed. What is needed instead is a better #ifdef statement that recognizes the situations where the function already exists. Any ideas?

[Self_Destructo]

I'm using the latest Code Blocks. I'm not a programmer so I wouldn't have an idea as to what to do.