Update for 4.2.16.

Remove spurious "Clean sandbox"...

Always report 0 minutes login time and 640 btus to ease diff-ing.

Use "Thu Jan  1 00:00:00 GMT 1970\n" as output for the ctime() hack,
instead of an empty string.

New, more comprehensive guidelines.  Compiled existing Empire
practice, rounded off with classical C usage, edited into a readable
form.  Work in progress.

First step to add testing of the server to the nightlybuild.
For now, just some simple checks are added.

Patch to make ctime return "" for ease test output diff-ing.

(upd_buildeff): Previous revision broke work after sector
construction.  Closes #972065.

4.2.15 last minute changes.

Poorly structured and not entirely accurate.  Rewrite.  Closes
#934465

Didn't quite match the code and was confusing.  Rewrite.

Fix missing include.

(nstr_comp): Fix handling of condition type mismatch.  This bug could
trigger oopses in nstr_exec().

(cond_type_mismatch): Clearer diagnostic message.

(rada): Fix embarrassing coding error in previous rev.

Update for 4.2.15.

(doship, doplane, dounit): Fix warnings introduced by previous rev.

(shp_set_tech, pln_set_tech, lnd_set_tech): New, factored out of build
and upgrade code.
(build_ship, build_plane, build_land, supgr, pupgr, lupgr): Use them.
Upgrading planes no longer sets plane range to maximum.

(pupgr): Just clear the mission, don't bother to clear op area.

(doship, doplane, dounit): Editing tech now updates stats, like an
upgrade command.  Proper range checking.
(warn_deprecated): New.
(pr_ship, doship): Deprecate key 'D'.
(pr_land, dounit): Deprecate key 'A' and 'D'.

(move): Don't ask user whether he wants to abandon the sector when
running on behalf of the test command.  Closes #935470.

(explore): Clear start sector's MOVE_IN_PROGRESS after updating end
sector.  Otherwise, clearing MOVE_IN_PROGRESS can abandon the start
sector when it shouldn't, leading to bogus records in the `lost' file.
Closes #931184.

(dship, dounit, doplane): No longer accept efficiency values that
would destroy the edited unit.  Deities can still delete a units by
setting the owner to zero.  Efficiency values less than 10 used to
delete the unit without informing the owner.  Ship efficiency values
between 10 and 20 did inform the owner, but didn't give him any clue
about the reason.

(SCT_BLD_WORK, SHP_BLD_WORK, PLN_BLD_WORK, LND_BLD_WORK,
NUK_BLD_WORK): New, to encapsulate the formula in one place.

(SCT_MINEFF): New.  Use it instead of literal 20.

(buil): Use CANT_HAPPEN() for internal error.

(build_ship, build_land, build_bridge, build_nuke, build_plane,
build_tower): Simplify avail calculation.

(lupgr, supgr, pupgr): Charge 15% of total build avail instead of an
undocumented amount computed from build lcm and hcm.

(build_ship, build_land, build_bridge, build_nuke, build_plane,
build_tower, lupgr, supgr, pupgr): Fix reporting of required avail
when there's not enough.  Closes #942823.

(lupgr, supgr, pupgr): Code printed cost rounded down, but required
and charged unrounded cost.  Confusing.  Round cost down, like the
build command.  Closes #942811.

(UPGR_COST, UPGR_EFF, lupgr, supgr, pupgr): New UPGR_COST, UPGR_COST replace
literal values.

(show_bridge, show_tower): Don't claim bridge building requires
`workers' (whatever that is), since it doesn't.

(eff_bomb, sect_damage): Damage avail just like efficiency.

(do_feed): Rev. 1.4 accidentally limited avail after rollover to 999,
even when option ROLLOVER_AVAIL was off.  Previous revision correctly
limits rollover to rollover_avail_max, but missed the limiting after
rollover.  Fix.

(prod): Change variable work from double to int.  This matches actual
production code better, and fixes a small inaccuracy introduced in
rev. 1.10.

(prod): Production is limited to 999 units.  Fix for production
efficiency > 1, e.g. agribusiness.

(prod): Raw material consumption was incorrectly limited to 999.

(prod): Production backlog was handled incorrectly; it claimed `will
make' lots, `max' less, which is backwards.

(prod): Match materials_cost() more closely.  No functional changes.

(produce, prod): Remove some code related to depletion that has no
effect and makes no sense.

(prod): Rename variables to match produce() more closely.  No
functional changes.

(new_work, do_feed, prod, newe): Factor out calculation of work into
new_work().

(ROLLOVER_AVAIL, opt_ROLLOVER_AVAIL, opt_ROLLOVER_AVAIL, Options,
rollover_avail_max, configkeys, vers, new_work): Replace option
ROLLOVER_AVAIL by configuration parameter rollover_avail_max.

Should have been committed yesterday, together with sona.c.

(multifire): After charging ship mobility (option NOMOBCOST off), the
code sometimes failed to call putship().  Firing at ships was usually
free.

