Bump version to 4.3.1.

(multifire): Retreat when helpless used RET_INJURED instead of
RET_HELPLESS.  Closes #1146364.

(retreat_ship1): Fix test for seamines.  Before, retreating ships
could sweep and hit landmines.  The bug fixed in rev. 1.35 is believed
to have masked this bug.  Charge mobility for sweeping.  Closes
#1346756.

(retreat_ship1, retreat_land1): Simplify.

(retreat_land1): Fix land mine hit chance (was too high).

(MINE_LDAMAGE): Change to match damage suffered in lnd_hit_mine().
This increases damage suffered in retreat.  Fix info Damage.
(shp_hit_mine): Use MINE_DAMAGE().
(lnd_hit_mine): Use MINE_LDAMAGE().

(shp_check_one_mines): Simplify slightly.

(retreat_ship1): Fix sector ownership test.  Before, retreating ships
could enter sectors owned by whoever caused the retreat, but not own
sectors.  Reported by Ray Hyatt.  Closes #1346750.

(command): Simplify mapping of condarg to lower case.

(lost): Simplify code for argument default value.

Cosmetics.

(map, rout, sct, surv): Remove special case for sects argument `*',
snxtsct() handles it just fine.

(map): Simplify slightly.

(player_coms): test needs C_MOD flag, because it calls draw_map()
through map().

(tran_map): Set player->argp[0].  Required since bmap() was merged
into map() ca. 1995.  Fixes 'm' at test's move_ground() prompt.
Simplify clearing player->argp[2].  Don't clear beyond that.

(tran_map): Set player->argp[0].  Required since bmap() was merged
into map() ca. 1995.  Clear player->argp[2].  Required since map
acquired its second argument in the early 90s.  Fixes 'm' at
transport's move_ground() prompt.

(unit_map): Don't use snxtsct(), it obeys conditionals.  Use
snxtsct_area().  Callers march() and navi() clearly don't want it to
use conditionals.  navi() used to zap them (just removed).  march()
doesn't, which was a bug similar to #785447.  It's less clear for
caller map().  I'd argue that conditionals apply to map's argument,
the unit number, not to the map itself.  draw_map() zaps them anyway.

(unit_map): Simplify.

(explore_map, nav_map, move_map, fly_map): Don't use snxtsct(), it
obeys conditionals.  Use snxtsct_dist().  Closes #785447.  Only march
and test were actually broken, the other uses zapped conditionals.
(move_map, fly_map, nav_map, navi): Don't zap conditionals.

(bsanct): Use snxtitem_all() instead of snxtitem().  The latter uses
player->condarg, which lets players keep sanctuaries.  Broken in the
early 90s.

(retreat_ship1, retreat_land1): Use snxtitem_group() instead of
snxtitem() for RET_GROUP.  The latter uses player->condarg, which lets
crafty attackers suppress retreat.

(conditions): Give sentinel a legible desc[], just in case.
(findcondition): Oops on hitting the sentinel.

(retreat_ship1, retreat_land1): Don't bother to map code to lower
case, it doesn't occur.

The previous revision is wrong, back it out.

The revision before the previous one is partially wrong, back out the
bad parts.

The revision before the previous one is wrong, back it out.

(file_sct_init, fl_sct_init): Fix parameter type from s_char * to
struct sctstr *.  Callers changed not to cast.

(retr, lretr, doship, dounit): Zero-terminate retreat paths.  The last
character in the array must be zero already.  Closes #1225708.

(sd): Supply missing parenthesis for a cast from double to int.  The
cast is only there to express intent, it doesn't change the meaning of
the code.

(shut): Fix misuse of logerror(): passed formatted string instead of
format string and arguments.  Correctness depends on formatted string
not containing '%', which is the case.

(show_sail): Fix misuse of pr(): passed formatted string instead of
format string and arguments.  Correctness depends on paths not
containing '%', which is correct, but not as robust as it could be.

(unit_interdict, knockdown): Simplify.  No functional change.

Code formatting.  No functional changes.

(show_nuke_capab, show_ship_capab, show_plane_capab, show_land_capab)
(prxy, border, ac_encounter, look, budg): Fix misuse of pr() and PR():
passed formatted string instead of format string and arguments.
Correctness depends on argument values and tables not containing '%',
which is correct, but fragile.

(comm_bomb, ship_bomb, land_bomb, pinflak_planedamage): Fix dangerous
misuse of pr(), PR() and mpr(): passed formatted string instead of
format string and arguments.  Crash bug if formatted string contains
'%'.  Players can put that in ship names...

(prunits, prplanes): Simplify, compute relation string instead of
format.

(prplanes): Fix dangerous misuse of pr(): passed formatted string
instead of format string and arguments.  Crash bug if formatted string
contains '%'.  Players can put that in country names...

(prod): Simplify printing of materials used.  Don't assume amount fits
into eight characters and item mnemo can't be '%'.

(head_describe): Print the headline instead of returning the format
string.  This simplifies format string checking.

(prthresh): Remove format string parameter.  Callers don't really need
that flexibility, and it makes format checking harder.

Remove useless multiplications with sizeof(char) and sizeof(s_char).
Replace by literal 1 in other contexts.

(gift): Fix parameter types: natid instead of int, void * for generic
pointer, char * for string.

(gift, load_comm_ship, load_comm_land): Fix dangerous misuse of wu():
passed formatted string instead of format string and arguments.  Crash
bug if formatted string contains '%'.  Players can put that in ship
names...

Remove silly superflous parenthesis.

(getele): Zero-terminate empty telegrams.  Returned telegram was
broken in rev. 1.9.  ~p never worked.

Fix land unit speed.  Reported by Pat Loney.  Fix spelling.

(buil): Simplify the loop for building multiple units.  Unit type
argument "" no longer makes it prompt for the unit type on every
iteration.

(buil): Return RET_FAIL on insufficient tech for bridge instead of
literal 2 (which is RET_SYN).  Return RET_SYN on incorrect first
argument.

(build_ship, build_land, build_plane): Compute cost in floating-point.
It's better to round just once, when subtracting player->dolcost from
nat_money.

(build_can_afford): New.
(build_ship, build_land, build_bridge, build_nuke, build_plane)
(build_tower): Use it.  This changes some messages.  Fixes ancient
bridge building race condition: while prompting for direction, another
build could clobber cash.
(cash): Remove.

(trade_check_item_ok): New.
(trade_check_ok): Use it and simplify.

(set): Use new trade_check_item_ok().  The code replaced by it didn't
work (abusable bug).

(ontradingblock, trdswitchown): Second parameter is a generic pointer,
change it from int * to void *.  Callers changed not to cast to int *.

(genstr): Redundant.  Replace by struct genitem.

Define feature test macro required for PTHREAD_STACK_MIN on some
systems, e.g. Solaris.

Fix the previous revision.

(empth_create): LWP yields, and it's the reference implementation.
Document and fix the others.

(u_char, u_short, u_int): BSDisms.  Figuring out whether sys/types.h
defines them would be possible, but isn't worth the trouble.  Replace
by unabbreviated types.

Use NULL instead of (FOO *)0, it's easier to read.

(dirchar, neighsects, login_coms, lookup_list, g_distptrs, level_easy)
(level_log, levelnames): Internal linkage.

(map): Simplify loop control.

(marc, navi): Permit any whitespace between route and optional
argument for radar, lookout, sonar, not just spaces.

(do_conv, enli, explore, move, shoo, s_commod): sct_mobil was changed
from u_char to short in 3.0.0, but many assignments still cast to
u_char.  Remove casts.

(do_mob_ship, do_mob_land): Remove superflous casts.

(do_mob_land): Remove superflous check for excess mobility.

Previous revision broke canal navigation, fix.

Bump version to 4.3.1.

Fix editing accident in revision before the previous.  More 4.3.0
polishing.

Update 4.3.0 date.

Terminology fix.

Hints on GNU make.

(CFLAGS, CPPFLAGS): POSIX make has only CFLAGS.  Fold CPPFLAGS into
CFLAGS.  Use CFLAGS when linking, to match POSIX make implicit rule.

Search for terminal stuff in -lcurses as well.

Correct reference to doc/xdump.

Refer to xdump.

(may_play): More helpful diagnosis and log.

(privip): Lacked the IPv4 mapped IPv6 address for localhost.

Use touch to touch stamp files, > doesn't update mtime of existing
file on some systems.  Weird.

(client): Don't make $(termlib) a prerequisite, pass it in $(LDLIBS).
The old version didn't work on some systems.

Final polish for 4.3.0.

Doc fix.

*** empty log message ***

(termio) [_WIN32]: Change remaining "n"s to "records" for the
WIN32 ReadConsole().  Fixes broken exec command.
Broken in Rev 1.17.

*** empty log message ***

Update.

Update for 4.3.0.

Update.

Cosmetics.

xdump's no longer experimental, update accordingly.

Caution deities on customization.

New doc/xdump.

Doc fix.

(ef_open): Zero cache size (empty EFF_MEM file) can lead to null cache
on some systems, which confuses assertions.  Force non-zero size.

AIX needs -lcurses for setupterm().

MSYS ships an obsolete version of GNU make which doesn't grok
order-only prerequisites.  They were used to make directories
info.html, info.nr, lib.  Make them ahead of time, in configure.  Also
fixes distclean not removing them.

Check for -lws2_32 properly.  The old code relied on $host_os, which
wasn't defined.

Doc fix.

Doc update.

(player_coms): Update xdump's c_form to current syntax.

Remove note on self-documenting econfig, as it's of historical
interest only.

Explain how to use custom_tables.

Update to current command syntax, cover meta.  Refer to doc/xdump (to
be written).  Less detail on output.  Document current shortcomings.

(xdump): Nicer prompts.

(sources.mk): Avoid harmless circular dependency.

Update known contributors comment.