Extend test coverage. read sign in endgame.

Elimination of OBJSND.

Extend test coverage of vending-machine actions.

Improve test coverage.

Eliminate setting of OBJTXT sell for oyster - OBJTXT is now static.

A necessary step towards getting rid of it.

Abolish unclean operation on OBJSOUND - it's now static.

Save/Restore no longer needs to treat it specially.

Name the lamp, grate, and bird states.

Proof of cobcept for state defines.

It is now possible in the YAML to declare defines for all state values
associated with an object.  This are visible in the C code and can be used
to replace magic numbers.

Documentation polishing.

Support loud locations.

Location sounds are now declared by YAML.

objsound.py: add a (temporary) YAML transformer for translating Section 13.

This will go away after it's applied/

Add YAML markup for location sounds.

Toss unused parts of dungeon.c and init.c

Replace macros with condition bits.

Give the FORCED property a real cond bit.  Delete some unused code.

Switch fully over to YAML generation of condition bits. COND is gone.

Generate initializer from YAML for Section 9 bits.

Use YAML references to tie hints to locations.

This change makes locbit.py obsolete; it's deleted.

Fixups to the adventure.yaml commentary.

Location YAML now has conditions.

COND initionalization doesn't use these yet. That's the next step.

locbit.py: Can't set a FORCED attribute in the YAML...

...it has to be computed from the travel arrays.

Repair a check file.

Check in the tool for folding Section 9 COND bits into YAML.

Needs to be in the repo history for reference, though it won't
be manifest after the conversion.

We shouldn't actually apply this until the murk around FORCED has been
dispelled.

Improve namespace hygiene.

Patch around non-working pillo bug fix.

Quick-fix the 'pillo bug'.

Once parsing no longer depends on text being broken up into
5-char tokens, the ugly hack to adventure.yaml must go away.

Simple fix for GitLab issue #14: Behaviour of magic words changed

Say "Nothing happens" on "foo" not in Giant Room, rather than "What's
the matter, can't you read?  Now you'd best start over." which only
makes sense there.

It would be funnier to say something like "Well, that was remarkably
pointless." Maybe later.

Fix GitLab issue #13 - The "eat" command exhibits strange behavior

The problem wasn't actually with "eat" but with "grate".  Processing
of any two-word command with an object of "grate", when given at certain
locations including the start, inappropriately jumped directly
to the movement code (ignoring the verb).

There was a similar bug in the ancestral 2.5, though it manifested slightly
differently.

Revert "Test coverage - Almost all of score.c".  It's breaking tests.

Revert "Test coverage - 100% score.c". It's breaking tests.

Test coverage - 100% score.c

Test coverage - Almost all of score.c

xstrdup() not called anywhere in source. Removed.

Switch to 0-indexing for hint handling code.

Abolish HNTMAX and HNTSIZ in favor of HINT_COUNT.

This change necessitated include guards on newdb.h.

Replace SETPRM/[PR]SPEAK with variadic [pr]speak

Rename/rewrite old speak to vspeak and take a va_list
Add new speak that takes variadic parameters

Remove SETPRM & PARMS[]

Resolve GitLab issue #3.  Applies MISRA and CERT guidelines.

MISRA C++:2008, 2-13-4 - Literal suffixes shall be upper case

MISRA C:2012, 7.3 - The lowercase character "l" shall not be used in a
literal suffix

CERT DCL16-C. - Use "L," not "l," to indicate a long value

CERT DCL16-CPP. - Use "L," not "l," to indicate a long value

CERT, DCL50-J. - Use visually distinct identifiers

Test Coverage -- lots of new pieces of actions.c

Documentation polishing.

Documentation and polishing.

YAMLify section 11 (hints).

Document the YAML, remove some dead code, fix typos.

Section 10 (class thresholds) is now parsed entirely from YAML.

Further code simplification.

Code simplification.

Chage read handler argument to by-value.

This (sort of) documents the fact that, unlike say() and attack(),
read() never scribbles on the interpreter's command block.

What those other functions are doing is unclean - though less so than
fully exposed globals. I don't see a way to fix it that isn't pretty
intrusive, alas.

WD* globals abolished. They're now members of the command block.

Fix another ob1 error.

Clean out some disused variables.

Fix off-by-one error.

Refactor how turn threshold penalties are accounted for.

Test coverage -- more corner cases in actions.c

Introduce command encapsulation structure.

Refactored dungeon generator.

Clean up adventure.yaml.

Don't exit on EOF from get_input().

Test coverage - more corner cases in actions.c

Comment polishing.

Add test for "cage bird".

Polish help text.

Add -l to coverage testing.

Documentation update.

Fix off-by-one error that enabled the fuzzer to find a crash hole.

One has to allocate space for the trailing NUL, too.

Test logic for lamp dimming triggering battery replacement.

Merge branch 'master' into magic-number

Fix typo and compiler warning.

Add token_to_packed() utility function.

Merge branch 'master' into magic-number

Cleanup Arithmetic on message number

Test coverage More misuse of transitive verbs

Test coverage - failure to fill

Test coverage -- intransitive carry when one object is present

Test coverage - drink with no valid object

Mostly confine assumptions about what token_t is to misc.c

The token_t things like WD* are presently longs and will someday be
char[6].  By introducing some trivial functions - wordeq(),
wordempty(), and wordclear() - we mostly hide the difference.

All runtime knowledge about packing now lives only in misc.c and the
list of magic WORD_* defines in advent.h.  Outside this, literals are
now accessed through #define names that could expand to either longs
or strings.

Still to be done: WD* values are sometiimes compated to zero in
ways implying they can be negative. Must figure out wat thus means.

Concentrate all magic-number values for packed-string constants in one place.

More semantic typing - a step towards eliminating packing.

Really 100% test coverage for saveresume.c

Should have 100% coverage on saveresume.c now

Publish more things on the website.

Boolify some logical variables.

Even less magic

Save/resume fail test coverage -- test works in Docker now

-r doesn't show with NOSAVE build

No need to tease with excluded options

Revert "Improved test coverage -- save/resume fail"

Fails in GitLab CI - looks like /dev/badfilename can actually be opened there.

Improved test coverage -- save/resume fail

Magic-number elimination.

Magic-number elimination. Fix possible glitch in troll-bridge death.

Add axeorama log, extending test coverage.

Restructure some code.

L12 is gone again. Magic numbers have been almost eliminated.

Magic number elimination, repair some careless replace damage.

Documentation tweak.

Remove goto L12 - take two

On't know if you let me near this code again but here is a second
stab at removing "goto L12"

Previous attempt failed because of trying to continue the outer
do{}while(false)
Adding an extra for(;;) loop where the only repeat is the former
"goto L12" is replaced by continue. All other exits are returns
and a break;

Improve test coverage.

Reorganize debug build options.

* 'make advent' by itself can be considered the 'release' build.
* 'make debug' now does 'make linty'.

Get usage message into coverage tests.

Abolish MAXDIE in favor of maximum_deaths.

Clean old obit messages from adventure.yaml.