(info): Fix when with last character is special.
Also reorganized to check for special characters then blank.

(info): Add length boundary check to filename variable.
Change WIN32 to use the same code.

(info) [_WIN32]: Fix check for invalid file attributes.

(apro): Ensure the user does not see the directory information on error.
See #1028552 for reference.

(apro): Ensure the directories, operating system files, and
hidden files are skipped.

i_type shift (include/item.h 1.12) broke the production of tech/res/edu/hap.
Switch p_type to use I_NONE for products that to not produce items.

(dchr): Negative d_cost now means that only deity can designate.
(nullify_objects, do_desi): Simpify designate restrictions of options
NO_LCMS, NO_HCMS, NO_OIL.

(show_sect_build): Suppress printing of sectors only deity can
designate.  This was broken in rev. 1.15.

Correct the instructions for the Win32 build.

(main,termio): Add the ability to set the username.
Get it from environment variable LOGNAME if set,
else get the user name from the system.

(main, termio): Add the ability to set the username.
Get it from environment variable LOGNAME if set,
else get the user name from the system.

(main) [_DWIN32]: Add stdin redirection ability to the Win32 port.
The client reads until stdin redirection empty and then terminates.

(var.h, plague.h): Since all that was left in var.h were some plague
defines, import these defines into plague.h, drop var.h and include
plague.h where appropriate.
Remove some 'register' keywords at the same time.
No functional changes.

Do not include var.h where no longer needed.  Clean up register keywords in these file at the same time.  No functional changes.

(i_type, I_NONE): shift the enum down 1, so that I_NONE does not need to
be skipped or stored in arrays of size I_MAX. I_NONE's int equivalent
becomes -1 by this patch, so all array indices of type i_type have been
checked not to use I_NONE as index.  This change reduces the size of the
arrays stored in files.

(i_type): one patch too many in the previous commit.  No functional changes.

(ac_encounter): Ignore embarked land units when checking for planes
overflying land units.  Reported by Doug Wescott.

(new): When first sanctuary is a wilderness, but second isn't, code
complained about the first one.

(mcost): Unused, remove.

(info): Don't leak potentially sensitive information to player in
error messages.  Properly log all system errors and return RET_SYS.
Closes #1028552.

(info): Fix use of uninitialized variable when fopen() succeeds but
fstat() or the st_mode check fails.

(info) [_WIN32]: Fix '//' to be '\\'.  Add file type check to prevent
opening hidden or operating system files.

Work around for broken MSC C++ 6.0 (VC98) choking on empty macro arguments.
Fixed in MSC C++ 7.0.

Remove the unused/not needed code.

(item_prewrite): New.
(checksect, shp_prewrite, lnd_prewrite): Use it.

(shoo): Code to enfore the mobility limit assumed double m were
integer.  When 5 * sect.sct_mobil - 4 < nshot < 5 * sect.sct_mobil,
nshot was rounded up to the next multiple of 5.  This could shoot more
people than present, resulting in negative population.  Before
rev. 1.3, putvar() swept this error under the rug.

(load_it): Loaded negative amount if ship had more cargo than ordered.
This can happen when unload_it() can't unload, e.g. because sector is
full.  This can push the sector's amount beyond ITEM_MAX, and can even
overflow to negative amounts.

(dodistribute): Import could export a civilian when the dist center
didn't have any.

(ac_doflak): Don't supply more shells than actually required for flak.

Doc fix.

(scra): Obey ITEM_MAX.

(multifire): Remove some dead code.

(do_demo): Simplify.

(dchr): Init the packaging of trailing dummy structs with IPKG instead
of 0.  No functional change.

(I_NONE, I_CIVIL, I_MILIT, I_SHELL, I_GUN, I_PETROL, I_IRON, I_DUST,
I_BAR, I_FOOD, I_OIL, I_LCM, I_HCM, I_UW, I_RAD, I_MAX): Turn macros
into enumeration constants.
(i_type): New.  Use where appropriate.  No functional changes, except
that I_NONE is now catched properly in a few places.

Fix the previous rev.

(NSC_SITYPE): New.
(dchr_ca): Use it to fix selector pkg's ca_type.

(sect_ca, ship_ca, plane_ca, land_ca, nuke_ca): Make selector coastal,
nplane, nland, nchoppers, nxlight, timestamp NSC_EXTRA.

(ship_ca): New selectors xstart, xend, ystart, yend, cargostart,
cargoend, amtstart, amtend (for order) and mobquota, path, follow (for
sail).
(news_ca): New selector uid.
Selectors are now believed to be complete, except for nat_ca[], which
is not used.

(valstr, nstr_exec, nstr_comp_val, nstr_exec_val, xdprval): Support
strings with length limit instead of zero termination.
(ship_ca, land_ca): New selectors rflags, rpath.  The latter is such a
string.

