X-Git-Url: https://jxself.org/git/?p=open-adventure.git;a=blobdiff_plain;f=notes.adoc;h=75c2333375275de6032624803ed92f26757160c8;hp=00a733a0e3dd76a5ada810bbf12e3171ac30a773;hb=18a9be501ff8f0c57c5d540d0f5cd080c0438cd0;hpb=a0a35d8c7b8ed65230748647ff3c40e8119a643f diff --git a/notes.adoc b/notes.adoc index 00a733a..75c2333 100644 --- a/notes.adoc +++ b/notes.adoc @@ -85,7 +85,7 @@ 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 comprehensive test suite that we built first and verified -with coverage tools (we have 88% coverage, with the remaining 12% +with coverage tools (we have over 90% coverage, with the remaining confined to exception cases that are difficult to reach). This is what you are running when you do "make check". @@ -117,9 +117,9 @@ ways: 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. +* There are a few gotos left that resist restructuring; all are in the + principal command interpreter function implementing its state + machine. * Linked lists (for objects at a location) are implemented using an array of link indices. This is a surviving FORTRANism that is quite unlike @@ -127,9 +127,6 @@ ways: to fix it because doing so would (a) be quite difficult, and (b) compromise forward-portability to other languages. -* The code still has an unfortunately high density of magic numbers - in - particular, numeric object IDs. There are plans to fix this. - * Much of the code still uses FORTRAN-style uppercase names. * The code still assumes one-origin array indexing. Thus, arrays are