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.

Document blocking constraints.

(loc_GVAR): Putting file local variables into a struct serves no
useful purpose whatsoever.  Peel off the struct.

(main) [_WIN32]: Windows putchar() screws up when printing multibyte
strings bytewise unless the stream is buffered.  Switch stdout to
line-buffered mode.
(login): Explicitly flush stdout, because Windows doesn't implement
line-buffering faithfully.

(loc_Thread, loc_Sem_t): Remove typedef, use the typedef name defined
in the header instead.

Remove some superflous casts.

(loc_Sem_t, empth_sem_t) [_WIN32]: empthread.h defines empth_sem_t as
incomplete type struct loc_Sem_t.  Instead of completing it here, the
code defined an unrelated type loc_Sem_t as (complete) type struct
empth_sem_t.  The resulting type errors were swept under the carpet
with casts.  Obvious fix.

(deliver): Do not deliver friendly civilians into occupied sectors.

Minor clarifications.

(io_write): Use IO_NOWAIT rather than 0.  No functional change.

New doc & doc fixes.  Remove s_char on the way.

(C_MOD): Turn into enumeration constant.

(empth_sleep) [_WIN32]: If the sleep returns early, go back to sleep.
Short sleeps can cause double-updates and other trouble.

[_WIN32] (loc_SleepThisThread): Not used, removed.

(create_pidfile): Silence compiler warning on printing pid_t with %d.

Document Empire's thread abstraction.  Minor cleanups:
(empth_init): Change first parameter to void **.
(empth_sleep): Change parameter to time_t.

More 4.2.21 changes.

Based on a page Doug Wescott created for his Petting Zoo.  I restored
the ASCII art to the original, gave credit to its creator, quoted from
WordNet (free) instead of American Heritage Dictionary, omitted parts
that might be illegal to distribute in certain countries, and improved
see also.

Doc clarification.

Remove some dead, disabled code.

A better than nothing joke.

Rewritten in an attempt to present all the initial revision's valuable
information in a more accessible form, and then some.

Protocol, session options and traditional dumps are complete, xdump is
a stub, parsing human-readable output is missing.

Update for 4.2.21.

(att_move_in_off): When boarding from a sector, mil entering the ship
less one were not removed from the sector.  The problem is putsect()
overwriting data from prior put_combat() with old data.  Broken in
rev. 1.17.  Change lunchbox code to match that of 1.16.  Closes
#1219918, reported by Doug Wescott.

Update for src/util/fairland.c:1.25 and include/prototypes.h:1.84

Update patch for pr.c:1.22

(copy_ascii_no_funny, copy_utf8_no_funny)
(copy_utf8_to_ascii_no_funny): Don't consider newline funny.  This
fixes output filtering broken in rev. 1.17.

Document use of UTF-8.

(player_login): Replace pr_player() by equivalent pr_id().
(pr_player, upr_player): Internal linkage.

(pr_hilite): Unused, remove.

Document, in particular use of UTF-8.  Simplify code in a couple of
places, but no functional changes.

(outid): Fix recovery from bad argument N.  Simplify.

(uprnf): A print function must not change its text argument!

(copy_ascii_no_funny, copy_utf8_no_funny)
(copy_utf8_to_ascii_no_funny): New.
(flash, wall, prmptrd, uprmptrd, getcommand): Use them to filter
input.
(uprnf, pr_flash): Use them to filter output.
(prtoascii): No longer used, remove.

(player_commands, player_commands_index): Internal linkage.

(upr_player): A print function must not change its text argument!

(lrange, shoo, execute): Buffer passed to getstarg() was too small.
Player could overrun it!

(player_login, player_main): Move the C_INIT that terminates the login
phase to player_login().  No functional change.
(showvers): Trivial, used just once.  Inline, remove.

(parse): Support 127 arguments instead of 100, to match
player->argp[].  Assign complete argument vector.  This should make
buffer overruns blatantly obvious.

(parse): Don't bother checking SPACE argument, just crash.  It is
extremely unlikely that such a programming error wouldn't be
discovered by even them most superficial testing.

(emp_config): Buffers passed to parse() were too small.  econfig file
could overrun them.  Closes #899546.

(dispatch): Argument buf is user text and must be printed as such.  In
error messages, print just the command name instead.

(play_list): player->combuf[] is user text and must be printed as
such.
(ufindbreak, ufindpfx): Rename, move to pr.c, external linkage.

(shut): Buffer passed to getstarg() was much too small.  Deity could
overrun it.

(shut): Fix retrieval of second argument.

(shut): Simplify formatting of broadcast messages.

(rea, turn): Use a different buffer for message text.  Just for
clarity.

(typed_wu): Treat overlong message as internal error instead of just
truncating it silently.  Simplify.

(getele): Abort telegram when uprmptrd() fails.  Previously, failures
that didn't set player->aborted sent the telegram, e.g. EOF.

(getele): Fix misleading size of left[].

(getele, tilde_escape): Simplify.  tilde_escape() now returns the
escape code instead of comparing it to second argument.

(turn): Buffer passed to getele() was one too small.  Deity could
overrun it by entering a message of maximum size.

(gamedown, show_motd): Fix message buffer size.  Messages of maximum
length were considered overlong and ignored.

Remove some redundant casts.

