Update TODO.

[open-adventure.git] / TODO

diff --git a/TODO b/TODO
index 8db67c45d6b91a694d4a83b0dd443686a75f31b8..6abcb59866ad0c68ac07bac9f51b669acfa3d59f 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,5 +1,21 @@
 = Open Adventure TODO =
 
-* Translate the FORTRANish mess to actual C.
+The FORTRANish mess that once was is now mostly idiomatic C. Some issues
+remain to be cleaned up:
+
+* The remaining shared globals: K, WD1, WD1X, WD2, WD2X. These are used
+  rather promiscuously to pass around information that ought to be function
+  arguments in a modern language. K is a particular offender.
+
+* Remaining unstructured gotos in playermove() and do_command(). The goto L12
+  in playermove() is particularly horrible, jumping backwards into the
+  middle of a loop.
+
+* In playermove(), why doesn't replacing the global motion number K
+  with a passed-in argument work? There's some unobvious information
+  leak here.
+
+* The way return values from handlers such as 2012 trigger side effects
+  based on global state is nasty.