(comtch, stmtch): Make more similar, document.  No functional changes.

Doc fix.

(mchr_ca, lchr_ca): Fix selector flag's ca_type.

(xdprval): Make sure floating-point numbers always contain a decimal
point, so they're easily distinguished from integers.

(fldoff): Use offsetof() instead.  Rest of struct.h is unused, remove.

(plnstr): Member pln_fuel was always equal to plchr[pln_type].pl_fuel.
Replace all uses, so pln_fuel can be removed later.

Use plain char * instead of s_char *.

Update copyright notice.

Fix incomplete comment.

Update for 4.2.18.

(info): `info' was not checking for all types of common files in
Windows, therefore could miss info files depending on the status
(archive, read-only).  Also changed the error message to user to
provide a more accurate description of the problem.  Closes #1018297.

Explain roots of LWP.

Unused, remove.

(tran_plane): Fix major, abusable bug: getsect() before move_ground(),
putsect() afterwards.  This allowed two cooperating countries to
duplicate items.

(tran_plane): Don't report interdiction damage twice.

(tran_plane): Simplify loop collecting planes.

(news): Add a comment to explain why argument is ignored when HIDDEN
is on.

(mchr_ca): Add missing entry for member `frnge'.

(xdprval): Silence compiler warning.

(NEWS_PERIOD, news_keep_days): Replace macro NEWS_PERIOD by new
configuration key news_keep_days.

(xdprval): Fix escaping of negative characters.

(xdprval): Escape the space character so that a line can be simply
split into fields at space.

(keymatch): Replace member km_func by km_type.  Initializers adapted.
(emp_config, print_config): Cope with km_type.
(worldxset, intset, floatset, optstrset, doubleset, longset): Unused,
remove.

(emp_config): Fail if specified file can't be read or contains errors.
Used to succeed always, returning RET_OK.  RET_OK is not appropriate,
since this is not a command.  Return 0 on success, -1 on failure.
Callers ignore failure at the moment.  A missing or unreadable econfig
file used to be silently ignored.  It is still ignored, but no longer
silently.  It is questionable whether ignoring is wise, but that's
left for another day.

(emp_config): Improve diagnostic messages.

(set_option): Move diagnostics to caller.

(emp_config): Ignore leading whitespace in `#' comment lines.

(print_config): Simplify printing of km_comment.

(set_option): New parameter val, so it can set and clear options.
(delete_option): Remove.

(KM_ALLOC): Turn macro into enumeration constant.

(KM_INTERNAL): New.
(xdump, xdver): New version dump.

(keymatch, infodir, datadir, teldir, upfil, downfil, disablefil,
telfil, annfil, banfil, timestampfil, privname, privlog, update_times,
update_demandtimes, game_days, game_hours): Use plain char * instead
of s_char * for strings, void * for generic pointers.

(xdprval): Fix for strings containing backslashes.

(xdprval): Fix for strings with characters printed as escapes.  Print
null pointer as `nil' to distinguish it from empty string.

(hard_tech): Unused, remove.
(configkeys): Fix km_flags of key anno_keep_days.

Undo the previous revision (bad idea).

(EMPCF_COMMENT): New; for readability.

(keymatch, configkeys): Move to from optlist.h to emp_config.c; static
linkage.

(xdump): New options dump.
(xdopt): New.
(xdhdr1): New, factored out of xdhdr().

(ship_ca): Fix selector name's ca_len.
(xflds): Ignore ca_len for NSC_STRINGY.

(pln_arm): Callers don't use parameter tech; remove it.  Callers
changed.

(ANNO_KEEP_DAYS, anno_keep_days): Replace macro ANNO_KEEP_DAYS by
new configuration key.  Closes #780425.
(delete_old_announcements): Do nothing when anno_keep_days is
negative.

(trad, set): Treat trade file slots with zero owner as unused, so that
freshly extended trade file makes sense.  When set() removes a lot,
simply zero the slot.

(set): Initialize trd_uid.

(trade_check_ok): Use check_land_ok() & friends instead of doing it by
hand, but wrong.  This fixes bying planes and land units loaded on
ships that moved after loading.

(check_trade_ok): New.
(trade_check_ok): Use it.  Remove unused argument lot.  Callers
changed.

(show_land_build, show_plane_build): Adjust the position of the $ column to match
'show ship b'.  Adjust the crew column (show_plane_build) and the adjacent columns
to be right-justified.

(gift): Default case can't happen.  Make that explicit for safety and
to silence compiler.

(launch_missile): Simplify to avoid compiler warning.

(empth_ctx_t): Unused, remove.

(min_to_next_daytime): Happily returned zero if DTIME happend to be in
TIMES.  This can make callers fire another update right away, possibly
several times.  Observed for update policy UDP_TIMES while updates
were disabled, which makes them complete in much less than a second.
Fills up server.log real fast.  Fix to consider DTIME tomorrow.

