projects
/
open-adventure.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Eliminate L8 label.
[open-adventure.git]
/
TODO
diff --git
a/TODO
b/TODO
index 6abcb59866ad0c68ac07bac9f51b669acfa3d59f..62eef63e12afc119442a1d7813af9bfe4bd8127e 100644
(file)
--- a/
TODO
+++ b/
TODO
@@
-3,18
+3,14
@@
The FORTRANish mess that once was is now mostly idiomatic C. Some issues
remain to be cleaned up:
The FORTRANish mess that once was is now mostly idiomatic C. Some issues
remain to be cleaned up:
-* The remaining shared globals:
K,
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
rather promiscuously to pass around information that ought to be function
- arguments in a modern language.
K is a particular offender.
+ 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.
* 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 is nasty.
* The way return values from handlers such as 2012 trigger side effects
based on global state is nasty.