projects
/
open-adventure.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Use a header/source set common to both advent and the dungeon builder.
[open-adventure.git]
/
TODO
diff --git
a/TODO
b/TODO
index 3e7758d1e1c3e04b4051ee008af8e27b9db606c5..f8dc544d5af8234867f339f6dc653e322bfb1b38 100644
(file)
--- a/
TODO
+++ b/
TODO
@@
-1,13
+1,15
@@
= Open Adventure TODO =
= Open Adventure TODO =
-* Use a real pseudorandom-number generator with a seed rather than just
- time-sampling.
+The FORTRANish mess that once was is now mostly idiomatic C. Some issues
+remain to be cleaned up:
-* Add command log replay. Note that the replay log needs to begin
- with the random-number seed.
+* 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
-* Use that feature to make regression tests from walkthroughs.
+* Remaining unstructured gotos in playermove() and do_command().
+
+* The way return values from handlers such as 2012 trigger side effects
+ based on global state such as K and SPK is nasty.
-* Translate the FORTRANish mess to actual C.
-* Inline the database so the code doesn't need an external file.