4.3.8 last minute changes.

(upd_ship): Don't produce stuff when broke, to match sector behavior.

Update for 4.3.8.

(switch_leader): Generalize the code, no functional changes.
(navi): Change switch_flagship() to call switch_leader() instead.
(switch_flagship): Remove, not needed, replaced with switch_leader().

(take_casualties): Code for distributing casualties among units did
not make sense, and in its confusion destroyed land land units when it
shouldn't.  Minimally invasive fix; code is still ugly and
incomprehensible.

(guerrilla): Make security units not raid friendly che.

(struct ulist): Add abstract reference to types.h to accomodate
the ciruclar reference between struct ulist and union empobj_storage
and struct lndstr.

Combined struct llist and struct mlist into superset struct ulist.
(assa, set_leader, switch_leader, set_flagship, switch_flagship,
take_move_in_mob, get_land, ask_olist, att_get_defense,
get_dlist, get_ototal, get_dtotal, kill_land, att_infect_units,
put_land, att_reacting_units, count_bodies, att_fight,
send_reacting_units_home, take_def, ask_move_in, move_in_land,
lnd_print, lnd_delete, lnd_take_casualty, lnd_takemob,
lnd_sel, lnd_mar, lnd_put, lnd_sweep, contains_engineer,
lnd_check_mines, lnd_list, lnd_mess,
lnd_damage, lnd_easiest_target, lnd_mar_one_sector, shp_sel,
shp_nav, shp_put, shp_sweep, shp_check_one_mines, shp_check_mines,
shp_list, shp_mess, shp_count, shp_damage_one, shp_damage,
shp_contains, most_valuable_ship, shp_easiest_target,
shp_missile_interdiction, notify_coastguard,
shp_view, shp_nav_one_sector, shp_missdef, nav_ship,
fltp_to_list): Switch to struct ulist from either struct mlist
or struct llist.

(out5): Rev. 1.36. didn't fully fix the format conditional.
Fold negative cases into positive ones.

(apro) [!_WIN32]: Silently ignore directory entries starting with '.'.
Before, apro() logged complaints about . and .. not being regular
files.

Clean up the previous revision.

Document stop order expiry.

(produce_sect): Game state changes may be changed only if
player->simulation.  Fix a missing guard.  Existing callers should not
need it.

(produce_sect, finish_sects): Move stop order expiry to end of update,
so that stopped sectors don't repair units.  The update sequence
masked this bug until its change in src/lib/update/main.c rev. 1.25,
but use of budget priorities (gone since rev. 1.24) could unmask it.

(sect_has_dock): NAV_CANAL in sect.h rev. 1.36 broke loading in
canals, fix.  Can't enforce M_CANAL condition here, so leave it to
callers.  Callers don't enforce it either, which means landlocked
ships can load in canals.  Oh well.

(player_main): Delay journal_login() until after error exits, to
ensure it is properly paired with journal_logout().

(tend_land): Fix to not let land units be tended that could not be
loaded.

Remove bogus paragraphs on RES_POP and NO_PLAGUE.  Clarify sentence on
max safe population.

Update for maximum population change in sect.config rev. 1.3.  Remove
incorrect claims about mil.

(genlist): Switch thing to use struct empobj * instead of void *.
Remove type from struct genlist as it can be determined from thing.
Users changed.

(struct trdstr): Change trd_type to be short  to be consistent with the
rest of the definitions of ef_type.

(empobj_chr): New, create a new (imcomplete) structure for
empobj characteristics.
(get_empobj_chr):  Update get_empobj_chr() to use new
struct empobj_chr.

(radar): New, moved functionality from rada() except ef_type determination.
(rada): Call radar() with EF_SHIP.
(lrad): New, call radar() with EF_LAND.
(empmod[]): Call lrad() for lradar command.
(march, navi): Call radar() with appropriate type.

(EF_IS_VIEW): New.
(xdump): Use it to deny access to views when denying access to game
state.  Fixes the previous revision.

(obj_nameof): Return safer error value.

(get_empobjp, get_empobj_chr, emp_obj_chr_name, get_empobj_mob_max):
Style.

Allow xdump before break; partial solution for #1417871:
(player_coms): Set xdump's c_permit to VIS.
(xdump): Deny countries that couldn't run xdump before access to game
state.

(ef_open) [_WIN32]: Add shared.h for MinGW environment,
needed for WIN32 file locking. Broken (missed) in rev 1.70.

(ef_open) [_WIN32]: Provide file locking for WIN32.

(rada): Replace the "unit" text with "land" to be more clear.

(rada): Using struct empobj to generalize the code.

(makelost, makenotlost, findlost, struct loststr): Change type to be a
short instead of a char.  This will make lost_type consistent with
ef_type in the rest of the structures.

(unit_map): Replace player->owner with direct owner check and
player->god check.  player->owner is not set if ef_ptr() is used
as no post_read() is done. Broken in rev 1.38.  Not completely fixed
in rev 1.40.

(ef_open): Lock the file.  This prevents two programs (servers in
particular) running on the same game state.  fairland does not use
ef_open() at this time, and thus ignores the lock.

(produce): The reported amount produced (returned through last
parameter) did not reflect production cut for ITEM_MAX.  Broken in
3.0.14 or so.  Reported by Stefan Hauser.

(gift): Remove the type argument, can be determined from the ptr
argument. Replace prship/prland/prplane/prnuke code with
obj_nameof().

(unit_map): Replace player->owner with direct owner check.
player->owner if not set if ef_ptr() is used as no post_read() is done.
Broken in rev 1.38.

(getpath): Remove check for DIR_MAP.
DIR_MAP is not a value that can be entered by the user.

(draw_map): Use struct empobj to generalize the code.

(map): Prompt only for sects for map and bmap commands.
Broken in 1.27.

