Whitespace fix.

No need to pass -nsob, it's implied by -kr.

Indentation fix.

(fly): Fix confusing line break.

(doland): Fix output (run-away replace?).

Fix previous rev.

(ca): Unused, remove.

(LINUXPTHCFLAGS): Really don't define __USE_BSD_SIGNAL.

(LINUXCFLAGS, LINUXPTHCFLAGS): Don't define lint and __USE_BSD_SIGNAL.
The latter can't be found anywhere in source or system headers.

(OSXCFLAGS): Don't switch gcc to cruft mode.
(OSXLFLAGS): Don't sweep undefined references under the carpet; link
with the right libraries instead.
(LINUXCFLAGS, LINUXPTHCFLAGS): Compile with symbols.

Correct info on mining harbors.  Closes #827442.

Fix BTU cost.  Closes #782956.

Delete lie about give without arguments.  Closes #804709.

Document effect of BLITZ.  Delete lies about the total number of
players display.  Closes #809867.

.SA starts a paragraph.

Spelling fix.

Properly escape `\'.  Fix info on sector capabilities.  Closes
#861874.

Fix off-by-one in arguments of substr() that ate first letter of .SY,
.EX and .L text.

Garbage collection to side-step sparse file descriptor problem.
(io_select, io_flush, io_iopfromfd): Unused, remove.
(io_open): Allow more than one open for the same file descriptor.
The check is the last remaining use of io_list[], and it's
unimportant.
(io_list, niop, io_init): io_list[] no longer serves any purpose,
remove.  Sparse allocation of file descriptors (deplorable misfeature
of Winsock, at least some versions) is no longer a problem.
(getfdtablesize): Unused, remove.  Was broken for Windows anyway.

(io_open, io_iopfromfd): Properly check argument FD to protect
io_list[] indexing.
(niop, io_init): Store length of io_list[] in new niop.
(player_accept): Checking the socket file descriptor here is now
redundant.  Remove.

(msl_hit): Tell victim coordinates of incoming missile.  Closes
#799873.  From Marc Olzheim.

(rpt): Fix rpt[N_NUKE_SSTOP]; not every sub-launched missile is
nuclear.  Closes #797330.  From Marc Olzheim.

(landmine): Add a missing call of putland().  Closes #682570.  Don't
complain about running out of things when the job actually completed
fine.  From Marc Olzheim.

(player_coms): Fix flag argument of map commands in c_form.  From Marc
Olzheim.

(map): Reject flags `t' and `r' unless it's a bmap command.  Both used
to change the map command into the corresponding bmap command, which
was undocumented and didn't make sense.  From Marc Olzheim.

(wripglob): Don't emit unnecessary includes.

(wrgamesdef): Generate EMPDIR instead of EMPPATH().  Change callers of
EMPPATH() to use EMPDIR.  This avoids triggering MSVC bugs, and drops
support for traditional, pre-ISO cpp.  Closes #753935.
(gamedir, lostfil, playerbin, updatebin): Unused, remove.

(payo, scuttle_tradeship, sdump): Disclose construction site only to
original owner and deities.  Closes #815197.

(KILL): Off by one in fnlist[].  This is why client option -k didn't
work.
(WAT, GKILL): Unused, remove.

Remove outdated version information for now.  Keeping it up-to-date by
hand is too much work; should be automated.

Protect space between option and option arg, so that typesetting doesn't
swallow it.

(do_mob_land): Do not regenerate mobility faster while it is negative.

Add missing pages on sbmap, pbmap, lbmap.  Document flags * and h
where that was missing.  Make all pages fully consistent.  Form Marc
Olzheim.

(file_lock, file_unlock, ef_lock, ef_unlock, commlock, communlock):
File locking doesn't make sense, must be a leftover from the distant
past.  Remove.  Callers changed.

(fort): Refuse to fortify land units embarked on land units.  Properly
report failure to fortify.

(lnd_fortify): Do not fortify units embarked on ships or land units.

(take_move_in_mob): Automatic fortification on excess mobility may
fortify units while they are `at the prompt'.  That's okay.  But
action that consumes mobility must zap the fortification again.

(player_coms): Fix c_form for upgrade.  From Marc Olzheim.

Use strict & warnings.  Fix several minor bugs uncovered by it.

Semantics of assigning to $[ changed in Perl 5, and its use is `highly
discouraged'.  Programs obviously assumed $[ affects all arrays, but
it doesn't in Perl 5.  If these programs ever worked, then certainly
not with Perl 5.  Remove the assignments and shift indexes
accordingly.

Document automatic fortification.  Clarify engineer bonus.

