[Scummvm-cvs-logs] CVS: scummvm/base gameDetector.cpp,1.19,1.20
Max Horn
fingolfin at users.sourceforge.net
Fri Oct 10 04:16:05 CEST 2003
Update of /cvsroot/scummvm/scummvm/base
In directory sc8-pr-cvs1:/tmp/cvs-serv31269/base
Modified Files:
gameDetector.cpp
Log Message:
added my thoughts on our command line options (please feel free to comment and tell me what you thinkg about it!); fixed program exit code for -z
Index: gameDetector.cpp
===================================================================
RCS file: /cvsroot/scummvm/scummvm/base/gameDetector.cpp,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -d -r1.19 -r1.20
--- gameDetector.cpp 9 Oct 2003 14:05:09 -0000 1.19
+++ gameDetector.cpp 10 Oct 2003 11:15:14 -0000 1.20
@@ -48,6 +48,17 @@
current_option = NULL
// DONT FIXME: DO NOT ORDER ALPHABETICALY, THIS IS ORDERED BY IMPORTANCE/CATEGORY! :)
+// FIXME: Reconsider and reevaluate our command line options. I propose some changes:
+// * add long alternatives for (almost?) all options. There should be some generic way
+// to specify this, in order to avoid code duplication and to simplify the code
+// * some more obscure options which currently have a single-letter options assigned
+// really don't need it. E.g. the "-t<NUM>" option probably is rarely used, and anybody
+// who needs it should still be happy with "--tempo=NUM"
+// This will make room in the ranks of single-letter options for other more useful options
+// (for example, "-t" could stand for "List (T)argets")
+// * -y, -t, -c, -j all are candidates for this, IMHO: their settings are rather obscure,
+// and anybody who needs them always should put them into the config file anyway
+//
#ifdef __PALM_OS__
static const char USAGE_STRING[] = "NoUsageString"; // save more data segment space
#else
@@ -427,7 +438,7 @@
case 'z':
CHECK_OPTION();
list_games();
- exit(1);
+ exit(0);
case '-':
// Long options. Let the fun begin!
if (!strncmp(s, "platform=", 9)) {
More information about the Scummvm-git-logs
mailing list