Spelling fix.

Remove some commented out code that is obviously useless.

Fix bad line breaks.

Update for include/prototypes.h:1.90 src/lib/subs/attsub.c:1.28 src/lib/subs/shpsub.c:1.22 src/lib/update/human.c:1.16

Update for 4.2.22.

(get_assembly_point): Close loophole that allowed players to find all
unowned sectors.  The assembly point must now be owned or allied, or
have an owned or allied ship.

Info pages didn't match the old rule.  Fix them to match the new rule.

(pin_bomb): Remove code to compute bestacc.  It is not only wrong,
bestacc is not used.  Been that way since Chainsaw.

(ac_landflak): Don't let embarked land units fire flak.

Check against source and update accordingly.

(spy): Catching a spy makes contact.

(ac_encounter): Non-recon missions no longer make contact to owner of
sectors overflown, because only recon missions identify the owner.
This matches info Hidden.

(notify_coastguard): Failed to make contact when NF_COASTWATCH was
off.

Document.

(satmap): Make contact iff displaying sectors.  Shouldn't make a
difference, but is clearer.

(satmap): Spotting land units makes contact.

(ac_dog): Make contact both ways.  This is more for completeness, as
contact should have been made before planes rise to fight.

(notify_coastguard): Use FOUND_COAST, not FOUND_LOOK.

(add): Initialize nat_last_login and nat_last_logout to zero rather
than current time.  Nation addition time is irrelevant.
(natstr): Document zero nat_last_login, nat_last_login mean never
logged in/out.
(player_main): Test nat_last_login zero instead of nat_hostaddr empty.

(player_main): Assign nat_userid, nat_hostname, nat_hostaddr straight
from player.  The last connection's host is now shown as dotted quads
instead of nowhere when the name isn't known.
(natstr): Document that nat_userid, nat_hostname may be empty.

(player_main): Simplify printing last connection address: nat_hostaddr
can't be empty here; don't bother to substitute "nobody" for empty
user.

(match_user, main): Use full buffer for fgets(); was off by one.

(main): Delete some code that had no effect.

(retreat_ship1, retreat_land1): Delete some code that had no effect.

(chan): Do not arbitrarily truncate representative to 8 characters,
use the full size of struct natstr member nat_pnam.

(add): Use size of struct natstr members nat_cnam and nat_pnam instead
of literals.

(lndstr, natstr, shpstr): Fix string types from s_char[] to char[].
They were declared NSC_STRINGY in their selector descriptors, and
NSC_STRINGY is specified to represent char[].
(player): Get rid of s_char.

