Test coverage - add lots more coverage from actions.c

Not the best idea to try to rewach Don Woods for help any more.

Typo fix.

Add Aaron Traas to credits.

Start another round of magic_number removal, this time for motions.

Documentation and comment cleanup.

Also, renove uused enums.

'd' had a collision; 'drop' vs ;down'. Remove from 'drop' synonyms.

Documentation polishing and minor test improvements.

Breaking one-line conditionals

Re-indented.

Clean out the now-disused vocabulary structure.

Updated documentation

Cleanup test case

Coverage - Die while closing after lamp dim

k2 is dead

Untangling variable reuse

Separate debugging of linenoise

Fix an OB1 bug in allocation.

Unshadowed k2 into k3 in main.c
Reindented everything

Upstream linenoise rebased their project.

Transition to YAML is done, adventure.text and travel.py can go away.

Comment polishing.

Finish newdb -> dungeon renaming.

newdungeon.py is now make_dungeon.py.

newdb.[ch] is now dungeon.[ch].

Remove MAPLIN(). Fold common.[ch] into other files and remove them.

Jettison MAKEWD(), GETTXT(), vocab(), GETIN(), and the old db compiler.

Coverage - Test going back when you can't.

Fixed automated testing

Add make target for coverage

Simplify conditional evaluation when processing travel opcodes.

Address GitLab issue #16: `make dist` produces incomplete tarball

Completely wire vocab words in YAML to the code.

This massive patch:
* Finishes working all the vocab words into YAML structures.
* Adds vocab ID generator functions.
* Redoes the input-getting system, removing the need for GETIN(),
  GETTXT(), etc.
* Changes advent<->ascii mapping to avoid the special 'shift'
  character.
* Works around some bad behavior in the dragon attack logic.
* Handles the reservoir magic word without changing the database
  contents.

Eliminate some promiscuous variable reuse.

This was making the opcode-conditional evaluation logic much more
difficult to read than it needed to be.

Improve documentation if the *most* obscure span of code.

Make clean now removes cheat binary

More magic-number removal.

Magic-number elimination.

Implement and document state-changes messages in YAML.

Examples at the lamp and grate.  This is a way to pull strings out of
arbitrary_messages and make them part of the object definition.

Some more linter warnings

Removed advent.info and added to .gitignore

advent.info is a generated file. No need to keep it in the repo.

Test coverage: make LCOV ignore code unreachable without interactive shell

Test coverage: make LCOV ignore bug() and all calls to BUG()

In theory, these should be unreachable, so they shouldn't be covered
with testing.

Test coverage: make LCOV ignore sig_handler and OOM check

More test repairing.

Repair test breakage from previous commit.

Test consolidation.

Merge two tests so the suite can run a little faster.

Update linenoise.

Declare one-letter synonyms for some commands.

As issue #6 (Add single-letter verb synonyms) says:

Modern interactive fiction has converged on a fairly standard
vocabulary of one-letter abbreviations for common verbs. Besides
directional commands, they are:

d - drop
g - get
i - inventory
l - look
x - examine (synonymous in ADVENT with look)
z - wait

Open Adventure should implement these.

This commit doesn't make them work yet, but it declares them for when the
lexical analyzer is fully YAML-driven.

Magic-number elimination.

Cleaned up tests/Makefile - added save generation to its own target

Test coverage: command-line options for advent

Test coverage -- 100% cheat for real this time

Comment polishing.

De-macroize references to travel opcode fields we won't unpack further.

Break travel array into three struct fields.

More macro abstraction of the travel opcodes.

After splitting out the stop field, nothing ever negates a travel opcode...

...so all those labs() calls can go away.

Break travel opcodes into a two-element structure.

All travel[] references are now wrapped in extractor/predicate macros.

This is a step toward pulling the magic numbers apart into data structures
so there will be fewer magic numbers in the code.

Begin abstracting out the magic properties of travel-array opcodes.

Remove unused enums.

More use of state labels.

Minor refactoring step.

More use of state labels.

Allow state labels in not clauses.

Test Coverage - elicit hints for grate and snake

Remove unused code

Test coverage -- additional coverage in actions.c

buildregress now generates save files correctly.

100% coverage of cheat.c

Allow cheat to generate save files with bogus numbers.

Also remove pre-built save file for resumefail.log

"cheat" now has command-line arguments for generating cheat files

-d number of deaths. Signed integer value
-s number of saves. Signed integer value
-o file name of save game to write

Travel tables are all done from YAML now.

Leaves only Section 4 from asventure.text still relevant.

This looks like it gets the terminators right.

travel is correct except for position of terminators.

Documentation polishing.

Move the travel rule documentation to newdungeon.py.

Update linenoise from upstream.

Abolish VOCWRD().

Action word mnemonics are now enums from adventure.yaml.

Remove actspk[].

Clean up missed actspk[] references.

Use actions[].message instead of actspk[].

Add missing declaration.

Express actions[] in newdb.[ch].

Combine actspk and the action words into a single structure.

This version of newdungeon.py can exactly recover Section 3...

...from the YAML data. Next, to pack the recovered data the way
dungeon.c does and wite it in.

Reexpress the motion words in adventure.yaml.

First cut at compiling travel rules from YAML.

Not fully wired in yet.

Repair variois problems in travel YAML.

Repair a missing travel element it YAML.

Got left off due to an edge condition in travel.py.

Update cheat.c to unbreak the build.

Create a cheater to test strange save/resume cases.

Patch due to Aaron Traas, but needed modification because of code
drift since submission.

Jettison some disused macros.

Use enums for object number mnemonics instead of VOCWRD() calls.

Fold object vocab words into the objects[] structure.

object_descriptions[] is now objects[].

Add vocab length define to newdb.

Squash a compiler warning.

Change the name of the KEY array to avoid an upcoming macro clash.