Remove a bunch of redundant casts.

(rea): Portability fix.

[_WIN32] (loc_StripDels): Remove.  Different behavior depending on the
platform is evil, and line editing is none of the server's business
anyway.  If your client can't deal with the delete key, well, here's a
nickel, kid, get yourself a better computer.

(ioq_gets): Support telnet line termination "\r\n".  Previously,
string input other than through parse() or getele() failed to strip
'\r', which could end up in ship names and such.
(getele): Code dealing with '\r' is now redundant, remove.  It ceased
to work in rev. 1.6 because input filtering replaces '\r' by '?'.

Fix markup.

Spelling fixes.

Fix the previous revision.

(sendmessage): Skip players not in state PS_PLAYING.  Flashes to them
get ignored, hence telling the player that the flash was sent is
wrong.  Skipping fixes that.

(mobupdate): Don't print '\r' after '\n'.

(play): Simplify counting of players.

(options_cmd): List options only if none are set.

(list_cmd, login_coms): Login command list is obscure and redundant;
remove.

(login-coms): Improve doc strings.

(nstr_string_ok): New.
(nstr_comp): Use it to fix oops on `des='.  Reported by Doug Wescott.

(getsose): Port from obsolete termcap to terminfo.  The old code
manually stripped off padding, which is evil.  The new code does
nothing when the stdout is not connected to a terminal.
(SO, smso, SE, rmso): Rename, static linkage.
(putso, putse): New.
(screen): Use it.

(hidd,peek): Rename hidden command to peek.
This prevents a conflict in the info files with
HIDDEN option.

Update for src/util/fairland.c:1.23 and src/server/main.c:1.62

Spelling fixes.

(program_name, help): New.
(main): New option -h.  Terminate unsuccessfully on invalid option.
(usage, parse_args): Factored usage() out of parse_args().
(parse_args): More useful diagnostics.

(prod_plane): Remove redundant initialization.

(sarg_type): Simplify, no functional change.

Doc & formatting fixes.

(flee, army, wing): Print how many new members were added.
Previously, it printed how many were selected, which can include old
members of the group.

(flee, army): The rather obscure feature to assign the fleet's /
army's retreat orders to members was broken.  It ignored ownership,
and thus could copy some other player's retreat orders.  Abusable.
Copying the first member's retreat orders is less than useful, in
particular for group ~.  The code now chooses the first one in the
same sector with RET_GROUP set.  RET_GROUP is never set for group ~.

Fix .SA.

Cosmetic change of markup.

Fix the previous revision.

Updated from man/emp_client.6 rev. 1.9.

Add notes for Client Implementors.

(output,screen): In ASCII mode, remove standout bit if the client
does not support standout mode.
In UTF8 mode, remove SO/SI characters if the client does not support
standout mode.

Add the See Also "Moving" to the commands described
in the info file.

Correct the number of commands in the described in the text.

Changed the name of Attack and Move concept files and
a number of Subjects files to prevent name
conflicts where the only the case is difference.
Attack -> Attacking
Moving -> Transportation
Move -> Moving
Market -> Commerce
Nation -> Nations
Production -> Producing
Update -> Updating

(EFF_OWNER): The assertion that EFF_OWNER implies ef_read() sets
player->owner is no longer used.  Use it to indicate that owner can be
accessed through struct genitem, similar to EFF_XY and EFF_GROUP.
(xdump): Fix ownership checking.  Old version worked only when
nxtitem() set player->owner.

(empfile): Set EFF_OWNER for EF_LOST.  This fixes `xdump lost'
disclosing other countries' losses.
(loststr): Document implications of EFF_OWNER.

(empfile): Set EFF_OWNER for EF_NATION.
(natstr): Rearrange members for EFF_OWNER, document.
(nat_ca): Unused so far.  Clean it up, add most missing selectors.
(fileinit): Use it.  This implements `xdump nation'.

(fileinit): Simplify setting map file size.

(nxtitem): Skipped unowned sectors (could not occur), ships, planes,
land units and nukes for selection type NS_LIST.  Thus, unowned items
explicitly selected by item number are skipped, while the same items
selected implicitely by group, area or whatever are not.  This makes
no sense.  Some callers break the nstr_item abstraction to prevent the
skipping, so they get all items regardless of how they were selected.
All other callers filter out unowned items.  This skipping misfeature
dates back to the very oldest known versions of Empire.  Remove it.
Also remove skipping prevention hacks from callers.

(tend_nxtitem): Now identical to nxtitem(), except it lacks some extra
normalization of item coordinates, which should have no effect.
Replace by nxtitem().

(findlost): Static linkage.

(LIBS, NTLIBS): Remove libsubs.a.

(main): Fix checking of ef_open()'s value.  Fixes program exit status
when files can't be opened.

(logerror): The version from libcommon.a does the right thing since
log.c rev. 1.17.  Use it, remove local version.

(log_last_commands): No longer referenced by library code, remove.

(ef_write): Don't call log_last_commands() directly, use oops().  If
it's worth logging commands, then it's worth dumping core.

(debug): Move to log.c, so other programs don't have to define debug
themselves just to use logerror().

Clean up the previous revision; no functional changes.

Update patch for some minor offset changes.

Update patch for pr.c:1.12

Remove some redundant casts.