(babies): Factored out of grow_people().  Identical, except it doesn't
bother to `optimize' the maxpop case, and it drops the weird food
limit computation for opt_NOFOOD, which didn't limited for any
reasonable game parameters.

(ac_encounter): Make contact on spotting planes.  Closes #857594.

(look_ship, look_land): Failed to make contact on sighting ships,
planes and land units.  Partial fix for #857594.

(natstr): Member nat_connected is inappropriate, because it is
transient server state, not persistent game state.  Remove.
(player_main, edit): Remove uses.
(coun): Use getplayer() instead of nat_connected.  Delete the message
on command being out of order.

(player_find_other): Replace by getplayer.  player_find_other()
searches for a player other than US for country CNUM, which is in
state PS_PLAYING.  Since there can be at most one player in state
PS_PLAYING, and no caller pass that player as US, they can simply use
getplayer() instead.

(getplayer): There may be multiple players for the same country in the
list of players, but at most one in state PS_PLAYING.  getplayer()
used to get the one first in the list.  However, its callers need the
one in state PS_PLAYING.  In particular, typed_wu() notifies the
player obtained from getplayer().  If the player in state PS_PLAYING
isn't first, say because another one is trying to log in, the
notification gets lost.  Fix by making getplayer() return the player
in state PS_PLAYING.

(SNEAK_ATTACK, opt_SNEAK_ATTACK, Options, do_sneak, att_abort)
(att_fight): Remove option SNEAK_ATTACK.

(setrel): Relation change establishes contact like a telegram.
Partial fix for #857594.

Remove some redundant parenthesis; no functional change.

(prod_land, prod_ship): Simplify.

(do_feed): Simplify.
(starvation): Pointless.  Inline into do_feed() and remove.

(growfood): Doc clarification.

(att_get_combat): Fix test whether land unit exists.  Closes #1034283.

(att_fight): Fix bad line break.

(get_assembly_point): New.
(bomb, drop, fly, para, reco): Use it.

Don't point to ftp.wolfpackempire.com, it does not work.

(plist): Member state is used write-only, remove.
(P_OK, P_FIRED, P_DEL): Unused, remove.

(N_FLAK, N_DISS_GOV, N_DECL_NEUT, N_RIOT, N_ASSAULT_LOSE): Unused,
remove.
(no_news): new.
(rpt): Use it for r_newsstory of unused slots.

(empfile): Member itemoffs is unused since file.c rev. 1.15; remove.

(IO_CONN, io_conn, IOQ_BUFSIZE, MAXIOV, LND_NOTANY, plur)
(techfactfire, reltech, P_DISTING, PLN_MAXDEF, filetruncate, sct_init)
(nxtsctp, AN_SAILDIR, UDTIMES_MAX, BLITZTIME): Unused, remove.

Unused.

Remove leftover declarations that should have been removed along with
the definitions.

(PP_MAIN): New.
(empth_init): Use it rather than literal.

(getpath): Fix buffer overflow.  Remote hole in commands bomb, drop,
fly, para, reco, sail, sweep.  Old code also screwed up when
getstring() failed, unless player->aborted.  Fixed code is still ugly
and cries for a rewrite.

(getpath): Fix message when no path to destination sector can be
found.

(getpath): Parameter showxy makes no sense and is not used.  Remove.
Get rid of s_char.  Callers changed.

(fly): Check for player aborting the command at the `transport what?'
prompt.  Closes #771801.

(pathcost): Fix the previous rev.

(deli): Diagnose invalid direction.

(retreat_ship1, retreat_land1): Used to accept DIR_VIEW, which
happened do do nothing.  Ignoring it like other invalid directions is
clearer.

(marc, navi, retreat_ship1, retreat_land1): Treat any negative value
of chkdir() as failure, not just -1.

(diridx): New.  Use instead of chkdir() where direction characters
must be valid.  Oopses on bad direction characters.
(pathtoxy, ac_encounter): Bad direction characters used to lead to bad
array subscript and potential disaster.
(pathrange, path): Stop on DIR_STOP as well as on bad direction
characters.  This is just for consistency with other code; DIR_STOP
should occur only last in a path here.
(sail_nav_fleet, nav_ship): No change except for the oops.

(dirindex): Narrow member type to signed char to save cache.
(pathcost): Catch bad direction characters (should not happen).

(SELL_NOT_DELIV, getdir, chkpath): Unused, remove.

(doexecute) [_WIN32]: Open the exec file in text mode.
The exec file contains the user commands created by text editor.

(listen_addr): New econfig key.
(player_init): Implement it.

(player_init): Support service name in econfig key loginport.  Service
empire no longer takes precedence over econfig.  Closes #797096.

(hostaddr, hostconnect): Leave printing messages to caller.
(main): Clarify messages.

(hostport) [_WIN32]: Remove useless conditional code.

(main): Fall back to compiled-in host only if environment doesn't
supply one, not if it can't be resolved.  Same for port.

(deli): Break the loop if player->aborted.  Without that, it can
prompt for more input, violating the client protocol.

(prmptrd, uprmptrd): Should not be called while player->aborted.  Check.

Change info pages to reflect the code change that sets the build reaction
radius to zero for land-units.

(upd_ship): Limit oil production to what's actually in the ground.

(io_output): Remove a redundant check.

(move): test command checked start sector for space and some other
things instead of end sector.

(move): test command now skips code dealing with return of goods to
start sector.

(want_to_abandon): Simplify.

(do_conv): Use maximum population instead of 999 to limit number of
new uw.

(shoo): Remove the arbitrary limit of 999 people to be shot.

(ltend): Fix printing of land unit in diagnostic message.

