Another 4.3.3. change.

Tag 4.0.0 with the date of the r.g.e. announcement.

Update for current code.

(powe): Simplify.
(gen_power): Remove redundant parenthesis.

Fix rev. 1.31.

(powe): Rewrite computation of targets[] for power c using
snxtitem()/nxtitem(), for simplicity and more uniform syntax.  Country
names no longer work when specifying more than one country (snxtitem()
restriction).  As usual with nxtitem(), silently ignore unsuitable
items (the old code complained and failed).  Don't skip powerless
countries for power c.  Closes #1025607.
(set_target): Unused, remove.

Fix markup.

Update for 4.3.3.

Provide example, fix pasto.

(shiprepair, planerepair, landrepair): Simplify materials code.  No
functional changes.

(shiprepair, landrepair): Always return 1.  Change to return void.
Simplify callers.

(prod_ship, upd_ship, prod_land, upd_land): Move test for mineff.

(upd_land, feed_land): Move food resupply.

(landrepair): Don't putsect(), update code works directly on cache.

(upd_plane, planerepair): New, factored out of prod_plane().  No
functional changes.

(prod_plane): Simplify carrier repair code.  Oops on bad carrier.

(prod_plane): Rename plp to pcp, remove redundant desc.  Tally build
cost without rounding, like ships and land units do.

(prod_plane): Simplify test for moving satellite.

(effrange, torprange, fortrange): New.
(multifire, quiet_bigdef, torp, anti_torp, sd, sb)
(lnd_fort_interdiction, lnd_support, perform_mission, oprange)
(shp_fort_interdiction): Use them.  This fixes forts shooting below
FORTEFF in quiet_bigdef(), lnd_fort_interdiction() and
shp_fort_interdiction().  Change variables for rounded ranges to int.

(quiet_bigdef, sd): Don't scale ship firing range by efficiency.

(dd, sb): Simplify.

(sb): Internal linkage.

(perform_mission): Rename range2 to vrange.

Break lines more tastefully.

(nuke): Players need the uid to work with nukes, show it.  Show
members introduced in nuke.h rev. 1.19 and 1.20: nuk_plane, nuk_effic,
nuk_tech, nuk_stockpile.  Make output similar to the other unit
reports while we're at it.

(detonate): Change parameters from carrying plane to nuke and
airburst.  Leave taking nuke off the plane to caller.  Use coord for
coordinate parameters.
(pln_damage): Take nuke off the plane.

(detonate): Destroy the nuke.  This is required since arming doesn't
remove nukes from the nuke file anymore (arm.c rev. 1.15).

(nuk_postread): Update nuke location to match that of the plane
carrying it.  This is required since arming doesn't remove nukes from
the nuke file anymore (arm.c rev. 1.15).

(nuk_prewrite): Properly destroy nuke.  This is required since nuke
file contains individual nukes instead of stockpiles (nuke.h
rev. 1.19).

(coun_cmd): Multiple client commands could overrun player->client[].
Reported by Luigi Auriemma.

Remove unused variables.

(look_land): Use roundintby() instead of ldround() for int argument.

Remove redundant casts.

(knockdown): Destroy nukes on collapsing bridges.  Before, you could
rescue them by building a new bridge or arming them on naval planes.

(pln_prewrite): Destroy nuke along with the plane carrying it.  This
is required since arming doesn't remove nukes from the nuke file
anymore (arm.c rev. 1.15).

(player_coms): Break lines more tastefully.

(bp_lbcost): Don't convert the cost from double to float and back.
Simplify.

(hap_req): New.
(nati, populace): Use it.

(populace, nati): Change hap, pct from float to double, because they
are used as double.

(nati): Print happiness needed with just two decimals, like the other
levels.

(fuel): Change extra from float to double, because its value is always
computed and used in double.

(build_ship, build_land, build_plane): Compute initial efficiency in
double rather than float, because all its uses convert it to double.
Remove redundant casts.

(comm_bomb): Compute damage percentage in double rather than float,
because all its uses convert it to double.

Do not cast to float where default argument promotion obviously
converts the cast's result to double.  Such casts are ugly and may
lose precision.

Do not cast to float where usual arithmetic conversions obviously
convert the cast's result to double.  Such casts are ugly and may lose
precision.

(nukstr): Make it a struct genitem: add members nuk_effic, nuk_mobil,
nuk_tech, nuk_stockpile, nuk_opx, nuk_opy, nuk_mission, nuk_radius.
These have no effect, yet.
(build_nuke): Initialize them.
(nuke_ca): Add genitem selectors: effic, mobil, tech, group, opx, opy,
mission, radius.
(tran_nuke): Zap mission, just in case somebody invents missions for
nukes.
(trade_desc): Show tech and efficiency, for consistency with other
units.

Change nuke storage and commands to match other units:
(nukstr): Hold one nuke instead of a stockpile: replace members nuk_n,
nuk_types by nuk_type, nuk_plane.  Add nuk_land for completeness, like
nuk_ship it's not yet used.
(nuke_ca): Update accordingly: replace selectors number and types by
type and plane.
(build_nuke): Update accordingly.
(ndump, nuke): Update accordingly.  Output is no longer sorted by
location, and nukes in same location no longer share id.  nuke's extra
columns for nuclear plants are repeated for every nuke.
(trade_desc): Update accordingly.
(cede, grab_sect, check_nuke_ok, detonate, trade_nameof): Talk about
nukes instead of nuclear stockpiles.
(arm, disarm): Rewrite, split off new disarm().  Don't remove nuke
from the nuke file on arm.  Allow usual plane syntax, not just plane
number.  Change second argument from nuke type to nuke number.  When
plane is already armed, ignore nuke argument and rearm (broken in
4.2.6).
(tran_nuke): Update for changed struct nukstr, make as similar as
possible to tran_plane.  Change syntax to match transport plane.
(player_coms): Update arm, disarm and transport accordingly.
(prnuke, nuk_on_plane): New.
(nuk_add, nuk_delete): Stockpile management, remove.

Don't wrap global variables in a struct for no reason:
(LwpMaxfd, LwpNfds, LwpReadfds, LwpWritefds, LwpFdwait, LwpDelayq)
(LwpSelProc): New, matching members of LwpSelect, internal linkage.
(lwpSelect, LwpSelect): Remove.

(lwpReschedule): Use CANT_HAPPEN() and abort() instead of lwpStatus()
and exit() for fatal internal errors.

(lwpSelect): Use logerror() instead of lwpStatus() to report fatal
error.

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.