Putting cleanup in the right place, too

Fix make debug check

Without IGNORE, -Wunused-result throws a warning on fread(2).

I've seen it happen on fwrite(2) as well; wrapping that as well in
case some compiler decides to be finicky.

Magic-number elimination.

Test modification for little more coverage

Stop leaking memory

Various cleanups.

* Remove disused macro.
* GCC doesn't seem to mind if IGNORE() is left off.
* Enumify phase codes.
* Use EXIT_* macros in all exit() calls.
* How did this even work without extern?
* Give advent.h a much-needed makeover.
* Use the chomp indicator in YAML string blocks to avoid code ugliness.

Error message when you make if without retrieving git modules yet

Use YAML block-literal syntax to avoid wrapped lines.

In vspeak(), andle text with extra trailing \n.

The bugtype enum doesn't need explicit numbers.

Plugged little linenoise memory leaks.

Implement and document %V escape so version only needs to be set once.

Remove disused function definitions.

Remove more disused macros.

Remove some disused stuff.

Magic-number elimination.

Removed line as redundant

As of 8675c4facd154b9 lines 612-613 in main.c unreachable due to earlier
dispatch at line 1132 that redirects all find/inventory verbs into actions.c
Message NEARBY fires from YAML link to those verbs.
Also, reduced signature of playermove() to motion only.

Less magical numbers.

Improve test coverage.

Magic-number ellimination, contiguity warnings, cleanup.

Tests for misc and main

Remove gaps from arbitrary_messages, there are no more number sependencies.

Some spans within the list may still rely on contiguity.

Fixed previous test, added nibble more

Magic-number elimination.

Fix date typo.

Last one test case

Some more easy tests

Update illformed.chk for current YAML.

Deduce version from NEWS.

Update NEWS.

Implement 'version' command.

Test coverage -- fix regression

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