X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=TODO;h=f8dc544d5af8234867f339f6dc653e322bfb1b38;hb=0e160a665eeb24d4614ff14e779d1c13bec3e778;hp=4c7ab1bb9e628a08bd5e8f187b6217308efc8953;hpb=8e86d0dca3e10ec1ed9f205b9e8222f6555612ad;p=open-adventure.git diff --git a/TODO b/TODO index 4c7ab1b..f8dc544 100644 --- a/TODO +++ b/TODO @@ -1,9 +1,15 @@ = 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: 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 + +* 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.