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: 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.
-* 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.