X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=TODO;h=6abcb59866ad0c68ac07bac9f51b669acfa3d59f;hb=04db8370752c6600b05d2059f8a15d1340d380fd;hp=37857feb7015fb6501f11ad3eb43fb59b26389d1;hpb=55610bb1de8f861af7567d8b6d8ab47006148fce;p=open-adventure.git diff --git a/TODO b/TODO index 37857fe..6abcb59 100644 --- a/TODO +++ b/TODO @@ -3,9 +3,9 @@ The FORTRANish mess that once was is now mostly idiomatic C. Some issues remain to be cleaned up: -* The remaining shared globals: K, SPK, WD1, WD1X, WD2, WD2X. These are used +* 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 and SPK are particular offenders + 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 @@ -16,6 +16,6 @@ remain to be cleaned up: leak here. * The way return values from handlers such as 2012 trigger side effects - based on global state such as K and SPK is nasty. + based on global state is nasty.