X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=TODO;h=62eef63e12afc119442a1d7813af9bfe4bd8127e;hb=696779d3377edf363e8d829c9931f9e2330b9128;hp=4c7ab1bb9e628a08bd5e8f187b6217308efc8953;hpb=8e86d0dca3e10ec1ed9f205b9e8222f6555612ad;p=open-adventure.git

diff --git a/TODO b/TODO
index 4c7ab1b..62eef63 100644
--- a/TODO
+++ b/TODO
@@ -1,9 +1,17 @@
 = Open Adventure TODO =
 
-* Add command logging and command log replay.
+The FORTRANish mess that once was is now mostly idiomatic C. Some issues
+remain to be cleaned up:
 
-* Use that feature to make regression tests from walkthroughs.
+* The remaining shared globals: WD1, WD1X, WD2, WD2X. These are used
+  rather promiscuously to pass around information that ought to be function
+  arguments in a modern language.
+
+* Remaining unstructured gotos in playermove() and do_command(). The goto L12
+  in playermove() is particularly horrible, jumping backwards into the
+  middle of a loop.
+
+* The way return values from handlers such as 2012 trigger side effects
+  based on global state is nasty.
 
-* Translate the FORTRANish mess to actual C.
 
-* Inline the database so the code doesn't need an external file.