projects
/
open-adventure.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (parent:
0dcd872
)
Questions for the future.
author
Eric S. Raymond
<esr@thyrsus.com>
Sun, 11 Jun 2017 13:24:52 +0000
(09:24 -0400)
committer
Eric S. Raymond
<esr@thyrsus.com>
Sun, 11 Jun 2017 13:24:52 +0000
(09:24 -0400)
TODO
patch
|
blob
|
history
diff --git
a/TODO
b/TODO
index f8dc544d5af8234867f339f6dc653e322bfb1b38..37857feb7015fb6501f11ad3eb43fb59b26389d1 100644
(file)
--- a/
TODO
+++ b/
TODO
@@
-7,7
+7,13
@@
remain to be cleaned up:
rather promiscuously to pass around information that ought to be function
arguments in a modern language. K and SPK are particular offenders
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().
+* 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.
* The way return values from handlers such as 2012 trigger side effects
based on global state such as K and SPK is nasty.