X-Git-Url: https://jxself.org/git/?p=open-adventure.git;a=blobdiff_plain;f=TODO;h=188da49212512a4d920be5a560034df9b720bf13;hp=37857feb7015fb6501f11ad3eb43fb59b26389d1;hb=da6a090b833070da1726a92d5db3b476c2313050;hpb=55610bb1de8f861af7567d8b6d8ab47006148fce diff --git a/TODO b/TODO index 37857fe..188da49 100644 --- a/TODO +++ b/TODO @@ -3,19 +3,12 @@ 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 - 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 goto L12 - in playermove() is particularly horrible, jumping backwards into the - middle of a loop. - -* In playermove(), why doesn't replacing the global motion number K - with a passed-in argument work? There's some unobvious information - 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. +* Remaining unstructured gotos in do_command(). +* The program is still pretty much typeless - full of magic numbers being + sliced and diced in cryptic ways. Some attempt has been made to introduce + semantic types, but the job is barely started. +We are aware that the lamp currently used as the project logo is a +hurricane lamp, not a proper miner's lamp. Submissions of potential +replacement images would be welcome.