X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=notes.adoc;h=cc4246c440abadd0b4b0605988261214a9e2ba3a;hb=da883d53c09875635a9be1730371bebdaf9ef262;hp=e92d9407914844e1bc2e9852e00419befc8c51a9;hpb=7018354e228c7a25cc77257edb20305c79973ae4;p=open-adventure.git diff --git a/notes.adoc b/notes.adoc index e92d940..cc4246c 100644 --- a/notes.adoc +++ b/notes.adoc @@ -10,7 +10,7 @@ separate link:history.html[history] describing how it came to us. The principal maintainers of this code are Eric S. Raymond and Jason Ninneman. Eric received Don Woods's encouragement to update and ship the game; Jason signed on early in the process to help. The assistance -of Peje Nilson in restructuring some particularly grotty gotos is +of Peje Nilsson in restructuring some particularly grotty gotos is gratefully acknowledged. == Nomenclature == @@ -62,7 +62,9 @@ a "seed" command) will replay reliably, including random events. The adventure.text file is no longer required at runtime. Instead, it is compiled at build time to a source module containing C structures, -which is then linked to the advent binary. +which is then linked to the advent binary. There is an adventure.yaml file +as well; this is also compiled to C code, and will eventually replace +adventure.text altogether. The game-save format has changed. This was done to simplify the FORTRAN-derived code that formerly implemented the save/restore @@ -79,8 +81,10 @@ ugly and quite unreadable. Jason Ninneman and I have moved it to what is almost, but not quite, idiomatic modern C. We refactored the right way, checking correctness -against a comprehesive test suite that we built first and verified with -coverage tools. This is what you are running when you do "make check". +against a comprehensive test suite that we built first and verified +with coverage tools (we have 88% coverage, with the remaining 12% +confined to exception cases that are difficult to reach). This is +what you are running when you do "make check". This move entailed some structural changes. The most important was the refactoring of over 350 gotos into if/loop/break structures. We @@ -109,7 +113,6 @@ ways: We don't need whatever minor performance gains this might collect, and the choice to refrain will make forward translation into future languages easier. - * There are a few gotos left that resist restructuring; all of these are in the principal command interpreter function implementing its state machine.