(dchr_ca, packing, empfile): Add packing symbol table.

(sect_ca, ship_ca, land_ca, plague_stages, empfile):
Add plague stages symbol table.

(treaty_ca, loan_ca, agreement_statuses, empfile):
Add agreement status symbol table.
Combine treatystatuses (TS_FREE, TS_SIGNED and TS_PENDING)
and loan statuses (LS_FREE, LS_SIGNED and LS_PENDING)
into a single agreement status.

(pchr_ca, level, empfile): Add packing symbol table.

(output): Remove C_FLUSH from output() as C_FLUSH is dealt with
by servercmd() and output() is not called from servercmd() with a
code of C_FLUSH.

(xunsymbol): Fix xunsymbol() to parse multiple symbols for a
symbol_set (NSC_BITS).

(xundump): Add prototypes.h to get a prototype for
xundump().

(servercmd): Remove newline for C_INFORM, C_FLUSH,
C_REDIR, C_PIPE and C_EXECUTE.
Fixes the problem created when the newline was left in io_gets().

(buil, vers, opt_NONUKES, show_nuke_build, show_nuke_capab):
Remove opt_NONUKES options.  A nuke characteristics file can
be created with no entries to create the emulate the removed option.

(dchr_ca): prd indexes product table.

(EF_MISSION_FLAGS, EF_MISSIONS, mission_flags, missions): Rename, as
these are not flags.

(init_server, ef_load, xundump): Add ability to customize game
configurations.  The custom game configurations are loaded via
text files in the data directory.  Closes #752881.

(io_gets, output): Remove the removal of \n in io_gets() and the
readdition of it in output().

(servercmd, io_gets, output): Fix client to read lines longer than
1024 bytes from the server.  Closes #1223255.

(xdump): Add EF_MAX check for table selection by number.
Fixes a crash when selecting a table over EF_MAX.

(plane_flags, plane_ca): Add plane flags table.

(retreat_flags, ship_ca, land_ca, plane_ca): Add retreat flags table.

(nation_flags, nation_relations, nat_ca): Add nation flags and
nation relations tables.

(empfile_ca): Remove flags and file name fields.
They are internal and not needed by the client.

(start_server, empth_start): Passed uninitialized sa_mask to
sigaction().  This used to broken only when SA_SIGINFO wasn't defined,
until main.c rev. 1.46 and pthread.c rev. 1.10 broke it
unconditionally.

Fix bad line break.

(ef_init_chr): Sentinel names are now null, simplify sentinel check.

(init_server): Remove ef_load() call.
New function not ready yet, incorrectly
checked-in.

(itm_maxno): Remove itm_maxno, it is not used.

(nullify_objects): Fix sentinel row checks for plchr, lchr and mchr loops.

(shp_maxno): Variable's value is well-known constant N_MAXSHIPS + 1.
Useless, remove.  Users changed to check mchr's sentinel, because
that's clearer than comparing the index to N_MAXSHIPS or sizeof(mchr)
/ sizeof(*mchr).

(mchr): Change sentinel's m_name to NULL.

(pln_maxno): Variable's value is well-known constant N_MAXPLANE + 1.
Useless, remove.  Users changed to check plchr's sentinel, because
that's clearer than comparing the index to N_MAXPLANE or sizeof(plchr)
/ sizeof(*plchr).

(plchr): Change sentinel's pl_name to NULL.

(pstat): Remove inappropriate plane sanity check; should be done
elsewhere.

(lnd_maxno): Variable's value is well-known constant N_MAXLAND + 1.
Useless, remove.  Users changed to check lchr's sentinel, because
that's clearer than comparing the index to N_MAXLAND or sizeof(lchr) /
sizeof(*lchr).

(lchr): Change sentinel's l_name to NULL.

(carg, lcarg, fort, land, ldump, lsta, orde, payo, retr, lretr)
(cmd_sail_ship, cmd_unsail_ship, show_sail, sdump, shi, sstat, work):
Remove inappropriate ship/land sanity checks; should be done
elsewhere.

(show_nuke_build, show_nuke_capab): Simplify.