(guerrilla): Give wandering che inexact, somewhat randomized
information on military.  Closes #750533.  Why?  Wandering che pick a
sector with minimal military from the adjacent sectors that can get
che.  Ties are broken in favour of the first sector examined.  Since
sectors are examined in the order given by by diroff[], che prefer
directions with low index in diroff[].  This tends to channel che
towards the northeast.  But the change makes ties very unlikely.

(lnd_sweep, lnd_mar_one_sector): Automatic fortification on excess
mobility may fortify units while they are `at the march prompt'.
That's okay.  But action that consumes mobility must zap the
fortification again.

(do_mob_land): Use excess mobility to fortify.  Note that units
requiring fuel to gain mobility never get excess mobility.  Adapted
patch from from Marc Olzheim.  Closes #725457.

(is_engineer, has_helpful_engineer): Rename.  Require allied
engineers.  This affecs command `fortify'.

(lnd_fortify): New, factored out of fort().
(fort): Use it; no functional changes.
(is_engineer): Move to logical place, external linkage.

Remove declarations of stuff that doesn't exist.

Fix typo.  Closes #824252.

(player_new): Proper error checking.  Oops, now it may return NULL.
Proper error checking in callers required.
(update_wait, market_update): If player_new() returns null pointer,
skip update.
(player_accept): If player_new() returns null pointer, close
connection.

(map, draw_map, MAP_HIGH): New map command flag `h' to highlight own
sectors; from Marc Olzheim.

(sabo): Damage report to player was in deity coordinates, fix.  Closes
#856856.

Don't explain sector production twice; replace one copy by a
reference.  Document execution of ship orders and sail paths.

(ROLLOVER_AVAIL, opt_ROLLOVER_AVAIL, Options, do_feed, prod, newe):
New option ROLLOVER_AVAIL; from Drake Diedrich.

[apollo]: Tables were gutted for unknown reasons.  Don't.

[_WIN32] Fix previous rev.

(empth_exit): Fix shutdown on reading "quit" from stdin (crash bug,
reportedly).

Fix previous rev.

(io_select, io_init, io_open, io_close, io_output, io_flush, io_write,
io_puts, iom_create, iom_getmask, iom_set, iom_clear, iom_zero):
Portability bug: io_select() passed bit_fdmask to select().  This
breaks the fd_set abstraction, and works only with the traditional
Unix implementation of fd_set.  Use fd_set and its operations instead.
Remove unused source files.  Note: the offending code is currently
unsued.  But it's a time bomb, and fixing it also replaces a bunch of
code by standard library services.

Unused, remove.

Merge Makefile rev. 1.5.  Update dependencies.

Portability fixes: don't declare library functions, include
appropriate headers.  Also remove some unused declarations.

(main): Portability bug: code passed bit_fdmask to select().  This
breaks the fd_set abstraction, and works only with the traditional
Unix implementation of fd_set.  Use fd_set and its operations instead.
Remove unused source files.

Fix previous rev.

Regenerated.  Should be automatically generated by make and removed
from the repository.

(player_coms, lstats, lsta): Rename command `lstats' to `lstat' to
match info as well as related commands `sstat' and `pstat'.  Rename
lstats() to lsta() to uphold the convention that functions
implementing commands are named like the command or its four character
prefix.  Closes #804587.

(player_coms, best): `bestdistpath' and `bestpath' violate the rule
that commands must be unique within the first four letters.  best()
implements both.  It calls BestDistPath() for the former, and
BestLandPath() for the latter.  However, BestDistPath() is just a
wrapper around BestLandPath(); it does exactly the same.  Drop
`bestdistpath'.

(mineq): Rewrite.  mineq("dumpcrap", "dump ") now yields ME_MISMATCH
instead of ME_PARTIAL.  Closes #804611.

(index, rindex): Obsolete BSDisms; remove.  Use standard strchr() and
strrchr() instead.

(ask_olist): Fix reporting of mobility required for attack.  Closes
#798045.

(do_desi): Move check for redesignated capital behind the actual
redesignation.  Closes #803768.

(index, rindex): Obsolete BSDisms; remove.  Use standard strchr() and
strrchr() instead.
(setbuffer): Obsolete BSDism, unused; remove.

(bzero, bcopy): Obsolete BSDisms; remove.  Remove some calls without
effect.  Replace calls by struct assignment where possible.  Replace
clear buffer, copy string to buffer by strncpy().  Use assignment to
clear when that's clearer.  Replace overlapping copy through bounce
buffer by memmove().  Replace rest by standard memset() and memcpy().
Also use sizeof() instead of literal array sizes for robustness, and
instead of symbolic array sizes for clarity.

Fix missing include <string.h>.

(copy, atopi, same): Remove, use strcpy(), atoi(), strcmp().  Callers
changed.  Remove empty source files.

Remove superflous include.

(ffs): Unused, remove.

(login): Fix reading of country name and, if we lack getpass(),
password.  Used to strip \r from country name (superflous), and not to
strip \n from password (required).

Fix previous rev.

(wrauth): Emit example `nowhereman@nowhere.land.edu' commented out.
nowhere.land.edu doesn't exist right now, but it might pop up any
time.

