Am Sonntag, 3. September 2006 15:06 schrieb Dennis Schridde:
> 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()
PS: In r322

--Dennis

Attachment: pgpyAqJQdZBwh.pgp
Description: PGP signature

_______________________________________________
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev

Reply via email to