(total_work): Remove redundant conversion to int.
(total_work): New parameter maxpop.  Callers changed.
(newe, prod): Use total_work().  Old code failed to limit military
workers, thus didn't match the update.
(do_feed): Simplify.

(prod): Widen column will make so that product p_sname is not
truncated.

(prod): Several columns were too narrow, and didn't align properly for
large values.  Remove column wkfc to make space, widen other columns.
Columns use2, use3 and max2, max3 lack separating space on the left
for large values, which don't occur in normal game configurations.

(upd_ship): Fix the previous rev.

(prod, produce, upd_ship): struct pchrstr p_nrndx refers to u_char,
not s_char.  Fix types.

(upd_ship): Catch resource underflow (should not happen).

(max_population): Fix for opt_RES_POP and d_maxpop != 999.

(max_population): New, factored out of max_pop().
(max_pop): Use it.  Don't support null argument anymore.
(nati, gen_power): Use max_population() instead of max_pop() with null
argument.
(show_sect_stats): Use max_population() to simplify the code.
(prod, upd_buildeff): Use max_population() to get correct new maximum
population for limiting work when tearing down a big city.
Previously, this was broken in upd_buildeff().  In prod(), it was
broken when the new designation had odd maximum population, which
should not occur.

(prod, nati): Use max_pop() instead of hardcoded values.  Before,
struct dchrstr member d_maxpop had no effect in some cases.

(newe): Fix for max_pop() > 999.

Break an inclusion cycle missed by yesterday's change.

`make depend' attempts to make dependencies for *.c.  This is plain
evil, and ought to be fixed.  In particular, it chokes on this file,
because it includes Windows specific headers.  Which is fine, as the
file is compiled only under Windows.

Quick band-aid: wrap the file contents in #ifdef _WIN32.

Break inclusion cycles.  To simplify the change, move a few
declarations to other, no less sensible places.

(dchr_ca): New selector maxpop.

(multifire): If firing at a submarine with something that can't drop
depth charges, don't disclose the target is a submarine.  Old code
allowed players to find all submarine uids.

Products table used tlev and tech synonymously.  Confusing.  Just use
tech.

From Ray Hyatt:
(dchrstr): New member d_maxpop.
(dchr): Initialize it.
(max_pop): Use it.  Use dchr[SCT_MINE].d_maxpop when called with null
argument.  RES_POP now affects mountains and plains as well.

[service_main]: Code cleanup, remove commented out debugging,
remove unnecessary include files.

Update patch for src/lib/commands/news.c:1.8  No functional changes.

(news, fina): Report bad conditionals as syntax error.  Closes #775475.

Clean up register keywords.  No functional changes.

Use _l_rand48(). in src/util/fairland.c

Fix the previous patch by using emp_{s,}random() instead if {s,}rand48()
in src/lib/commands/{new,mfir}.c

Update patch for src/util/fairland.c:1.16 src/lib/commands/new.c:1.19
src/lib/commands/mfir.c:1.20 src/server/main.c:1.35.
No functional changes.

[_WIN32]: Upgrade to Windows Sockets 2.

(tend_nxtitem,nxtitem,sarg_type,snxtitem,snxtitem_all,snxtsct,nxtitemp):
change NS_ALL to NS_EVERYTHING in ns_seltype because of a
conflict with winsock2.h which is used in the WIN32 build.

(rea): Add brackets to prevent 'else' ambiguity.

(wrauth,wrgamesdef,wrmakesrc): update copyright notices to 2004.

(main,wr_auth) [_WIN32]: switch the default file mode to
binary to match POSIX.  Add text mode for auth file.

(new) [_WIN32]: Rely on misc.h to provide BSD random functions.

(multifire) [_WIN32]: Rely on misc.h to provide BSD random functions.

(multifire): switch the random seeding with the current random
value to seeding with the current time.  Seeding with the
current random value does not actually improve the randomness.

The previous rev. was committed accidentally, revert.

(main,install_service,remove_service,service_main) [_WIN32]:
Add the ability to specify the service name, to install
multiple services and to set -D and -e options during install.

(rea): Code cleanup.

(main,mobility_check,upda,turn,rea,mobupdate,ef_open,logerror,
emp_config,gamedown,show_motd,match_user,typed_wu,
delete_old_announcements) [_WIN32]: switch the default file mode to
binary to match POSIX.  Remove the now unneccessary ifdef for
the open/fopen. Add text mode for user files (econfig/auth).

(main,ef_open) [_WIN32]: switch the default file mode to
binary to match POSIX.  Remove the now unneccessary ifdef for
the open.

(main,allocate_memory) [_WIN32]: switch the default file mode to
binary to match POSIX.  Remove the now unneccessary ifdef for
the open.

(main) [_WIN32]: remove getpid() from the random seed as
it did not improve the randomness of the seed.

(main): switch to use standard srandom() and random().
Remove the SRANDOM and RANDOM defines.

(parse_args): fix the print usage for script file name to print the
default instead of the current file name as per the description.

(main) [_WIN32]: enable command line options for WIN32.

Update for 4.2.19.

Indentation fix.

Updated from man/emp_client.6 rev. 1.7.

Fix spelling of daemon, for crying out loud!

Remove irrelevant information on player thread.

Clarify that -d makes the server abort on internal errors.

Update patch for src/server/main.c:1.29,include/prototypes.h:1.61 and
src/lib/commands/laun.c:1.9.  No functional changes.

Fix typesetting of option arguments.

Fix English in section authors.

Really fix typesetting of option arguments.

Move arguments behind options in the synopsis.

Fix typesetting of option arguments.

Document environment variables more clearly, and without assuming csh
syntax.

(main, start_sever): Previous revision broke POSIX threads under
Linux.  Undo the problematic bit of the change.

Realign comment with its object; got separated in the previous rev.
No functional changes.

Fix rev. 1.5's placement of braces.  No functional changes.

Explain that -s and -p imply -d.

switched comment from // format to /* */ format.

(main,close_files,loc_NT_Term,empth_exit):
Add Windows Service to the server.
This includes adding install service and remove service option.
Move the start code from main to separate function.
Make close_files() and loc_NT_Term() global.
Move disassoc() to be earlier in the startup sequence.

switch comments from // format to /* */ format.

[_WIN32] (panic): Unused under Windows, remove from build.

Change -2outfile to -2 outfile for both empire.6 and emp_client.6.

(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.

(panic): Instead of exiting successfully, commit suicide by re-raising
the signal.

(termio): Fix crash bug: when input line starts with '?', `s' was used
uninitialized.