(nuk_maxno): Variable's value is well-known constant N_MAXNUKE + 1.
Useless, remove.  Users changed to check nchr's sentinel, because
that's clearer than comparing the index to N_MAXNUKE or sizeof(nchr) /
sizeof(*nchr).

(nchr): Change sentinel's n_name to NULL.

(buil): Use typematch() to search for ship, plane, land unit and nuke
type.  The old code didn't work correctly when "sbb  super battleship"
came before "sb   submarine" in mchr[].

(arm, tran_nuke): Lookup of the nuke name was broken by nuke.h
rev. 1.10.  Replace by typematch().

(arm): Simplify search for nuke at sector.

(prd_maxno): Variable's value is well-known constant P_MDUST + 2.
Useless, remove.  Users changed to check pchr's sentinel, because
that's clearer than comparing the index to P_MDUST or sizeof(pchr) /
sizeof(*pchr).

(pchr): Change sentinel's p_name and p_sname to NULL.  Sentinel
checks changed.

(empfile,mission_flags,ship_ca,land_ca,plane_ca):  Add mission flags table.

(nullify_objects): Fix option NO_OIL.  The attempt to remove ships
with capability M_OIL from mchr[] actually replaced them with
sentinels.  Just clear the capability instead.

(xdopt, xdver): Simplify the previous rev.

(xdump): Support selecting a dump by table number in addition to name.

(ef_ptr, ef_read): Oops on bad ID.  Some callers rely on these
functions to fail at the end of the table.  To support that, the first
ID beyond the end is not considered bad.

(ef_ensure_space): Oops on bad ID.

(NSC_BITS): Remove the trailing comma as it is last row.

(install_service):  Use the compiled in default for location of econfig
when the -e is not specified while creating a service.

(xdhdr,xdhdr1,xditem,xdmeta,xdopt,xdver,xdfldnam):
Remove the column header.
Not need as the xdump meta is functional.

(xdopt,xdver,xdump): Add meta data for opt and ver.

(xdmeta): Fix the printing of the number of rows dumped.
Incorrectly included the skipped rows.

(xdmeta): Don't dump meta data for stuff that is not dumped by
xdflds().

(NSC_BITS): New.
(meta_flags): Update for it.
(ship_ca, mchr_ca, plane_ca, plchr_ca, land_ca, lchr_ca, nchr_ca)
treaty_ca, nat_ca, empfile_ca, mdchr_ca): Use it.

(symbol_ca): Move value selector to beginning of array.

(meta_flags): Downcase names.

(mission_short_name):   Remove mission_short_name, not used.