(nbomb): Rev. 1.17 made a questionable initializer incorrect, and thus
broke pin-bombing for hosts where sizeof(i_type) != sizeof(int).  Fix.

(nstr_resolve_id): Fix spelling mistake.
unkown -> unknown

Change 0 to NULL for pointer initializations.  No functional changes.

(io_output_all,update_main,update_wait): Do not block on writing when
aborting commands for an update.  Move update_pending to the start of
the update sequence where the commands are aborted.

(force): Change to create update thread instead of calling
empth_sleep() as the empth_sleep() can not be aborted for an update.
(update_force): New.

[_WIN32]: Use pconfig.obj instead of pconfig.o for ntpconfig:

Update for src/lib/subs/attsub.c:1.28 and src/lib/update/revolt.c:1.16

Update for src/lib/subs/pr.c:1.27

(empth_yield): Remove sleep().  Delay not necessary.

Update to current output format.

(show_opts): New, factored out of vers().  Fix output punctuation.
(vers): Fix output punctuation and grammar.

(wrgamesdef): Don't point to ftp.wolfpackempire.com, it does not work.

(SHIPNAMES, opt_SHIPNAMES, Options, carg, name, sorde, qorde)
(show_sail, shi, att_get_combat, pr_com, satdisp, satmap, prship):
Remove nooption SHIPNAMES.

(sorde): Output messed up for ships with sail path.  Fix and
simplify.  Remove inappropriate ship sanity check, should be done
elsewhere, and differently.

(qorde): Fix the previous rev.

(qorde): Output messed up for ships with sail path.  Fix and
simplify.  Remove inappropriate ship sanity check, should be done
elsewhere, and differently.

(checksect, give_back_cap, guerrilla): No longer change capital to
agribusiness.  That was done when the occupier abandons the old-owners
capital or it revolts, to prevent players from sacking the same
capital many timers.  The server has a much better way to prevent that
since 4.2.7, so this weird hack is no longer useful.  Thanks to Pat
Loney for pointing this out.

(meltitems): Remove unnecessary checks.

(meltitems): Move melt_item_denom into the ichr structure.

(prmptrd,uprmptrd): Remove unused variable cp.

(getcommand): Remove unused variable i

(lwpReschedule): Internal function, move declaration from lwp.h to
lwpint.h.

(empth_yield) [_EMPTH_LWP]: Blocked the current thread forever; fix.

(prompt,dopipe) [_WIN32]: Enable the pipe capability.

Remove some redundant parenthesis; no functional change.

(player_init) [LISTENMAXCONN]: Comment seems to claim Linux doesn't
have SOMAXCONN, but has LISTENMAXCONN.  In fact, it has SOMAXCONN, and
LISTENMAXCONN doesn't exist anywhere in /usr/include or the Internet.
Remove the special case.

(rpt): Restore tabular formatting destroyed by emp-indent.

Delete some code that has been disabled out for ages and looks
useless or wrong.

(empth_create): Adjust stack size to at least PTHREAD_STACK_MIN, not
PTHREAD_STACK_MIN + 1.

Cleanup, no functional change.

(empth_t, empth_sem_t): Define as incomplete types in empthread.h,
complete in pthread.c, to avoid exposing implementation details.

(EMPTH_KILLED): Move from empthread.h to pthread.c.

[_DECTHREADS_] Drop support for DECthreads d4, a.k.a. DCE threads,
which are obsolete since at least 1997.

(empth_terminate, empth_wakeup) [__ppc__]: Ancient versions of OS X
lacked pthread_kill().  As a work-around, its use was disabled here
for all versions of OS X.  The work-around could lead to hangs.
Remove it and drop support for ancient versions of OS X.

(empth_create) [__linux__]: Linux has supported thread attribute
stacksize for ages.  Remove the special case.

(_MIT_POSIX_THREADS): Unused, remove.

(empth_setctx): Unused, remove.

(trade_desc, get_materials): Use i_type variable instead of int for
iterating over item types.

(sell): Prompting for `amount' is confusing: is it the amount of goods
to offer or amount of money to ask for?.  Ask for `quantity' instead.