(buf, exec, termio): Make buf[] and exec[] local to termio().

(NUKEFAILDETONATE, opt_NUKEFAILDETONATE, Options, msl_hit): Remove
option NUKEFAILDETONATE.

(ORBIT, opt_ORBIT, Options, launch_sat, pdump, plan, prod_plane):
Remove nooption ORBIT.

"(turn): Fix comparing the output of write() to an unsigned sizeof by
casting the sizeof to write's most used return-type (ssize_t). Introduce
a typedef for ssize_t for _WIN32 in include/misc.h."

"(turn): Fix comparing the output of write() to an unsigned sizeof by
casting the sizeof to write's most used return-type (ssize_t). Introduce
a typedef for ssize_t for WIN32."

(info): Polish output for ambigous topic argument.

Belatedly remove ROLLOVER_AVAIL.

(GUINEA_PIGS, opt_GUINEA_PIGS, Options): New option GUINEA_PIGS.

Initial version; needs work.

Canonicalize line endings.

(player_coms): Implement c_form for xdump and enable the command.

(xdump): Fail unless GUINEA_PIGS is enabled.

(GUINEA_PIGS, opt_GUINEA_PIGS, Options): New option GUINEA_PIGS.

(print_config): Undo rev. 1.17's incorrect simplification of
km_comment printing.

Belatedly remove ROLLOVER_AVAIL and PLANENAMES.

(ac_calc_flak,ac_fireflak,pinflak_planedamage): New ac_calc_flak().
Move common flak damage code from ac_fireflak() and
pinflak_planedamage() to ac_calc_flak().

(ac_calc_flak,ac_fireflak,pinflak_planedamage): New ac_calc_flak().
Move common flak damage code from ac_fireflak() and
pinflak_planedamage() to ac_calc_flak().

Remove carriage return from comment.

(comtch): Fix the previous rev.

(turn): Change the command to allow for both an MOTD and a no-login
message.  Clean 'register' and 's_char' remnants on the way.
Circumvents bug #813167.

(ship_bomb): Prevent bombing of subs that where not found by the ASW plane.

(ef_extend): free allocated tmpobj before early returns as well.

(EFF_COM,ef_items): No longer in use after removal of 'variables'. Removed.
Clean 'register' and s_char on the way. Remove dead code.
Rename some variables to better reflect their meaning.
Add function comments.

(EFF_COM,ef_items): No longer in use after removal of 'variables'. Removed.
Clean s_char on the way.

(EFF_COM): No longer in use after removal of 'variables'

(have_looked,have_found,set_have_looked,set_have_found,print_found):
collapse to shiplist function set (add,free,print,on). Fix bug where ship
uid was a unsigned char instead of short.  Switch to head pointer instead
of head record with a linked list. Add a free().

(getopt.h) move to src/lib/gen directory to prevent conflicts with
/usr/include/getopt.h.

Fix bug #883113. Differentiate between sct_work and sct_loyal.

(scuttle_tradeship): Remove carriage return.

(getopt): Remove carriage return.

(show_sect_build): Remove carriage return.

(upd_ship): Remove carriage return.

(prod): Fix bug #1083175. Use max_pop() not just when opt_RES_POP is
enabled to determine the new working population of a sector.

(orde,eta_calc): Remove unaccessed leftover variables nships,mcp.
No functional changes.

(ship_bomb,land_bomb): Remove unaccessed leftover variables lcp,onsea.
No functional changes.

(resupply_commod,has_supply,use_supply): Remove unaccessed leftover
variables lcp.  No functional changes.

(ac_airtoair): No need to pass def_own, as the owner of the intercepting
planes is used anyway.  Clean up other unaccessed variables in the process.
No functional changes.

(scut): Remove unaccessed remnants from before 1.1.  No functional changes.

Update patches for #include cleanup, changing offsets.  No functional changes.

Cleanup #includes of (mostly a long time) unused header files.
No functional changes.

Only prototypes.h is needed, not all of gen.h.  No functional changes.

Update patch for src/server/main.c:1.23.  No functional changes.