[SDL] New SDL app : LaserBoy Multiplatform
Ulrich Eckhardt
doomster at knuut.de
Mon Apr 28 02:49:23 PDT 2008
On Monday 28 April 2008 08:55:35 another james wrote:
> Check it out!
>
> http://akrobiz.com/laserboy/code/LaserBoy_04_27_2008.zip
>
> Let me know what you think!
Just downloaded it and started browsing the content of the zipfile, and here
are a few things I noticed:
1. It includes unnecessary stuff, in particular win32 executables and empty
directories.
2. __BMP_FUNCTIONS__: you must not use names containing two consecutive
underscores or beginning with an underscore followed by an uppercase. Those
are simply reserved and IMHO they don't add anything anyway.
3. 'extern' doesn't have any meaning for function declarations.
4. Functions must not be followed by a trailing semicolon after the closing
curly brackets. Inside a function, it is called an empty statement, but at
namespace scope (outside a function), empty statements are not allowed. Note
that they are actually allowed after inline memberfunction definitions,
though they are neither required nor do they make sense there.
Other than that, I'll try to compile it on a PPC platform, so I'll be able to
tell you if it's endian-clean code later.
cheers
Uli
More information about the SDL
mailing list