1 = Open Adventure TODO =
3 The FORTRANish mess that once was is now mostly idiomatic C. Some issues
4 remain to be cleaned up:
6 * The remaining shared globals: K, SPK, WD1, WD1X, WD2, WD2X. These are used
7 rather promiscuously to pass around information that ought to be function
8 arguments in a modern language. K and SPK are particular offenders
10 * Remaining unstructured gotos in playermove() and do_command(). The goto L12
11 in playermove() is particularly horrible, jumping backwards into the
14 * In playermove(), why doesn't replacing the global motion number K
15 with a passed-in argument work? There's some unobvious information
18 * The way return values from handlers such as 2012 trigger side effects
19 based on global state such as K and SPK is nasty.