Remove superflous include.

(lwpNewContext) [MAKECONTEXT_SP_HIGH]: Work around makecontext()
lossage.
(IRIX5CFLAGS, SOLCFLAGS): Define MAKECONTEXT_SP_HIGH.  This is wrong
for Solaris 10.

(lwpCreate): Check for lwpNewContext() success.

(lwpNewContext) [UCONTEXT]: Fix the previous revision.

Redesign interface to machine-dependent code to cleanse lwp.c of
machine-dependencies:
(lwpNewContext): New, factored out of lwpCreate().
(lwpSwitchContext): New, factored out of lwpReschedule().
(LWP_EXTRASTACK, STKALIGN, lwpInitContext, lwpSave, lwpRestore): Move
to arch.c.  Remove for UCONTEXT, else static linkage unless macro or
AIX32.

(lwpReschedule): Skip lwpStatus() when not actually switching.

(LwpStackGrowsDown): New.
(lwpInitSystem): Initialize it.
(lwpStackCheck, lwpStackCheckUsed): Use it.

(lwpProc): Members lowmark and himark are confusing, replace by
ustack, usize.
(lwpCreate): Initialize them.
(lwpStackCheckInit, lwpStackCheck, lwpStackCheckUsed): Rewrite using
them.  lwpStackCheckUsed() was off by LWP_REDZONE when stack grows
down.

(lwpDestroy): Don't bother to clear memory to be freed.

Clean up the move of lwpProc (lwpint.h rev. 1.11, lwp.h rev. 1.8).

(lwpInitContext) [UCONTEXT]: Failed to initialized uc_stack.ss_flags.

Clean up the move of lwpProc (lwpint.h rev. 1.11, lwp.h rev. 1.8).

(empth_start, empth_init, empth_alarm) [_EMPTH_POSIX]: We do raise
SIGALRM!  Revert rev. 1.21.

(empth_alarm): Internal linkage.

(lwpCreate) [UCONTEXT]: ss_sp is the (base) address of the stack area,
not the initial stack pointer.  Fix.  ss_size was wrong as well.
Amazing that this ever worked.

[UCONTEXT] (lwpInitContext): Failed to initialize context.uc_link as
required by the API.

(lwpCreate): Check STKALIGN is sane.

(lwpCreate): Used to allocate one byte more than necessary.  The fix
won't save memory (malloc() will add the byte back), it's just clearer
this way.

(lwpCreate): Clean up computation of sp.  Rename parameter size to
stacksz.

(lwpCreate): Always compute newp->lowmark and newp->highmark from sp,
not from s.  Old code misaligned newp->himark for stacks growing
upward when stack size argument wasn't aligned.

(stkalign_t, STKALIGN): Replace.

(lwpProc, lwpSem): Declare as incomplete type in include/lwp.h.  Move
complete declaration to src/lib/lwp/lwpint.h.
(lwpQueue): Move to src/lib/lwp/lwpint.h.
(empth_main, empth_flags): New.
(empth_init): Initialize them.
(empth_create, empth_exit): Use them instead of lwpProc members.

(lwpReschedule): Fix the previous revision.

[BOUNDS_CHECK]: Defunct, remove.
(lwpReschedule): Fix potential crash bug that crept in with
BOUNDS_CHECK years ago.

(lwpSleepFd): More information in debug output.

(lwpSelect): Fix use of uninitialized variable in debug output.

Fix the previous revision.

Clean up includes.

Doc fix.

(oops): Move declaration to misc.h, to make CANT_HAPPEN() usable
with just misc.h included.

(lwpSave, lwpReschedule) [hpc]: Move machine-dependent code to arch.c,
where it belongs.  Code looks fishy.  Untested.
(initcontext, startpoint): Internal linkage.

Remove LWP code for targets we no longer support: VAX/MIPS Ultrix
(dropped in Make.sysdefs rev. 1.12), 386 USL/Unixware (unclear, long
ago), 386BSD (never been?)

Clean up the incomprehensible #ifdef jungle a bit.  Didn't compile if
both UCONTEXT and one of the other recognized preprocessor symbols was
defined; fix to prefer UCONTEXT.

(lwpCreate): Clean up the incomprehensible #ifdef jungle.  No
functional changes.

Remove silly superflous parenthesis.

(peek): Old owner is not hidden, remove column.

(peek, empmod): Remove obsolete command hidden.

Doc fix.

(xundump): Fix rev. 1.19.

(xuflds, xunsymbol, xuloadrow): Symbol sets.
(VAL_SYMBOL_SET, getid, xunsymbol1): New.

(xunsymbol): New argument n for more useful diagnostics.

(xuloadrow): Remove redundant semantic check.

(xunsymbol): Fix design flaw: negative symbol values were interpreted
as failure.

(xuflds): Allow unescaped space and tab in strings.  Also fix the
previous revision, which broke '#' in strings.

(xuloadrow): Fix crash.

