Properly declare stuff in headers; remove redundant declarations.

(logerror): The version from libcommon.a logs to a file, which is
inappropriate for this program.  Log to stderr instead.
(program): Unused, remove.

Properly declare functions in headers; remove redundant declarations.

(iovec, ioq_peekiov, ioqtoiov): Unused, minor portability
complication, remove.

(_noecho, _echo): Broken, portability headache, remove.
(output): Ignore C_NOECHO.  Server doesn't send it anyway.

Remove some redundant declarations.

<string.h> is ISO C, no need to #if it.

POSIX has all we need in <time.h>.  No need to fool around with
<sys/time.h>.

Don't declare library functions, include appropriate headers.

(deli): Partial rewrite for clarity, motivated by a `might be used
unitialized' warning.  Funcional change: you can now change direction
without threshold (matches documentation, clarify it anyway).  Change
prompt for the third argument to reflect that.  Also provide context
when prompting for the fourth argument; this is sorely needed we
didn't prompt for the third argument.

(player_accept): Back out rev. 1.9, it doesn't compile under Windows.
Needs fixing.

(player_coms): Fix c_form for fire.

Declare command functions in commands.h.  Delete their declarations in
empmod.c

Make users of struct cmdstr agree on prototype of member c_addr.
(player_login): Use player->argp[] to store arguments.
(client_cmd, user_cmd, sanc_cmd, coun_cmd, pass_cmd, play_cmd,
kill_cmd, list_cmd, quit_cmd): Remove parameters, fetch arguments from
player->argp[].
(cmdstr): Prototype member c_addr.

(play_cmd): Used to silently ignore up to two arguments if there
wasn't a third one.  Don't.

(count_planes): Remove declaration.  A prototype would need struct
shpstr, but not struct plnstr, which is a strong sign it doesn't
belong here.

[_WIN32]: Include required headers.

[_EMPTH_WIN32]: Previous rev. didn't even compile, fix.

Clean up previous revision.

Declare all configuration variables in optlist.h.  Include that
instead of declaring them all over the place.  This uncovered type
errors:
(s_p_etu, adj_update): Defined long, sometimes declared int.  Kills
big endian machines where sizeof(long) != sizeof(int).  Change to
int.

(set_option, delete_option, optstrset, intset, floatset, doubleset,
longset, optionset, optiondel, worldxset): Change linkage to static.

Remove some superflous declarations missed in the previous revision.

Cosmetic.

(empth_t, empth_sem_t) [_EMPTH_WIN32]: Use incomplete types, not void for
abstract data types.

Supply prototypes where possible.  This uncovered type errors with
thread entrypoints:
(lwpSelect, shutdown_sequence): Parameters didn't match thread entry
point prototype.
(lwpEntryPoint): Arguments didn't match thread entry point prototype.

Change linkage of functions without prototype declaration to static
where possible.

Remove some superflous declarations, replace others by suitable
includes.

Windows cleanup: include proper headers, remove unused variables, fix
type of signal handler.  Mostly from Ron Koenderink.

(move_ground): Remove superflous parameter `mobility'.  Callers
changed.  Edit for clarity.  No functional changes.

(emp_sig_t): Pre-ISO C compatibility cruft.  Just use void.

(payo): Previous revision screwed up ally bonus.

(upd_ship): Capability fish didn't work together with capability oil.

(player_accept): Passed pointer to incorrect type to accept().

(clean): Fix for Windows.

Fix markup of right quotes.

Fix generated HTML: proper !DOCTYPE, head, body, p.
Fix request parsing, so empty requests are properly ignored.

Spelling fixes.

One TH request per file; remove second one.

Clarification.

Use plain double quote instead of \*Q and \*U in .EX, because it's
typed text.

(Q, U): Expanded into nothing; fix.

(lF): Unused, remove.

Deviated from actual behavior (matched old behavior?).  Fixed.

empclient later than 2.8 support the Empire 4 protocol, too.  Don't
claim no other client supports it.

Fix title (second argument of TH).

Stricter tests for requests.
Request s3 doesn't exist, remove translation.

Fix indentation; $ in the middle of regexps confuses Emacs, so use \Z
instead.  No functional changes.

Parse requests properly.  Simpler, more correct and more robust.

Fix translation of escapes.  Still a hack.

Fix Empire client prompt in translation of EX and SY.

Fix markup: bogus numerical arguments that were mapped to zero
silently (troff) or under protest (groff); table column width (also
fixed some .L arguments to match actual game output); spacing in
no-fill text; stray .FI, .in and .ce messing up indentation; stray .L;
misspelled request and escape names; escaping of control and escape
characters; properly ignore form-feed; argument quoting; extra
arguments to .NF.  Still broken: .L abused for headings, which
triggers many groff warnings about negative indentation.

(nF, eF, bF, iF, lF, pF, cS, CS): Define to silence groff.

Convert tabs before \" comments into spaces (troff pitfall).

(SY, EX): Fix Empire client prompt.