(M_TORP_SHELLS): New.
(perform_mission, torp, fire_torp, quiet_bigdef, use_ammo): Use it.
Closes #917459.

(do_desi): Don't complain unless FOR_REAL.  Closes #923105.

(stre): Always report mines to deities.  Closes #917911.

(opt_SHIP_DECAY, Options, shiprepair): Remove option SHIP_DECAY.

(do_desi): Zap efficiency only when sct_type really changes.  Closes
#935750.

(line_of_sight): Normalize coordinates before indexing.  Closes
#950514.

(line_of_sight): There's just one straight line between two points on
a plane, but on a torus there are four.  Code assumed plane, and thus
could screw up near the origin.  Pick a shortest line of the four.
Partial fix for #950510.

(nstr_exec, nstr_comp_val, nstr_exec_val): Values of type NSTR aren't
implemented and thus cannot occur.  Document this more clearly.

(nstr_comp_val): More logical order.  No functional changes.

Remove superflous include.

(flash, wall): Long arguments could smash the stack; fix insufficient
buffers.  Closes #940792.

(wall): Don't strip first word from message.  Closes #940790.

Don't try to let patch read from unreadable or nonexisting files.

(comstr, buy, check_market, display_mark, reset, sell): Change comstr
member com_type from mnemo character to item type.
(whichitem): Unused, remove.

(check_market): Use full item name instead of mnemo in telegrams.

(display_mark): Separate arguments for item type and cheapest only.
Cheapest only of specific item type is not implemented.

(rese): Guard against bad com_type.  Delete some code that had no
effect.

(commfil, comminit, make): Unused, remove.

(buy, mark, display_mark): Move argument evaluation from
display_mark() to callers.  buy() no longer accepts "all".  It used to
display all lots, but didn't let you buy any.  Similar nonsense
happened for "" if buy() prompted for it.

(display_mark): Fix size of cheapest_items[].

(player_coms): Document mark accepting "all".

(i_packing): New.  Use where appropriate.
(NPKG, WPKG, UPKG, BPKG, NUMPKG): Turn macros into enumeration
constants.
(NPKG, WPKG, UPKG, BPKG): Move from sect.h to item.h.

(iop, iop_t): Unused, remove.

Remove types that no longer exist.

(player_coms): Fix move and test syntax.

(item_by_name): New, factored out of whatitem().  Use it to simplify
some uses of whatitem().

(build_ship, orde, qorde, nav_loadship, load_it, unload_it): Store
item types instead of mnemo characters in shpstr members shp_tstart[]
and shp_tend[].
(com_num): No longer used, remove.

(orde): Simplify swap code.

(prhold): New, factored out of qorde().

(I_NONE): New.  To be used as item type error value and such.

(getstring, getstarg): Use plain char * instead of s_char *.

(ac_doflak, ac_shipflak): Simplify, no functional changes.

(prod, upd_buildeff): Truncated work to even before sector building.
Fix.

(prod): The production limit due to work was rounded incorrectly.

(OFFSET): Remove, use standard offsetof().
(SETOFF): Unused, remove.

(strtox, strtoy): New.  Use it instead of inputxy() to avoid the
double-remainder problem: x-coordinate SHRT_MAX+1 is truncated to 0 by
cast to coord, then converted by xabs().  This is wrong unless WORLD_X
divides SHRT_MAX+1.
(sarg_xy, sarg_getrange, sarg_range): Use them.
(inputxy): No longer used, remove.

(sarg_type): Use NS_UNDEF instead of 0.

(sarg_list): Change confusing loop control.  Properly diagnose
overlong lists; used to silently ignore list tail and return MAX+1,
which made a later snxtitem_list() fail.

(atoip): No longer used, remove.  Parsing was broken anyway.

(sarg_type, sarg_xy, sarg_area, sarg_range, sarg_list, sarg_getrange):
Use plain char * instead of s_char *.

(NS_UNDEF, NS_LIST, NS_DIST, NS_AREA, NS_ALL, NS_XY, NS_GROUP): Turn
macros into enumeration constants.
(ns_seltype): New.  Use where appropriate.

(snxtitem): Assigning NS_UNDEF to NP->type is an application domain
type error not reflected in C.  The former is a selection type, the
latter is a file type.  Harmless, as the variable is overwritten on
all paths to successful return.

Use CANT_HAPPEN() rather than logerror() for some internal errors.

(weekday, daytime, daytime_range): New; replace kw_parse().  Parsing
is somewhat stricter.
(is_wday_allowed, is_daytime_allowed, is_daytime_nar,
min_to_next_daytime): New.
(demand_update_time, scheduled_update_time, next_scheduled_time,
update_time, gamehours): Use them to replace kw_parse().
(kw_parse): No longer used, remove.

(gamehours): All callers ignore the value assigned to hour[].  Remove
parameter.  Remove calls without effect.  Other callers changed.