Less rigid field separators, support comments:
(skipfs): New.
(xuflds, xundump): Use it.  Update field scanf stop-sets accordingly.
(xuflds): Update test for good field separator.

(xundump): Fail silently if nothing can be read and no specific table
is expected.

(xuloadrow): Obey NSC_EXTRA.

(xuloadrow): Remove redundant `not supported' check.

(ef_init_srv): Call ef_init() before ef_load().
(xundump): Set empfile member cids in addition to fids.

(xundump): Clarify shortcomings: requires EFF_MEM, appends sentinel
always, can't grow dynamic cache.

(xundump): Return table type on success.

(xuloadrow): Fix rev. 1.11.

(xundump): Fix parsing of XDUMP header (really!) and footer.

(xuflds): The way field separators were parsed lead to confusing
diagnostics on EOF.  Fix.  Also simplify treatment of newline.

(xuflds): Require identifiers to start with a letter.

(xundump): Fix parsing of XDUMP header.

(xuflds): Test for no fields was off by one.  Remove it; let caller
deal with missing fields.

(xuflds): Simplify test for space in values[].

(xundump): Improve diagnostics.  Remove superflous check for
unexpected EOF.

(xundump): Table names no longer contain whitespace.  Simplify.

(empfile): Space in Empire names requires players to quote the names
on the command line.  Replace by dash.

(xuflds): Fix confusing diagnostics on unexpected EOF.

Plug memory leaks:
(freeflds): New.
(xundump): Call it.
(xuloadrow): Leave freeing storage to freeflds().
(xuflds): Ensure values[] is terminated with a VAL_NOTUSED element
even on error.

Roll back to rev. 1.3 in preparation of a re-engineering of the
changes since then.

(verify_row): Remove // comment.
Change %d to %ld when printing a long.

(ef_init_srv): Add a call ef_verify() to verify game data and game
configuration consistency.

(ef_verify): New.  Verifies game data and game
configuration consistency.

(ship_chr_flags[]): Remove internal flags (M_XLIGHT, M_NCHOPPERS)
from ship_chr_flags[] so they do not appear in the show output.

(missions[]): Add MI_NONE to missions[] table.

(xuflds): Allow multiple spaces to separate fields or symbols in symbol set.

(xunsymbol): Fix formating.

(xuloadrow): Add missing case for VAL_SYMBOL_SET for trailing
junk row check.

(xuskipcommentlines): Make the function static.

(xuflds, xunsymbol, xuloadrow): Split VAL_SYMBOL into
VAL_SYMBOL and VAL_SYMBOL_SET.  Identify SYMBOL_SET
by the '(' and ')'.  Switch SYMBOL_SET separator from '|' to a space.

(xuskipcommentlines): Include the lines skipped in the line count.

(xundump): Skip lines beginning with '#'.
These lines are considered comments.

(xuskipcommentlines): New.

(plane_bomb, land_bomb): Apply collateral damage to target sector
instead of where the targeted plane/land unit is after the attack.
Different if the target retreats.  Can't happen for planes.  Reported
by Greg German.

Fix bad line breaks.

(close_files, finish_server, panic):
Rename close_files() to ef_close_srv() and move to fileinit.c.
Replace calls to close_files() with ef_fin_srv().
Call ef_close_srv() from ef_fin_srv().

(ef_close_srv, ef_fin_srv): New.

(init_files, init_server, ef_init_srv):
Rename init_files() to ef_open_srv() and move to fileinit.c.
Remove call to init_files() from init_server() and replace
with a call to ef_open_srv() from ef_init_srv().

(ef_open_srv): New.

(ef_load, init_server, ef_init_srv):
Move the call to ef_load() from init_server() to ef_init_srv().

(global_init, init_server, ef_init_srv):
Move the call to global_init() from init_server() to ef_init_srv().

(nullify_objects, global_init, init_server):
Split nullify_objects() into init_lchr() and init_dchr().
Locate init_lchr() and init_dchr() in init.c.
Add calls to init_lchr() and to init_dchr() from global_init().

(init_lchr(), init_dchr()): New.

(fillcache, do_write): Revert the previous revision.  The portability
issue applies only to sockets, and EWOULDBLOCK isn't defined on all
systems.

(show): Return syntax error for invalid qualifier.

(mchr[],m_flags,m_nland): Remove M_UNIT from m_flags in mchr[].
It is not used.

(lookup): Rename lookup() to symbol_by_value(), make it public
and move it to nstr.c.

(fillcache, do_write, io_input, io_output): Portability fix: always
check both EAGAIN and EWOULDBLOCK.  These are distinct on some
systems, e.g. HP-UX.
(io_output) [hpux]: Used to treat all errors as EWOULDBLOCK.  This
looks like a inept attempt to `fix' the above.  Remove.

(pln_equip): Do not abandon sector for paradrop, drop or fly.
Closes #982226