(info): Use ANSI strerror() instead of non-portable sys_errlist[].

(logerror): Remove unused cruft.

Fix title.

(at_minimum, blocksig, emp_bitinit, filelogerror, iceil, ifloor,
mapdsq, mask, signame, strscan, sys_siglist, typed_wire, unblocksig):
Unused, remove.

(add_airport, add_to_fired_queue, air_damage, anti_torp, best_path,
bitinit2, blankrow, bmnxtsct, bp_init, build_mission_list,
build_mission_list_type, buildeff, candchrg, canshoot, cantorp,
cede_sect, cede_ship, checksect, client_cmd, coun_cmd,
deity_build_land, divide, do_conv, do_defdam, do_demo, do_desi,
do_mob_land, do_mob_plane, do_mob_sect, do_mob_ship, do_prod,
docountry, doland, doplane, doship, dosupport, dotsprintf, dounit,
empth_start, enlist, errcheck, filereport, find_airport, find_escorts,
findcondition, fire_dchrg, fire_torp, fltp_to_list, get_minimum,
getin, goodsect, grab_sect, grab_ship, grow_people, growfood, in_addr,
infect_people, init_mchr, init_pchr, init_plchr, is_engineer,
itemname, kill_cmd, landrepair, limit_level, list_cmd, lupgr,
materials_charge, materials_cost, meltitems, mission_pln_arm,
mission_pln_sel, move_map, nat_cap, num_units, outid, pass_cmd,
perform_mission, pinflak_planedamage, play_cmd, pln_equip, prexpense,
print_res, prplanes, prunits, pupgr, quiet_bigdef, quit_cmd, radmap2,
retreat_land1, retreat_ship1, s_commod, sanc_cmd, sarg_getrange,
scuttle_it, scuttle_land, set_target, share_incr, shiprepair,
shp_check_mines, shp_check_nav, shp_check_one_mines, shp_hit_mine,
shp_interdict, shp_mess, sort_lookup_list, starvation, supgr, swap,
take_casualties, trunc_people, upd_buildeff, upd_land, upd_ship,
use_ammo, user_cmd): Change linkage to static.

List generated files that CVS doesn't ignore by default.

Cosmetics.

(show, show_item): New `show item'.  From Ron Koenderink; changes to
info files heavily hacked by me.

Spelling fixes.

errno may not be a variable; users must include errno.h instead of
declaring it like `extern int errno'.

(setfdtablesize): Remove.  This may drop support for Sequent.

(getfdtablesize) [hpux]: Client and server had different versions.
Standardize on server's.

Fix previous rev.

Makefiles worked only if certain directories and generated files were
already present.  These files are no longer in CVS.  Fix Makefiles.

(show, show_item): New `show item'.  From Ron Koenderink; changes to
info files heavily hacked by me.

Generated file, shouldn't have been committed.

(edit, prsect, doland): Fix `editing' sector coordinates (copies
sector).  New: edit distribution sector, designation and new
designation.  From Ron Koenderink.

(update_policy_check, diffx, diffy, takeover_plane): Prototypes were
missing.

Fix some damage done by indent-emp.  Code is still hard to read.

Remove pragmas that promote certain serious warnings to errors on
windows.  For obvious reasons, such warnings ought to be fixed.
Making compilation abort is the sledgehammer method to force that.
It's a stupid method for us, as we develop on various platforms, and
don't test every single commit under Windows.  It gratuitously breaks
programs that work fine.  Leave them as warnings, and fix them on
sight.

(edit): Rewrite of undocumented and broken feature: if there are extra
arguments, they are interpreted as key-value pairs, and edit does not
prompt for more.  From Ron Koenderink.
(player_coms): Document extended syntax.

(sorde): Output format fixes from Ron Koenderink.

(LND_SPY_DETECT_CHANCE): New.
(look_land, sabo, prunits, unitsatxy, lnd_mar_one_sector,
count_sect_units, takeover): Use it.  Closes bug#786369.

(decl): Remove broken and redundant check of declarer's country
number.  Check is redundant, because invalid values can come only from
natarg(), and those are checked above.  Check is broken because
natarg() returns negative value on error, not 255.