(nullify_objects): Patch intrchr[] according to opt_NO_LCMS,
opt_NO_HCMS.  This fixes output of `show se b', which used not to
reflect these options.
(improve): Simplify.

(xdmeta): Fix rev. 1.36.

(UNMAPPED_INIT, UNMAPPED_CACHE, ARRAY_INIT, ARRAY_TABLE, PTR_INIT)
(PTR_CACHE): Rename.

(ARRAY_CACHE): New.
(empfile): Use it to set csize of elements EF_SHIP_CHR, EF_PLANE_CHR,
EF_LAND_CHR, EF_NUKE_CHR.
(ef_init_chr): Now that empfile[]'s initializer sets csize, remove the
temporary csize initialization here.  The value wasn't correct, but
better than nothing.

(perform_mission): Do not pass the sub-owner to nreport().  News
doesn't show it, but selectors do.  Reported by Doug Wescott.

(nreport, ncache): Catch errors like the above.  Catch bad event
argument while we're at it.

(xdopt, xdver): Fix missing footer.

(xdmeta): Add NULL ca check.
Fixes a crash for xdump meta pow.

(ichr_ca, pchr_ca, sect_ca, dchr_ca, NSC_GENITEM, ship_ca, mchr_ca)
plane_ca, plchr_ca, land_ca, lchr_ca, nuke_ca, nchr_ca, treaty_ca)
loan_ca, news_ca, lost_ca, commodity_ca, trade_ca, nat_ca)
(empfile_ca): Move row index selector to beginning of array and set
its table.  Set table for NSC_SITYPE(i_type), NSC_NATID, NSC_TYPEID
selectors.
(news_ca): Set selector action's table.
(commodity_ca, trade_ca): Set selector maxbidder's table.

(trtstr): New members ef_type and trt_uid.
(do_treaty): Initialize them.
(treaty_ca): New selector uid.

(ef_ptr): Null cache can't happen.

(ef_open): EFF_STATIC and cache not null can't happen.

(ef_open): Leave check of fd to do_write().

(empfile_ca): Rename selector file_name to file, to match the member
name.

(ichr): Define without array size, like the other characteristics
tables.

Doc fix.

(diroff, dirch, routech): Declare with array size.  Define without.
This requires moving routech[] declaration from map.h to path.h.  Get
rid of s_char.

(ep): Global variable pointing to sector cache.  Gross.  Remove.

(bp_neighbors, bp_lbcost, pathcost): Replace pointer arithmetic by
array subscript.

C_EXECUTE security.

(coun): Last access used to report last login time.
Counter-intuitive.  Report last connection time range instead.
Unfortunately, this requires a change in output format.

(coun): Report on all countries even to mortals.  This used to be
done, and was most probably changed because it gave away too much
information.  Carefully limit the information given to mortals, as
follows.  Limit last access information to whether deities or allies
are logged on right now, no longer report connection time for deities
that are not logged on.  Remove column time for mortals, rename it to
BTU for deities.  Report status, but with less detail if HIDDEN is
enabled.

(buy, check_market, trad, check_trade): Remove the following virtually
undocumented feature: When a buyer didn't have enough money at the
time of the trade, the seller *automatically* granted a high-interest,
medium term loan for up to 90% of the price.  This is just a silly
trap for unwary sellers and buyers alike.
(N_FIN_TROUBLE, N_CREDIT_JUNK): Unused, remove.
(rpt): Update accordingly.

(empfile): Rename member ef_uid to uid for consistency with other
members.

(empfile): Reorder members for clearer documentation.

(EFF_IMMUTABLE): New.
(EFF_OPEN): Remove, use ~EFF_IMMUTABLE.

(empfile): Don't initialize member size to a useless value for EF_MAP
and EF_BMAP.  ef_init() writes the real value.

(ef_init_chr): Add a NULL check to sentinel check.

(ef_init): Fix check for sentinel.

(xdump): Use xditem() for all empfile tables.
(xdchr, xdmeta): Specialize for meta-tables, rename.

(meta_flags, meta_type): Change sentinel name to NULL to match the
other symbol tables.

(ef_init, ef_init_srv): Rename old ef_init() ef_init_srv(), call new
ef_init().  Move initialization of map sizes to ef_init().
(main): Don't initialize map sizes by hand, call ef_init().

(fileinit): Internal linkage.

(rpt, page_headings): Declare with array size.  Define without.

(empfile): Correctly initialize members cids, csize, fids for all
EFF_STATIC tables except EF_SHIP_CHR, EF_PLANE_CHR, EF_LAND_CHR,
EF_NUKE_CHR, EF_META and the symbol tables.

(EF_TREATY_CHR, EF_TREATY_FLAGS): Rename for consistency with other flags.

(treaty_ca): Add missing EF_TREATY_FLAGS to selectors acond, bcond.

(tchrstr, tchr_ca): Remove, use struct symbol and symbol_ca.  Users
changed.
(tchr, treaty_flags): Rename for consistency with other flags.

(lookup, symbol): Rename.  Rename members to match selectors and
better reflect their purpose.  Users changed.
(lookup_ca, symbol_ca): Rename.

(intmatch): Assumes that struct size is a multiple of sizeof(int), out
of laziness.  The assumption is true on any sane machine, due to
alignment constraints.  But the function doesn't provide enough value
to make me fix or document this.  Remove it.
(lookup): Replace it.

(dchr): Remove second sentinel.

(rpt): Add sentinel.
(EF_NEWS_CHR): New.
(empfile): Add its entry.

(ef_byname): Old version didn't work when a table name was followed by
one of its prefixes.  src/lib/global/file.c rev. 1.18 added such table
entries, and file.h rev. 1.34 made them visible to ef_byname().
Simply use stmtch().  This also rejects ambiguous abbreviations: "s"
used to return EF_SECTOR, it now returns M_NOTUNIQUE.

(nchr): Move the array size to nuke.h.  Add rows for configurable units.
(lchr,plchr,mchr): Add array size to land.h/plane.h/ship.h.  Add rows for
configurable units.
(ichr,pchr,dchr,intrchr): Add array size to item.h/product.h/sect.h.

(empfile): Add one to the array size for the sentinel row.

(NSC_CONST,castr,meta_flags): Add new flag NSC_CONST to castr.
(ichr_ca,pchr_ca,intrchr_ca): Use it.

(EF_IS_GAME_STATE): New.
(xdump, main): Use it.

(EF_MAX): Change to largest table ID + 1.  This is now possible
because the changes above get rid of the assumption that
empfile[0..EF_MAX-1] is only game state.  Code can now work on any
table using the advertized empfile interface, not just on game state.
(my_ef_byname): Remove, use ef_byname().

(set): Fix the previous revision.

(ef_byname_from): New.
(set, mission, multifire, fuel, cons_choose, cede): Use it instead of
ef_byname().  Remove the hacks to map unwanted results to wanted
results.  The hacks in set() were broken: bad input made the code cast
arbitrary items to union trdgenstr, with potentially disastrous
consequences.  Closes #906483 and #906486.

(set): Less cryptic prompt and diagnostics.

(xdftr): Change footer to / instead of dumped.

(xdump,xdchr,xdhdr,xdhdr1): Add meta data dump to xdump.

(ef_open): Implement EFF_STATIC.

(ef_open): Simplify dynamic buffer allocation.

(empth_start, empth_init) [_EMPTH_POSIX]: Do not handle SIGALRM.  We
don't raise it, and the signal handler is fishy.
(empth_alarm): Unused, remove.

(info): Remove dead code.

One more SNEAK_ATTACK leftover.

(disassoc): Rewrite for POSIX, return status.  Caller changed.
Problems with the old code:
* Insufficient error checking.
* It used TIOCNOTTY (obsolete BSDism) to get rid of the controlling
  tty, except for hpux || Rel4, where it attempted to use the POSIX
  way, but screwed up.
* It left file descriptors 0, 1, 2 in a somewhat weird state.

(empfile,mdchr_ca,meta_type,meta_flags):
Add meta table and associated symbol tables.

(main): Don't close EF_NATION twice.

(EFF_STATIC): New.
(empfile): Use it.
(ef_open): Oops on it (not implemented).
(ef_close): Implement it (trivial).

Comments.

(ef_nameof, ef_ensure_space): Oops on bad argument.

(ef_check): Use CANT_HAPPEN().

(ef_zapcache): Clearing the cache doesn't make sense.  No harm done if
the next user can't use the cache, it gets automatically zapped then.
Remove.

(ef_read): Catch null cache.

(ef_read): Used to get the file size again some circumstances; doesn't
make sense, as no other process may be writing this file.

(ef_write): Don't refuse to write beyond record 65536.  This module
can cope with that.  Some users have trouble long before that (short
record numbers), but that should be handled there.

(ef_write): Catch table not file-backed.

(ef_extend): Use do_write() & simplify.  Catch table not file-backed.
Catch negative count; used to have no effect with EFF_MEM, and
confusion otherwise.  Try to recover from partially successful
extension.  Need help from ef_write() to do a better job; to be done.

(ef_extend): Fix rev. 1.25.

Clean up rev. 1.30.

(ef_ptr): Change return type to void *.

(ef_ptr): Use CANT_HAPPEN() to ensure EFF_MEM; check it before less
serious index out of range failure.

(ef_open): Fix fatal typo.

(ef_open): Portability fix: malloc(0) may return null, which is not a
failure.  fillcache() chokes on null cache, so don't call it for empty
files.

(ef_open): Failed to reset empfile member fd on failure, and leaked
file descriptors.

(ef_flush, ef_close): Rely on negative fd instead of null cache to
detect unopened file.  This is more logical and consistent with the
use of cache by the new tables that are not file-backed (rev. 1.22).
Use CANT_HAPPEN().

(ef_close): Failed to report cash flush failure.

(ef_close): Clear all EFF_OPEN flags.  Should have been done in
rev. 1.27.