Remove redundant casts of argument to (prototyped) double parameter.

(starve): Prompted twice for sectors when first argument was empty,
fix.  Simplify.

(increase_mob): Call time() just once, the second call can only
introduce inaccuracy.  Simplify the loop; it still doesn't make much
sense.

(find_escorts, perform_mission): Simplify.

(fire_dchrg, fire_torp): Simplify.

(gen_power): Simplify.

(get_couval, get_outstand): Remove.  Unused since automatic loans were
removed (buy.c rev. 1.18, trad.c rev. 1.15), should have been removed
then.

Reorder function definitions.

(lload): Simplify.

(MAX): Remove redundant definition.

Fix another insufficiently parenthesized macro expansion.

(buil): Clarify.

Fix the previous revision.

(pln_arm): Change parameter mission from int to char to clearly
document its use.

Fix insufficiently parenthesized macro expansions.

Remove declaration that should have been removed in rev. 1.9.  Doc
fix.  register purge.

(sate): Use ef_byname_from() for second argument.

(TERR_MAX): New.
(terr): Use it.

(terr): When the territory number is bad, do not loop prompting, just
fail.  That's what the vast majority of commands do.

(retr, lretr): Readability fixes.

(bomb, drop, fly, para, reco): Simplify.

(islist): Simplify.

Normalize unsigned int to just unsigned.

Normalize long int and short int to just long and short.

(do_desi): Improve message when designation doesn't exist.

(ask_olist, att_fight): Oops on bad combat mode.  Fixes compiler
warnings.

(pr_ship, doship, pr_land, dounit): Remove keys deprecated in
rev. 1.20.

(prsect, doland): Deprecate key 'C'.

(pr_plane, doplane): Deprecate keys 'a' and 'd'.

(pr_land, dounit): Deprecate key 'X'.

(pr_ship, doship): Deprecate keys 'H', 'P', 'X' and 'Y'.

(docountry, dounit): Remove parameter farg.

(att_ask_offense, ask_move_in): land_answer[] is indexed by army
character, reduce size accordingly.

(att_prompt, ask_move_in_off): Remove dead code.

(att_reacting_units): Simplify.

(coastal_sea_to_land, coastal_land_to_sea): Remove unused parameter
des.

Remove dead code.

Fix long lines.  No functional changes.

(rea): Simplify.

(lowerit): Simplify.

(grin): Simplify.

(dist): Simplify.

(best): Rename local variable.

Doc fix.

(lnd_mess): Internal linkage.

(s_char): Remove.  Use signed char for small integers, plain char for
characters.

Recent s_char purge changed element types to s_char *, but missed
patterns malloc(N * sizeof(s_char *)), fix.

(cede_sect): Remove dead code.

(fly): Use void * for generic pointer dst_ptr.

(vis, visbuf): Rev. 1.15 changed base type from s_char to char.  This
is misleading, as the elements are numbers.  Change to signed char.

Bump version to 4.3.3.

s_char purge directed by compiler warnings.

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.