(tend): Send a bulletin to the ally when tending items to an ally.
Closes #929398.

(kill_cmd): Add authority check for deities before allowing a kill.
Closes #1215088.  Add banned host check before allowing a kill.

(natbyname): Do not look up a non-active country.
Previously a blank country name or deleted country name would
return a pointer to non-active countries.
This creates a security problem where some one can successfully
login into the server without country or password.  Closes #1219569.

(ship_chr_flags):  Add internal flags (M_XLIGHT, M_CHOPPER, M_UNIT)
to ship_chr_flags[].

(tend_land): Allow spy units to be tended to and from sub
with no land unit carrying capability.  Closes #1313537.

(dchrstr, pchrstr, rptstr): New member d_uid, p_uid, r_uid.
(dchr, bigcity_dchr, pchr, rpt): Initialize it.
(dchr_ca, pchr_ca, rpt_ca): New selector uid for it.

(ef_write): Don't memcpy() onto itself; behavior's undefined.

(multifire): When an argument is bad, do not loop prompting, just
fail.  That's what the vast majority of commands do.

(player_coms): Document second argument of launch.

(laun, launch_as, launch_missile, launch_sat): When an argument is
bad, do not loop prompting, just fail.  That's what the vast majority
of commands do.

(launch_as, launch_missile, launch_sat): Don't zap the target
argument.  If the target is given as argument, launch all missiles at
that target.  Previously, missiles after the first prompted for
another target.

(buil): When an argument is bad, do not offer to list types, just
print how to list them.

(buil): When an argument is bad, do not loop prompting, just
fail.  That's what the vast majority of commands do.

(getcontact,putcontact): Generalize the interface back to an int.
Add range checks to prevent unexpected results.

(buil): Remove N_NEUT check.  It should have been
removed when the opt_NEUTRON was removed.

(plurize): Fix the max. string length calculation for the 'y' case.
It was off by one because 'y' is removed from the string.

(nat_ca): Add nat_contact to nat_ca[].

(cons_choose): When an argument is bad, do not loop prompting, just
fail.  That's what the vast majority of commands do.

(ef_ptr, ef_read): Undo rev. 1.52 for these two, because there are
callers that rely on these functions to fail for bad ID.

(add): When an argument is bad, do not loop prompting, just fail.
That's what the vast majority of commands do.

(add): Do not silently truncate country number argument.

(add): Diagnostics were confusing because they lacked context.

The previous revision accidentally deleted commented out code.
Restore.

(getcontact, putcontact, nat_contact): Change nat_contact
to unsigned char to reduce the space required for natstr.

(prod_nat): Remove non-functional aging code for nat_contact.
Remove nat_contact debugging from server log.

(update_main, prod_nat): Move aging code for nat_contact
from update_main to prod_nat.

(agecontact): Remove the opt_LOSE_CONTACT, the calling
procedure update_main() already has opt_LOSE_CONTACT check.

(getcontact, putcontact): Remove the bit shifting that was in the
function to save space.  opt_LOSE_CONTACT changes negated
any space saving.

(cens, desi, dump, do_desi, doland, file_sct_init, grow_continents)
(grow_islands, set_coastal, update_coastal_flag, coast_sea_to_land)
(coast_land_to_sea): Determine sct_coastal flag when the sector type
is set.  Old method only updated the sct_coastal flag when doing
cens() or a dump().  Other uses of sct_coastal could get stale values,
including selector coastal.  Rework set_coastal() into NEW functions
update_coastal_flag(), coast_sea_to_land(), coast_land_to_sea() and
set_coastal().  Fix a bug where the deity designation would not
properly set the coastal flag.  Fix a bug where the one sector lake
would not properly compute coastal flag.  Fix a bug that makes
fairland treat the last sector it adds to an island as non-coastal.

(file_sct_init, main, fl_sct_init, write_sects): Assign sct_timestamp with the
time the sector was created in files and fairland programs.

(add): Convert bridge spans or bridge towers into sea sectors instead the
wilderness sectors when wiping sectors.

(cens, cens_hdr): Print the actual old_owner instead of just a star
when different.  This is change the format for the census output.

(plurize): Change variable name max_len to size to improve readability.

(plurize): Fix initial length check to check for zero length of buf
not non-zero lengths of buf.

(plurize): Change variable name size to len to improve readability.

Update patch for src/lib/commands/mfir.c:1.26,src/server/main.c :1.79,include/prototypes.h:1.98

(fuel, load, prod, max_population, shp_nav_one_sector,
nav_loadship, upd_buildeff): Identify BIG_CITY
(IS_BIG_CITY) by using packing type of UPKG instead of
opt_BIG_CITY and sector type of SCT_CAPITAL.

(plurize): Fix the strncat() len parameter to prevent an overflow of buf.

(numstr, esplur, splur, iesplur, plurize): Move the prototype defintions from
misc.h to prototypes.h.  Remove unneccessary headers in plur.c.