(get_empobj_chr, obj_nameof): Switch CANT_HAPPEN() to
CANT_REACH().

(get_empobj_mob_max): Remove carriage return.

(get_empobj_mob_max): Move the return -1 to the end of function.

(emp_obj_chr_name): Move the return NULL to the end of function.

(get_empobj_chr): Removed local cp.  Return the result directly.

(get_empobj_mob_max): Switch ef_type to type to be more consistent
with other functions.

(get_empobjp): New.  Returns a pointer an empobj.
(emp_obj_chr_name): New. Returns a pointer to the characteristics name
for an empobj.
(unit_map):  Simplify unit_map() by using get_empobjp() and
emp_obj_chr_name().
(map,  cmd_move_map):  Switch to EF_SHIP as unit_map does not default
to EF_SHIP for EF_BAD anymore.

(get_empobj_mob_max): New. Return maximum mobility based on ef_type.
(miss): Use new get_empobj_mob_max().

(swaps): Failed to update coastal flags.  Was missed in the changeset
containing coastal.c rev. 1.1.
(set_coastal): New parameter olddes, to simplify fixing swaps().
Callers changed.

(update_coastal_flag, coast_sea_to_land, coastal_land_to_sea): Return
new coastal flag instead of updating the sector.  This is a bit
simpler.  Change arguments from sectors to coordinates to make it
obvious that the sector is not touched.
(set_coastal): Update the coastal flag, but leave putsect() to
callers.  All existing callers do that anyway.

Clarify doc.

Bump version to 4.3.8.

(put_empobj): New.  Put generic empobj into the appropriate data file.
(get_empobj_chr): New.  Get characteristics for empobj.
(mission, build_mission_list_type): Switch to use new
functions put_empobj() and get_empobj_chr().

(march, navi): Provide equivalent functionality to map in move(), expl()
and tran(). Allow map and bmap to access the regular map command
parameters.

(prod_nat): Change the wording from "total pop is" to
"total pop was" to be less confusing.  The total population
display is the total at the start of the update, not after the
update is complete.

[_WIN32]: Add io.h to pickup open/close externs for MSVC environments.

Update for 4.3.7.

(draw_map): Simplify.

Change level to Expert to match stop's.

(powe): Remove unused local variable.
Remove CR's.

[_WIN32, __GNUC__]: Reorganize the include files for WIN32 to
separate the MinGW environment from the MSVC environment and
WIN32 environment.  Reorganize the order of the includes so
the system files are always loaded first to prevent with misc.h defines.
Remove system file includes from misc.h.

Fix power's c_form.

(powe): Fix `pow n' not to ignore second argument when BTUs don't
suffice for `n'.

(powe): Simplify evaluation of arg 1.  No functional change.

(prpower): New, factored out of powe().  No functional change.

Update known contributors comment.

(init_sanct): Spread food evenly among sanctuaries, for simplicity.
Give max_pop() civilians (changes civs from 999 to 1000).

(init_sanct): New, factored out of new().  No functional change.

Spelling.

Belatedly ignore subject Nukes.

(display_region_map): Remove unused local variable.

Improve the description for the additional commands.
Add description for the new drop command.
Correct the quotes.

Update description for map command.
Explain the additional parameters for the map command.

(do_map): New.  map() was split into two parts map() and do_map().
(map): Call new function do_map().
(display_region_map): Modify to use do_map() instead of map().
(tran_nmap, tran_pmap, cmd_move_map): Modify to accomodate
the modified display_region_map() function.

[_WIN32, __GNU__]: Use <string.h> for strncasecmp for MinGW.
MinGW environment has an issue with conversion of strncasecmp()
to support _WIN32 and a definition in <string.h> for strncasecmp().

Remove optlist.h, not required.
Add empthread.h, needed for empth_request_shutdown() and
empth_wait_for_signal().

Add missing prototypes.h to get logerror().

Fix the previous revision.

Fix error message.

Change system header inclusion order to make crufty old systems happy.

Fix rev. 1.37 some more.

Poorly worded, many little errors, not enough focus.  Rewrite,
basically.  Same outline, but almost every sentence is changed.
Replace some details by references.  Document show sect properly.

Don't refer to Innards, it's obsolete.  Clarify work.  Talk about
petrol, not petroleum.  Advise that resource depletion is
configurable.  Clean up prose somewhat.

Page managed to talk about BTUs in considerable detail, yet missed to
tell what they're good for.  Amazing.

Don't claim which resources are renewable and which aren't, because
that's configurable.

Document the transformation of sanctuaries into capitals or highways.

Don't refer to Innards, it's obsolete.

Clarifications.

Belatedly remove BIG_CITY and DEFENSE_INFRA.

(setstr): Supply missing cast.

(vers): Format string fix.

Fix the previous revision.

Fix the previous revision (system header inclusion cleanup), it
doesn't compile on crufty old systems.

Back out the previous revision (system header inclusion cleanup), it
doesn't compile on crufty old systems.

(bestownedpath): Failed to consider direction y due to off-by one
mistake in rev. 1.28.  Reported by Stefan Hauser.

[_WIN32]: Add missing include sys/types.h to get off_t.

Add missing includes config.h, prototypes.h and file.h.

(obj_nameof): New.
(nameofitem): Remove, switch callers to to obj_nameof().
(mission, show_mission): Convert functions to use obj_nameof()
instead of nameofitem().

Break inclusion cycle: prototypes.h and commands.h included each
other.  Ensure headers in include/ can be included in any order
(except for econfig-spec.h, which is special).  New header types.h to
help avoid inclusion cycles.  Sort include directives.  Remove some
superflous includes.

Fix missing include.

(coord): Unused & inappropriate, remove.

Fix the previous revision.