(eF, EF): Font C doesn't exist, use R.

(N, EF): Define to R to silence groff.

4.2.13 last minute changes.

Change tarball names to empire$part-4.2.13.tar.gz.  Tarballs now
unpack in a directory whose name matches the tarball name, as usual.

Set archive members' owner, group and force sensible permissions.

(lwpInitContext): Fix .function for current versions of as.

(main, USERNAME): Change Windows user name from win-empcl2.8 to nobody
to get rid of the version number.  Keeping it up-to-date by hand is
too much work, but leaving it at 2.8 would be confusing.

(FREEBSDLFLAGS): Compile with symbols.
(HPCFLAGS, HPUXCFLAGS, RTPCLFLAGS, SOLCFLAGS, AIXCFLAGS): NOFLOCK is
unused, don't define it.

Don't fool around with obsolete strings.h, just include standard
string.h.

Fix previous rev.

Fix the previous revision.

Document production, feeding and plague for units.

(move_ground): explore() used endsect without initialized in certain
cases, e.g. `expl X,Y N X,Y'.  Other callers might have had similar
problems.  Fix: always assign to *END.  Thanks to Marc Olzheim for
help with debugging and fixing this.

Do not regenerate mobility faster while it is negative.

Fix markup.

Fix .NA and .TH.

Create info.text.tar.gz and info.html.tar.gz.

(iom, fdmax, newoutput): Serve no purpose since previous rev., remove
along with code to maintain them.  This removes the last uses of
io_mask.c io_mask.h; remove them.

(install, ntinstall): Create the data directory.
(ntinstall): Remove erroneous and superflous mkdir bin.

Update for 4.2.13.

Remove outdated version information for now.  Keeping it up-to-date by
hand is too much work; should be automated.

New make target dist, to run new src/scripts/mkdist.

Creating make include files with make doesn't work with many makes out
there.  Back out for 4.2.13.  You have to create the make include
files by hand if they don't exist.  That's the case for freshly
checked out trees.

(show, show_item): No new `show' features without matching machine
readable dumps.  Back out `show item' for 4.2.13.

(nuk_add): Failed to call makenotlost() when reusing a `hole' in the
nuke file.

(RESOLVE_IPADDRESS): gethostbyaddr() uses a lot of stack space on some
versions of GNU libc, which can lead to nasty heap smashes (observed
with 2.2.93).  Don't define RESOLVE_IPADDRESS for now, until we
readjust thread stack sizes.

Document help command.  Closes #804550.

Useless copy of an old version of Makefile, remove.

Whitespace fix.

No need to pass -nsob, it's implied by -kr.

Indentation fix.

(fly): Fix confusing line break.

(doland): Fix output (run-away replace?).

Fix previous rev.

(ca): Unused, remove.

(LINUXPTHCFLAGS): Really don't define __USE_BSD_SIGNAL.

(LINUXCFLAGS, LINUXPTHCFLAGS): Don't define lint and __USE_BSD_SIGNAL.
The latter can't be found anywhere in source or system headers.

(OSXCFLAGS): Don't switch gcc to cruft mode.
(OSXLFLAGS): Don't sweep undefined references under the carpet; link
with the right libraries instead.
(LINUXCFLAGS, LINUXPTHCFLAGS): Compile with symbols.

Correct info on mining harbors.  Closes #827442.

Fix BTU cost.  Closes #782956.

Delete lie about give without arguments.  Closes #804709.

Document effect of BLITZ.  Delete lies about the total number of
players display.  Closes #809867.

.SA starts a paragraph.

Spelling fix.

Properly escape `\'.  Fix info on sector capabilities.  Closes
#861874.

Fix off-by-one in arguments of substr() that ate first letter of .SY,
.EX and .L text.

Garbage collection to side-step sparse file descriptor problem.
(io_select, io_flush, io_iopfromfd): Unused, remove.
(io_open): Allow more than one open for the same file descriptor.
The check is the last remaining use of io_list[], and it's
unimportant.
(io_list, niop, io_init): io_list[] no longer serves any purpose,
remove.  Sparse allocation of file descriptors (deplorable misfeature
of Winsock, at least some versions) is no longer a problem.
(getfdtablesize): Unused, remove.  Was broken for Windows anyway.

(io_open, io_iopfromfd): Properly check argument FD to protect
io_list[] indexing.
(niop, io_init): Store length of io_list[] in new niop.
(player_accept): Checking the socket file descriptor here is now
redundant.  Remove.

(msl_hit): Tell victim coordinates of incoming missile.  Closes
#799873.  From Marc Olzheim.

(rpt): Fix rpt[N_NUKE_SSTOP]; not every sub-launched missile is
nuclear.  Closes #797330.  From Marc Olzheim.

(landmine): Add a missing call of putland().  Closes #682570.  Don't
complain about running out of things when the job actually completed
fine.  From Marc Olzheim.

(player_coms): Fix flag argument of map commands in c_form.  From Marc
Olzheim.