(getminleft): Parameter hours unused since the previous rev., remove.
Change parameter mpd from int * to int, because its only use is *mpd
as r-value.  Callers changed.

(kw_read, kw_find, kwtab, kw_list): Were used to evaluate the hours
file until 4.2.7.  Since then kw_read() is unused, and kw_find()
always returns NULL.  Remove.  kw_find() callers changed.

(player_coms): Fix survey syntax.

Incomplete, inadequate, rewrite.

Selector rewrite: values other than long, interpret identifiers
according to context, to make `lstat * ?type#spy&spy>1' work.  Closes
bug#825363, #905809, #905814 and #922968.
(nsc_type, packed_nsc_type, nsc_cat, packed_nsc_cat, nsc_flags): New.
(valstr): New.  Old code encoded values in type long, which was
somewhat hard to read and could only support signed integer values.
(nscstr): Redesign.  Use valstr.  Typed operator.
(castr): Split ca_code into ca_type, ca_flags, ca_off.  Tables
changed.
(nstr_comp, nstr_exec): Redesign and rewrite.  Callers changed.  They
used the old design incorrectly, which let players smash the stack
by supplying more than NCOND conditions.
(encode, nstr_comp_val, decode, nstr_exec_val): Rename, redesign, and
rewrite.  Callers changed.
(nstr_coerce_val): New.
(var_ca, sect_ca, ship_ca, land_ca): Checking both var_ca[] and the
object's ca complicates proper recognition of unique abbreviations.
Copy contents of var_ca[] into the ca of objects, remove var_ca[].

(surv): Reject values with category other than NSC_OFF and types that
can't be coerced to NSC_LONG.  Old code happily passed values with
category NSC_VAL to code_char().  The previous version interpreted
them correctly, but earlier versions interpreted them as NSC_OFF, then
logged `bad type in decode: 0' and evaluated them into zero.

(code_char): Used to test category NSC_VAR to decide whether to
display tens or hundreds.  NSC_VAR no longer exists.  Test type
instead.  Makes more sense anyway.

(debug): New.
(main): Set it on -d.
(oops, CANT_HAPPEN): New.

Firing from fortresses needs 5 mil, not 2.  Closes bug#923723.

Remove CVS keywords Source and Id.  No functional change.

A sandbox system to automate the building of the source tree and
reporting differences in compiler output with the output of the
previous build.

Don't shadow existing variables.  No functional changes.

(qprint): Change it into a printf()-like function.  Simplify callers.
No functional changes.  From Marc Olzheim.

(RANDOM, SRANDOM): New, to allow overriding the PRNG on the compiler
command line for reproducible automated tests.  This is expected to be
a temporary solution until we get more capable system configuration.
Callers of random(), srandom() changed.  From Marc Olzheim.

(rnd_seed, main, parse_args, allocate_memory): New option -R to set
the PRNG seed.  Print the seed.  From Marc Olzheim.

Use .ig to comment out stuff, it doesn't produce empty lines.

Implement request ig, except for argument.

Fix markup.

The previous revision screwed up intentation.

4.2.14 last minute changes.

Can't use gcc function attribute `format' when s_char is not plain
char; gcc chokes on it.

(configkeys): Rev 1.5 failed to update member km_func for keys s_p_etu
and adj_update.  From Marc Olzheim.

Fix RES_POP maximum population formula.

Document all undocumented capabilities and stats.  Clarify meaning of
land unit capability `light'.  Closes #781116.

Document submarine's ability to carry landspies.  Closes #907978.

Said `enemy civilians', but code happily lets you shoot your own
civilians.  Fix.  Closes #905200.

Claimed security troops shoot more than you asked for.  This is
wrong.  They save you mobility.  Fix.  Closes #781117.

Delete lie about Steve still heading Wolfpack.  Closes #902831.

Update for 4.2.14.

Fix 4.2.10's release date to match doc/CHANGES4.X.

(lwpStackCheckUsed): Fix confusing output.

Should have been removed along with the commands.

Command has been commented out for ages.  Remove until it isn't.

Regenerated.

(main, write_newcap_script): Don't warn when resources were added,
that's expected nowadays.  Warn when resources were not added.  Don't
mention the ore program, as it doesn't work and we no longer
distribute it.

(gen_power): Force mtime update under Windows.  From Ron Koenderink.

Update to match code.  There's currently no command that allows trade
and commodity selectors; comment them out.

(loan_ca): Remove selector "sell".

Talks about files that were removed.  Remove.

(growfood): Simplify.  Rev. 1.6 made it less robust against overflow;
fix that.

(produce): Fix consumed materials when production hits the 999 limit.
This can't happen currently, because produce() never gets called with
work>999.

(produce): Fix rounding of consumed materials on production backlog.
Broken by the previous revision.

(buy, move, prod, dodistribute, produce): Obey ITEM_MAX.  Previous
change to move() was incomplete.

(play_list): Cast time_t to long for printing.