X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=TODO;h=218ca91a68841d75ee690b20fd5d188dbc0c3ed8;hb=c0a450c4d65d9164f9be806f6609b09e8d1180f2;hp=69ebc402368283dee98648031246c42f1618086b;hpb=b2c0107fed4a5ab70d1f8d44dd61c0450cbbbce6;p=open-adventure.git

diff --git a/TODO b/TODO
index 69ebc40..218ca91 100644
--- a/TODO
+++ b/TODO
@@ -1,7 +1,17 @@
 = Open Adventure TODO =
 
-* Make a full walkthrough.
+The FORTRANish mess that once was is now mostly idiomatic C. Some issues
+remain to be cleaned up:
 
-* Translate the FORTRANish mess to actual C.
+* 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.
+
+* The way return values from handlers such as 2012 trigger side effects
+  based on global state is nasty.