Am Freitag, 1. September 2006 20:26 schrieb Christian Ohm:
> On Wednesday, 30 August 2006 at 19:31, Dennis Schridde wrote:
> > Add a --shadow toggle to commandline options to enable or disable
> > shadows. Patch by Suprano.
>
> Hmmm... This patch doesn't convince me for several reasons:
> - There is already the Alt-S shortcut to toggle shadows, that makes a
>   command line option quite pointless as far as I can see...
> - The format is quite non-standard, I'd prefer --shadows/--noshadows
> - The declarations for setDrawShadows() and setWarzoneKeyNumeric() are
>   added to clparse.c instead of being imported by their header files
> - The help output says "YES/NO" even though uppercase is used for
>   variable values
>
> I'd change at least the last two points (patch attached).
--(no)shadows
#include "...configfile.h"
//setDrawShadows()

Attachment: pgptlsKeObxCN.pgp
Description: PGP signature

_______________________________________________
Warzone-dev mailing list
[email protected]
https://mail.gna.org/listinfo/warzone-dev

Reply via email to