aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--TODO1
-rw-r--r--base/gameDetector.cpp7
2 files changed, 0 insertions, 8 deletions
diff --git a/TODO b/TODO
index 07ee5a77c9..df77228498 100644
--- a/TODO
+++ b/TODO
@@ -7,7 +7,6 @@ General
we set when the application should be quit (e.g. when an EVENT_QUIT is
received). This is useful if multiple levels of event loops have to be ended
* Fix the Map<> template, make it more robust; maybe use a red-black tree?
-* Add iterators to List<> template and make use of them
* Allow for return-to-launcher instead of a normal "quit" ?
* Improve the argv (command line args) parser
* Extend the Plugin API to provide for "game detection": instead of the
diff --git a/base/gameDetector.cpp b/base/gameDetector.cpp
index b2447f1c18..062a17f7d1 100644
--- a/base/gameDetector.cpp
+++ b/base/gameDetector.cpp
@@ -259,13 +259,6 @@ void GameDetector::parseCommandLine(int argc, char **argv) {
bool long_option_value;
// Iterate over all comman line arguments, backwards.
- // FIXME: Looping backwards has a major problem: Consider this example
- // invocation: "scummvm -g 1x". It should work exactly like "scummvm -g1x"
- // but it doesn't! Instead of starting the launcher with the 1x sacler
- // in effect, it will give an error about target 1x being unknown.
- // This can be fixed by forward iterating the args. Of course doing that
- // essentially means we have to rewrite the whole command line parser,
- // but that seems like a good idea anyway.
for (i = argc - 1; i >= 1; i--) {
s = argv[i];