Make the YAML less fussy.

Don't string-quote things that are semantically like enums.

playermove() never returns false; simplify accordingly

Visit room with breathtaking view
Try to extinguish a volcano
Closes #22

Broke all logical or into multiline statement to show lcov's lies
Results are weird, some things that need to be checked aren't and
at least my local lcov doesn't show them as unchecked

Unspk'd bivalve action, exposing more untested surface

Unspk'd fly command

Unspk'd listen and lock commands

Unspk'd pour command

Unspk and streamline eat command

Removed helper function

Wake dwarves differently

Cover branch last branch of bird action
And remove distracting line from tests

Coverage -- attack bird after game closed

Remove testing for intransitive seperate from NO_OBJECT in attact()

because we look for it immediately below, and this code is never
reached. If it were, intransitive attacks would never work.

Coverage -- corner cases in lock and find

Coverage -- giant words

Coverage -- intransitive attack

Removed spk from wave, fixed compiler warning

Undid unspeaking of wave, fixed test again.
Changed property math to helper function.

Don't spk in wave and action

More magic number elimination

Unfalsify coverage check

Magic-number elimination.

More tests, multilined yaml

Magic-number elimination.

Attack content and dead bear

Imprive coverage of cheat.

Coverage - attack sitting bear

Eliminate the global game.blklin.

Rationalize attack logic
Fix the compiler warnings about bunching ands with ors

Magic-number elimination.

Bump save file version number.

Also add missing space in the version mesage.

Test coverage: intransitive lock

Eliminate unreachable code in lock()

Add capabilities to cheat. Expand testing to cover.

Moved application settings to settings_t struct

Since logfp, oldstyle, and prompt were application settings, rather than
have them all as global vars, move them to a single global var, seperate
from game state, as they aren't, technically, game state, but are
application settings.

Replace datime() with just time().

Also make the 'savetime' value meaningful.

This removes a separate library dependency on some systems.

Change VERSION_SKEW message - save file cersion is now...

...decoupled from program version.

Documentation polishing.

Documentation polishing.

Begin decomposing cond magic numbers.

Fixed logic, documented it in notes and tests

Fixed regression in #21 Cleaned up logic of light/extinguish

Fix for issue #21, hopefully

really capture it

Capture current state of bugs, for #21

Capture wonkyness of extinguishing urn

more tests

Ability to use non-standard FreeBSD gcov tool

Make include of dependency platform-generic

Localize scope for restore file pointer in main.

Coverage -- version command

centralize calls to make_zzword()

make_zzword() now called in set_seed(), because the only times it was
called in the entire application is after every invocation of set_seed().

Fixed -r option. Added test to make sure it doesn't break later.

Changed usage message to multi-line string for cheat

make_zzword() moved to initialize. Pruned unused stuff from cheat.c

Improve test coverage.

Rewrite dwarf-encounter ceck so coverage testing sees all predicates.

More magic-number elimination.

Magic-number elimination.

Fix typo in data comments

Fix typo in notes

Supply missing implementation of nodwarf bit in travel rules.

Another step in de-obscurifying the travel code.

Refactor specials[] to have strings instead of enum references.

Clean up arbitrary_messages[] accordingly.

Minor refactor - create an equality function to simplify skip logic.

Never write the input prompt to log files.

Forther break apart the magic enconding of travel arrays.

Remove debris from a misclick.

Ignore l g z i under oldstyle.

Change state of vase
Clear temporary save where it needs to be cleared instead of random places

Do not compare state to magic numbers

Little more less magic in dragon state

Documentation polishing.

Ensure libedit is installed for the pipeline jobs.

Attempt to unbreak pipeline.

Jettison the -s option.

It's no longer needed since libedit doesn't have the 'paste bug'.

Remove the linenoise submodule.

Move most build instructions into INSTALL.adoc.

Update documentation about the libedit conversion.

Replace linenoise with libedit in code and build.

Magic-number elimination for dragon and rug.

Reindend, compile out all save stuff

Put stuff back in init, C-style

Apply DEMOTE_WORD().

Magic-number and state-arithmetic removal.

Add a DEMOTE_WORD() macro.

Interim magic number replacement.

Move PRNG initialization to simplify cheat.c

Spread documentation from init.c It was empty without anything to document. Reindented, too

Cut initialize in favor of implicit/default initialization where possible

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.