Still more 4.3.2 changes.

(info.nr/all, info.html/all.html, sources.mk): Pass LC_ALL=C in the
environment to make output independent of user's locale.

Only L_SUPPLY units can't attack as of 4.0.0, fix info Attacking,
remove dead code.

Update known contributors comment.

Remove redundant declaration.

(sources.mk): Sort file names to make output deterministic.

(guerrilla): Remove unused variable.

(guerrilla): Round security unit raid value below one to zero instead
of one.  Before, an empty, inefficient unit could be used to kill one
rebel per update.

(guerrilla): Security units' raid was broken in 4.0.0: the number of
che killed was 100 times too small, thus zero for all practical cases.
4.2.7 attempted to fix it, but left it broken for efficiency < 100.
It also increased deadliness of 100% units by a factor of 2.5.
Restore pre-4.0.0 behavior.

More 4.3.2 changes.

(player_addrlen): Change type to size_t.  Fixes type error in
player_init(), which was most likely fatal on 64 bit hosts.  Broken in
rev. 1.34.

(snxtitem): s_char purge.

Fix char / s_char inconsistencies.  Build was broken on some systems
since the first one crept in with src/lib/subs/retreat.c rev. 1.34.

Update for 4.3.2.

(xdvisible): Add case EF_REALM.  Should have been added along with
EF_REALM (file.h rev. 1.52).  This made xdump leak everybody's realms.

(login_coms): Remove erroneous declaration.  Move definition before
uses.  Add required function declarations.

(map, sct, rout): Remove unused local variables.

(snxtsct): Iteration order for `*' disclosed the real origin.  Fix.
Mapping commands used to work around this bug, until map.c rev. 1.18,
rout.c 1.18, sct.c 1.17, surv.c 1.18 removed the work-around and thus
exposed the bug.

(login_coms): Make declaration consistent with definition.

Fix missing include.  Closes #1464069.

(att_prompt): Do the right thing when player aborts.  Simplify.

(CANT_REACH): New.  Use it instead of CANT_HAPPEN() where possible.
(oops): Cope with NULL argument passed by CANT_REACH().

Fix long lines.  No functional changes.

(XYOFFSET): Parenthesize macro parameters in expansion.

Trim superflous casts and parenthesis.

(noise): Drop second parameter, it is always true.

(getin): Confusing.  Redesign.
(edit): Simplify.

(doland): Spelling fix.

(doplane): Don't arbitrarily truncate value for key 'a' and 'd'.

s_char purge.

Bump version to 4.3.2.

Don't explicitly dereference function pointers in calls.  No
functional changes.

Fix long lines.  No functional changes.

Final polish for 4.3.1.

(sail): There is no qsail command, remove test for it.  Test the
second argument only when the command is sail.  Before, `unsail 42 q'
showed the sailing path instead of clearing it.

Update for 4.3.1.

(verify_row): uid check was too strict.

Comment grammar fix.

New section on booleans.  Elaborate thread safety.

Fix the previous revision.

(bestownedpath, owned_and_navigable): Terrain parameter isn't really
used, remove it.  Use null bigmap argument to ignore terrain (flying),
non-null for navigation.  Callers changed.

(bestownedpath): Return NULL on long path instead of "?".  Terminate
returned path with 'h'.  This makes BestShipPath() and BestAirPath()
more similar to BestLandPath().  Simplify callers.

(BestAirPath): Some callers assumed non-null return value must be the
first argument.  Fix them.  Simplify bestownedpath().

(owned_and_navigable): Rework rev. 1.18: do check the real sector, but
only if it's owned or allied.  Check it before the bmap, and do it
just like shp_check_nav() does (should use common code).

(getpath): Supply the missing code for P_SAILING.  This makes the sail
command accept destination sectors as well as paths.  Clean up the
case discrimination.

(P_NONE, P_WALKING, P_FLYING, P_SAILING, p_mode): Turn into new
enumeration p_mode.  Users changed.  s_char and register purge.

(tend): Fix rev. 1.20: used to send bulletin to last tendee's owner,
unless that's the player, regardless of success.  Send them for every
successful tend to a foreign ship.

(build_mission_list_type): Simplify.

(show_mission): Plug memory leak.

(mission): Silence compiler warning.

(mission): Plug memory leak.  s_char purge.

(iop): Remove unused members assoc, notify.
(io_open): Remove unused parameters assoc, notify.  Caller changed.

(main): Remove superflous calloc().  s_char purge.

(bestownedpath): Fix test of end sector for terrain string starting
with 'R'.  Broken in 4.2.2.  No such terrain strings occur.

(owned_and_navigable): Do not check the real sector, only the bmap.
Checking the real sector leaks information, which can be abused to
test whether an arbitrary sector is unfriendly or an inefficient
harbor or bridge, at the cost of some ship mobility (I think).
Initially broken ca. 1993, went through several changes, but the leak
survived them all.

(emp_config): Failed to call set_dirs() when default econfig file
doesn't exist.

(genlist): Move from include/mission.h to src/lib/subs/mission.c.

(genlist): Remove unused members x, y.

(genlist): Use void * for generic pointer members cp, thing.  Change
users not to cast.

(oprange): Oops on bad argument.

(as_data): Use void * for userdata member and userdata parameter of
callbacks.  Users changed.

(bp_init): Use semantically correct return type instead of s_char *.
Caller changed.

Remove superflous casts.

(move_ground): Drop first parameter of map callback, it's not used.
Callers changed.  s_char purge.

Type of gift()'s generic pointer parameter was fixed in the previous
revision of src/lib/commands/load.c, change callers.

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