[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