(ef_open): Failed to reset empfile member fd on failure, and leaked
file descriptors.

(ef_flush, ef_close): Rely on negative fd instead of null cache to
detect unopened file.  This is more logical and consistent with the
use of cache by the new tables that are not file-backed (rev. 1.22).
Use CANT_HAPPEN().

(ef_close): Failed to report cash flush failure.

(ef_close): Clear all EFF_OPEN flags.  Should have been done in
rev. 1.27.

(do_write): New.
(ef_flush, ef_write): Use it.  Old code couldn't cope with interrupted
or short writes (mostly harmless, as disk reads commonly complete).

(ef_write): Use CANT_HAPPEN() for catching unimplemented table
extensions.

(fillcache): Rewrite.  Old version failed to check success of lseek(),
couldn't cope with interrupted or short reads (mostly harmless, as
disk reads commonly complete), and left an empty cache after a failed
read.
(ef_open): Use fillcache() instead of read().
(ef_read): Fail if fillcache() fails to read at least one element.
Old version copied garbage to its caller and claimed success.

(ef_open): Fail if file size is not a multiple of record size.

(ef_open, EFF_RDONLY, EFF_CREATE, EFF_OPEN): Replace mode parameter
(really open flags, not mode) by new flags EFF_RDONLY, EFF_CREATE.
Open flags were needlessly general.  Callers changed.
(empfile): Member mode now unused, remove.

Update for src/lib/commands/new.c:1.24 and include/prototypes.h:1.92

(START_UNITS, start_unit_type, deity_build_land, new): Remove
undocumented compile-time option START_UNITS.  It hasn't been used in
ages; writing a quick build script is easier than recompiling the
server.

Clean up the previous revision.

More 4.2.23 changes.

Update example to current output.

Remove option SNEAK_ATTACK leftovers.

(draw_map): Parameter bmap abused EF_MAP and EF_BMAP, and required the
silly EF_NMAP.  Use characters instead: 'b' for bmap (was EF_BMAP),
't' for true bmap (was EF_MAP), 'r' for revert (was EF_MAP+EF_BMAP,
more sillness), and 'n' for nmap (was EF_NMAP).  Callers changed.
(EF_NMAP): Remove.

[NOSTRDUP]: Drop support for systems without strdup().  First commit
was incomplete.

(map, draw_map): Remove undocumented feature that lets deities run map
as another country.  It appeared around Chainsaw 3.0 and never fully
worked.  See also #1335316.

(map): Fix rev. 1.4: it treated nmap as a bmap command.

(ef_close): Reset member fd.
(ef_open): Catch attempt to open open file.

(EFF_OPEN): New.
(ef_open): Restrict HOW to EFF_OPEN.

(ship_ca,land_ca,plane_ca): Add missing EF_BAD for radius row in
NSC_GENITEM.

(ef_open): Remove code killed by the previous revision.

[aix]: Remove special case include, I doubt it's necessary.

[_NTSDK]: Remove _NTSDK not needed.

(ef_open): Don't assume blksize() is the same for all files.

(ef_open): Ensure the cache has at least one slot.

(empfile, empfile_ca): New struct empfile member ef_uid.

(blksize) [aix, sgi]: Drop special case; I bet these systems support
st_blksize by now.

(wripglob): No need to include misc.h since rev. 1.16.

(ioq_destroy) [aix, NeXT]: Remove special case, which leaked memory,
because freeing it didn't work at some time.  Well, it either works
now, or we'll find out what's wrong.

Ultrix is no longer supported, remove special case.

[NOSTRDUP]: Drop support for systems without strdup().

(castr,empfile,ship_chr,plane_chr,land_chr,nuke_chr):
Add table index to castr to create a reference to the symbol table.
Add "ship chr flags" table to empfile.
Add "plane chr flags" table to empfile.
Add "land chr flags" table to empfile.
Add "nuke chr flags" table to empfile.
Add EF_SHIP_CHR_FLAGS reference to ship_chr.
Add EF_PLANE_CHR_FLAGS reference to plane_chr.
Add EF_LAND_CHR_FLAGS reference to land_chr.
Add EF_NUKE_CHR_FLAGS reference to nuke_chr.

(show_ship_capab,show_plane_capab,show_land_capab,
show_nuke_capab,ship_flags,land_flags,plane_flags,nuke_flags):
Move ship_flags nsc.c from show.c
Rename ship_flags to ship_chr_flags.
Move plane_flags nsc.c from show.c
Rename plane_flags to plane_chr_flags.
Move land_flags nsc.c from show.c
Rename land_flags to land_chr_flags.
Move nuke_flags nsc.c from show.c
Rename nuke_flags to nuke_chr_flags.
This is to accomodate future symbol table for EF_PLANE table.

(ef_init): Remove cadef member from fileinit,
insert directly into the empfile structure.
Add ef_type member to fileinit to identify rows with data.
Remove unused rows of fileinit.

(empfile[]) Add table of tables to empfile[].

(player_init): Fix initialization of sin.  Some systems have a member
sin_zero and choke when it's not zeroed.  INADDR_ANY wasn't converted
to network byte order, but since it's zero, the bug couldn't bite.

(io_open, io_noblocking): FNDELAY is long obsolete, use O_NONBLOCK.

(new): Fix initialization of realms for players_at_00.

(empfile, chr_camap): Merge chr_camap[] into empfile[].  No existing
user sees the new elements of empfile[].
(EF_SECTOR_CHR, EF_SHIP_CHR, EF_PLANE_CHR, EF_LAND_CHR, EF_NUKE_CHR)
(EF_TREATY_CHR, EF_ITEM, EF_INFRASTRUCTURE, EF_PRODUCT): New.  Not yet
used.
(my_ef_byname): Replacement for ef_byname() that sees the new
elements.
(xdchr): Work with empfile[] instead of chr_camap[].
(xdump): Use my_ef_byname() instead of ef_byname() and
chridx_by_name().
(camap, chr_camap, chridx_by_name): Remove.

Fix the previous revision.

Another 4.2.23 change.

(show_opts): Pretend option SHIPNAMES still exists for the benefit of
clients that still depend on it.

Don't use conditional compilation to time out obsolete stuff.  Do it
by hand.

Update for 4.2.23.

(do_conv) Remove CR.

(INTERIXCFLAGS, LINUXCFLAGS, LINUXPTHCFLAGS): Don't define
LISTENMAXCONN; it's unused since src/lib/player/accept.c rev. 1.26.
That revision's change log is quite confused.

(do_conv) Move the initialization of the natp to be inside sector loop, this
allows the deity to use the research level of the sector owner.  This also
fixes the use of an uninitialized sector data structure.

(ichr_ca,pchr_ca,dchr_ca,mchr_ca,plchr_ca,lchr_ca,nchr_ca,
tchr_ca,rpt_ca,intrchr_ca ) Move from src/lib/commands/xdump.c
to src/lib/global/nsc.c; external linkage.  They were created in
xdump.c to minimize initial impact on the source, but they
really belong to nsc.c, where other code can use them as well.

(pin_bomb) Remove unused variable qp.

More 4.2.22 changes.

Replace incorrect claim on morale_base by reference to info morale.
Reported by Doug Wescott.

(play_cmd): Log begin and end of playing session.  Closes #911285.

(kill_cmd): Log killing.

Doc clarification.

(babies): Err, the maxpop case wasn't an optimization...  Without
that, we get negative babies, which leads to negative avail and baby
food (cannibalism?) in grow_people().  Restore it.

(nreport): Simplify.

Doc fix.

(set): Fix initialization of trd_y.  The initial value is believed not
to be used.

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.