(last_demand_update): Write-only.  Removed.

Replace incorrect comment on distribution mobility cost by a reference
to info Mobility.

(IPKG): New.
(ichr): Set IPKG packing bonus for all sectors to 1.
(move): Use it.  No functional change.
(deliver, dodistribute): Use it.  Inefficient sectors now get IPKG
instead of NPKG packing bonus, cutting civilian packing bonus from 10
to 1.

(dodistribute): Export uses the better of sector or distribution
center packing bonus.  Previously, it assumed that distribution center
WPKG packing is always best.  Which is currently true, but needlessly
breaks the abstraction.

(dodeliver): Move from sect.c to deliver.c.  No functional changes.
(deliver): Static linkage.

retreat.h is unused, remove it.

(build_land): unused variable natp, remove.

(news_ca): Change type of selector action to NSC_TYPEID.

(nat_ca): Split awkward selector level into selectors tech, research,
education and happiness.

(nstr_exec_val): Failed to update category when evaluating symbolic
value.

(chr_camap): Make table names differ from file empfile[] names, so
that they properly identify the dump.

(dchr_ca, intrchr_ca, rpt_ca, tchr_ca): New.
(chr_camap): Use them, but disable rpt_ca[], since rpt[] lacks a
sentinel.

(xhdr): Print `XDUMP' rather than `DUMP' to avoid confusion with old
dumps.  Don't print human readable timestamp.

(improve): Fix confusing diagnostics when refusing to improve defense
because option DEFENSE_INFRA is off.

(improve): Simplify use of `value' to avoid compiler warning.

(move, explore): Simplify computation of destination sector's work and
loyalty to get rid of compiler warnings.

(sect_ca): Fix selector che.

(getsect, putsect, getsectp): Remove superflous casts, parenthesize
macro parameters properly.

caddr_t is obsolete.  Replace by void *, except for struct empfile
member cache, which becomes char * to keep pointer arithmetic simple.

(sctstr, sect_ca): Make sct_mines signed for same reasons as
sct_item[] (see sect.h rev. 1.10).

Removed unnecessary #ifdef's for Windows.

(nstr_exec): Fix the previous rev.

(NSC_STRINGY): New.
(nstr_promote, nstr_exec_val): Implement.

(nstr_exec_val): Ignored IDX for NSC_STRING values.

(nstr_exec): Implement NSC_STRING.

(ship_ca): New selector name.

(sect_ca): New selectors loyal, access.

(NSC_GENITEM): New selector radius.

(ship_ca): New selectors nland, access, xbuilt, ybuilt, builder.

(plane_ca): New selectors access, theta.

(land_ca): New selectors react, nland, access.

(lost_ca): New selector id.

Doc clarifications.

Reorder selectors to match struct member order, for easier consistency
checking.

(NSC_EXTRA): New.
(sect_ca, NSC_GENITEM, plane_ca, land_ca): Use it for aliases and
cached functions of tech.
(xdflds, xdfldnam): Skip NSC_EXTRA fields.

(ldump): Remove superflous casts.

(NSC_NATID): New.
(sect_ca, NSC_GENITEM, nuke_ca, treaty_ca, loan_ca, news_ca, lost_ca,
commodity_ca, trade_ca, nat_ca): Use it.  Fixes sector selector
che_target.

(NSC_IELT, NSC_GENITEM, ship_ca, plane_ca, nuke_ca, treaty_ca,
news_ca, lost_ca, commodity_ca, trade_ca, nat_ca): Fix initializers
for ca_type.  Harmless signed/unsigned mismatches, except for plane
selectors att, def, nuke selector types, treaty selector exp, news
selector time, commodity selectors type, xbuy, ybuy.

(commodity_ca): Remove NSC_DEITY from selector price.

(xdeval): Use NSC_NOTYPE instead of 0.

(xdprval): Cope with NSC_TYPEID.  Check for bad types.

Doc fixes & clarifications.

(plchr): Fix the previous rev.

(PLANENAMES, opt_PLANENAMES, Options, plchrstr, plchr_ca, plchr,
main): Remove nooption PLANENAMES.

(xdprval): Passed ptrdiff_t as field width.

(delete_old_announcements, copy_and_expire): Split off
copy_and_expire() to simplify cleanup.  Previously, cleanup was
missing or incomplete on many less common code paths, which leaked
file descriptors and memory, and could corrupt the anno file.  Use
stdio instead of POSIX I/O for writing.  Check for I/O errors
properly.  Treat bad telegram type just like bad telegram length (give
up).  Closes #1013234.

Combine the definitions of include/optlist.h and the use thereof in
src/lib/gen/emp_config.c into a single file: include/econfig-spec.h.

Comments, cosmetics.