X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=TODO;h=f8dc544d5af8234867f339f6dc653e322bfb1b38;hb=0e160a665eeb24d4614ff14e779d1c13bec3e778;hp=3e7758d1e1c3e04b4051ee008af8e27b9db606c5;hpb=0599a20f1f56d045d694dab6fba0553799214663;p=open-adventure.git

diff --git a/TODO b/TODO
index 3e7758d..f8dc544 100644
--- a/TODO
+++ b/TODO
@@ -1,13 +1,15 @@
 = Open Adventure TODO =
 
-* Use a real pseudorandom-number generator with a seed rather than just
-  time-sampling.
+The FORTRANish mess that once was is now mostly idiomatic C. Some issues
+remain to be cleaned up:
 
-* Add command log replay.  Note that the replay log needs to begin
-  with the random-number seed.
+* The remaining shared globals: K, SPK, WD1, WD1X, WD2, WD2X. These are used
+  rather promiscuously to pass around information that ought to be function
+  arguments in a modern language. K and SPK are particular offenders
 
-* Use that feature to make regression tests from walkthroughs.
+* Remaining unstructured gotos in playermove() and do_command().
+
+* The way return values from handlers such as 2012 trigger side effects
+  based on global state such as K and SPK is nasty.
 
-* Translate the FORTRANish mess to actual C.
 
-* Inline the database so the code doesn't need an external file.