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.

Remove  message pointer math in croak().

Change YES() to take const char* arguments.

Give obituary messages their own data structure.

Typo fix.

More repairs on search-replace results.

Fix search-and-replace error.

No longer restricted to 6-character FORTRAN identifiers.

Attempt to deconfuse the coverage analyzer.

Reindent and cleanup.

Add project logo. It'll have to do until we find an image of a miner's lamp.

magic numbers, show usage, fixed linty warnings

Show usage when using bad parameter with ./advent
Converted magic numbers to enums for BUG().  Also bug now shows
stringify'ed version of bug enumeration (not just a number).

Fix bug that made YES() case-sensitive.

Also fix a related memory leak.

Gut and rebuild YES() with cleaner approach that doesn't rely on packing.

The new support functions get_input() and echo_input() (and others not made yet) will eventually replace GETIN() and MAPLIN().

This should get test coverage to 93%.

Improve test coverage.

Improve test coverage.

Improve test coverage

Repair plover teleport and add a test for it.

Extent test coverage to plover room access.

Code simplification.

Remove unused code.

Rename newspeak() to speak().

Refactor scoring so score() does not conditionally exit.

Improve test coverage.

Improve test coverage.

Improve test coverage.

Improve test coverage.

Improve test coverage.