[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