(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.

(bdes): Prompt was messed up when bmap contained designation 0.

(status): Internal linkage.

Fix lrangeedit to lrange.  Fix rangeedit to range.  Fix attack to Attacking.
Add html links for info references.  Remove inconsistent periods.
Add missing closing brackets. Change info to information to improve
readability.  Improve the readability for range/lrange sentence.

(stre): Fix to make it match what happens in attack sequence: land
units do not react to a mountain.

(stre): Add an empth_yield() per sector to make the server responsive
to other players as this command is CPU intensive.

(player_init) [_WIN32,__linux__ && __alpha__]: Remove the
SO_REUSEADDR socket option as it is not required for WIN32.
In WIN32, there is no waiting time when a port is closed before
it can be reused so SO_REUSEADDR is not required for WIN32.
Leave the option has a undesirable effect of allowing a second
server instance to start without an error and leaving both
instances running.  ALPHA and LINUX do support SO_REUSEADDR.

(player_main): Add empth_yield() in the command processing loop to ensure
after every command, other players gets a chance to execute a command.
This prevents denial of service attack that monopolizing the active server
thread by continuing feeding data to the server from a client.

(launch_sat): Incorrectly assumed that plane_mob_max is 127 when
calculating when the satellite will be ready for use.

Final batch of 4.2.21 changes.

Only of historical interest.

Document new files.

(melt_item_denom): i_type shift (include/item.h 1.12) broke
initializer, screwing up fallout damage.

Still more 4.2.21 changes.

(pln_equip): Fix the previous rev.

(mission_pln_equip, pln_equip): Fix the previous rev.

Doc fix.

(show_sect_capab, mission_pln_equip, pln_equip): Change i_type
references to use I_NONE instead of zero.  Zero has been invalid since
item.h rev. 1.12, and was questionable style before.

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

Still more 4.2.21 changes.

(pr_player, upr_player): Blocking I/O on another player's struct iop *
is unsafe!  By the time the blocked thread wakes up, that player may
be gone, along with its struct iop *, and io_write() follows a
dangling pointer.  Moreover, at most one thread may use empth_select()
on the same file descriptor.  Violations of that restriction cause
threads to hang under Windows since ntthread.c rev. 1.15.  Make all
output to another player non-blocking for now.  Historically, player
threads sent output only to their socket, though their own iop.  This
was broken by flash and asynchronous telegram notification a long time
ago.

(empth_init) [_WIN32]: Change flags to global_flags to prevent
scope conflict with the parameter flags.

Document Windows lossage and work-around.

Doc fix.