(copy_ascii_no_funny, copy_utf8_no_funny)
(copy_utf8_to_ascii_no_funny): New.
(flash, wall, prmptrd, uprmptrd, getcommand): Use them to filter
input.
(uprnf, pr_flash): Use them to filter output.
(prtoascii): No longer used, remove.

(player_commands, player_commands_index): Internal linkage.

(upr_player): A print function must not change its text argument!

(lrange, shoo, execute): Buffer passed to getstarg() was too small.
Player could overrun it!

(player_login, player_main): Move the C_INIT that terminates the login
phase to player_login().  No functional change.
(showvers): Trivial, used just once.  Inline, remove.

(parse): Support 127 arguments instead of 100, to match
player->argp[].  Assign complete argument vector.  This should make
buffer overruns blatantly obvious.

(parse): Don't bother checking SPACE argument, just crash.  It is
extremely unlikely that such a programming error wouldn't be
discovered by even them most superficial testing.

(emp_config): Buffers passed to parse() were too small.  econfig file
could overrun them.  Closes #899546.

(dispatch): Argument buf is user text and must be printed as such.  In
error messages, print just the command name instead.

(play_list): player->combuf[] is user text and must be printed as
such.
(ufindbreak, ufindpfx): Rename, move to pr.c, external linkage.

(shut): Buffer passed to getstarg() was much too small.  Deity could
overrun it.

(shut): Fix retrieval of second argument.

(shut): Simplify formatting of broadcast messages.

(rea, turn): Use a different buffer for message text.  Just for
clarity.

(typed_wu): Treat overlong message as internal error instead of just
truncating it silently.  Simplify.

(getele): Abort telegram when uprmptrd() fails.  Previously, failures
that didn't set player->aborted sent the telegram, e.g. EOF.

(getele): Fix misleading size of left[].

(getele, tilde_escape): Simplify.  tilde_escape() now returns the
escape code instead of comparing it to second argument.

(turn): Buffer passed to getele() was one too small.  Deity could
overrun it by entering a message of maximum size.

(gamedown, show_motd): Fix message buffer size.  Messages of maximum
length were considered overlong and ignored.

Remove some redundant casts.

Remove a bunch of redundant casts.

(rea): Portability fix.

[_WIN32] (loc_StripDels): Remove.  Different behavior depending on the
platform is evil, and line editing is none of the server's business
anyway.  If your client can't deal with the delete key, well, here's a
nickel, kid, get yourself a better computer.

(ioq_gets): Support telnet line termination "\r\n".  Previously,
string input other than through parse() or getele() failed to strip
'\r', which could end up in ship names and such.
(getele): Code dealing with '\r' is now redundant, remove.  It ceased
to work in rev. 1.6 because input filtering replaces '\r' by '?'.

Fix markup.

Spelling fixes.

Fix the previous revision.

(sendmessage): Skip players not in state PS_PLAYING.  Flashes to them
get ignored, hence telling the player that the flash was sent is
wrong.  Skipping fixes that.

(mobupdate): Don't print '\r' after '\n'.

(play): Simplify counting of players.

(options_cmd): List options only if none are set.

(list_cmd, login_coms): Login command list is obscure and redundant;
remove.

(login-coms): Improve doc strings.

(nstr_string_ok): New.
(nstr_comp): Use it to fix oops on `des='.  Reported by Doug Wescott.

(getsose): Port from obsolete termcap to terminfo.  The old code
manually stripped off padding, which is evil.  The new code does
nothing when the stdout is not connected to a terminal.
(SO, smso, SE, rmso): Rename, static linkage.
(putso, putse): New.
(screen): Use it.

(hidd,peek): Rename hidden command to peek.
This prevents a conflict in the info files with
HIDDEN option.

Update for src/util/fairland.c:1.23 and src/server/main.c:1.62

Spelling fixes.

(program_name, help): New.
(main): New option -h.  Terminate unsuccessfully on invalid option.
(usage, parse_args): Factored usage() out of parse_args().
(parse_args): More useful diagnostics.

(prod_plane): Remove redundant initialization.

(sarg_type): Simplify, no functional change.

Doc & formatting fixes.

(flee, army, wing): Print how many new members were added.
Previously, it printed how many were selected, which can include old
members of the group.

(flee, army): The rather obscure feature to assign the fleet's /
army's retreat orders to members was broken.  It ignored ownership,
and thus could copy some other player's retreat orders.  Abusable.
Copying the first member's retreat orders is less than useful, in
particular for group ~.  The code now chooses the first one in the
same sector with RET_GROUP set.  RET_GROUP is never set for group ~.

Fix .SA.

Cosmetic change of markup.

Fix the previous revision.

Updated from man/emp_client.6 rev. 1.9.

Add notes for Client Implementors.

(output,screen): In ASCII mode, remove standout bit if the client
does not support standout mode.
In UTF8 mode, remove SO/SI characters if the client does not support
standout mode.

Add the See Also "Moving" to the commands described
in the info file.

Correct the number of commands in the described in the text.

Changed the name of Attack and Move concept files and
a number of Subjects files to prevent name
conflicts where the only the case is difference.
Attack -> Attacking
Moving -> Transportation
Move -> Moving
Market -> Commerce
Nation -> Nations
Production -> Producing
Update -> Updating

(EFF_OWNER): The assertion that EFF_OWNER implies ef_read() sets
player->owner is no longer used.  Use it to indicate that owner can be
accessed through struct genitem, similar to EFF_XY and EFF_GROUP.
(xdump): Fix ownership checking.  Old version worked only when
nxtitem() set player->owner.

(empfile): Set EFF_OWNER for EF_LOST.  This fixes `xdump lost'
disclosing other countries' losses.
(loststr): Document implications of EFF_OWNER.

(empfile): Set EFF_OWNER for EF_NATION.
(natstr): Rearrange members for EFF_OWNER, document.
(nat_ca): Unused so far.  Clean it up, add most missing selectors.
(fileinit): Use it.  This implements `xdump nation'.

(fileinit): Simplify setting map file size.

(nxtitem): Skipped unowned sectors (could not occur), ships, planes,
land units and nukes for selection type NS_LIST.  Thus, unowned items
explicitly selected by item number are skipped, while the same items
selected implicitely by group, area or whatever are not.  This makes
no sense.  Some callers break the nstr_item abstraction to prevent the
skipping, so they get all items regardless of how they were selected.
All other callers filter out unowned items.  This skipping misfeature
dates back to the very oldest known versions of Empire.  Remove it.
Also remove skipping prevention hacks from callers.

(tend_nxtitem): Now identical to nxtitem(), except it lacks some extra
normalization of item coordinates, which should have no effect.
Replace by nxtitem().

(findlost): Static linkage.

(LIBS, NTLIBS): Remove libsubs.a.

(main): Fix checking of ef_open()'s value.  Fixes program exit status
when files can't be opened.

(logerror): The version from libcommon.a does the right thing since
log.c rev. 1.17.  Use it, remove local version.

(log_last_commands): No longer referenced by library code, remove.

(ef_write): Don't call log_last_commands() directly, use oops().  If
it's worth logging commands, then it's worth dumping core.

(debug): Move to log.c, so other programs don't have to define debug
themselves just to use logerror().

Clean up the previous revision; no functional changes.

Update patch for some minor offset changes.

Update patch for pr.c:1.12

Remove some redundant casts.

Clean up the previous revision.  No functional changes.

(nxtitemp): Remove last parameter.  Non-zero argument doesn't make
sense and is never passed.  Callers changed.

Document implications of EFF_OWNER.

Document struct equivalence hack and implications of EFF_XY and
EFF_GROUP.

Client UTF-8 support.
(login): New parameter utf8.  If set, request option utf-8 from
server.
(expect, recvline): Split recvline() out of expect().  Replace or
remove some unhelpful diagnostics.
(eight_bit_clean): New.
(screen): If eight_bit_clean is set, highlighting is switched with
SO/SI.  Else characters with MSB set are highlighted.
(main): New option -u to request UTF-8 and set eight_bit_clean.

Login options.
(player): New member flags.
(PF_UTF8, NF_UTF8): New PF_UTF8 replaces NF_UTF8.  Users changed.
(options_cmd): New.
(login_coms): New command `options'.
(toggle): Revert to the previous rev.

(coun_cmd): Fix return value when country doesn't exist.  No
functional change, as caller treats old and fixed value the same.

(uprmtrd): Remove the unsigned char cast for cp, not necessary.

(sendcmd): Change argument cmd to string.  Callers changed.
(fnlist, fn, USER, COUN, QUIT, PASS, PLAY, LIST, CMD, CTLD, KILL):
Unused, remove.

(getele): ~p printed user text as normal text.

Support UTF-8 encoded Unicode for user communications.
(NF_UTF8, togg): New client flag.
(flash, wall): User text input filtering.
(parse, prmptrd): Normal text input filtering.
(uprmptrd, ugetstring): New, to get user text.
(getele, flash, wall): Use them.
(getele): No need to filter out funny characters; input filtering
takes care of them.
(sendmessage, ufindbreak): Work on user text.
(uprnf): New, to print user text.
(rea, gamedown, show_motd): Use it.
(prnf): Unused, remove.
(pr, upr_player): Normal text output filtering (with highlighting).
(pr_flash): User text output filtering.

(lnd_support): New parameter defending, to report defensive support in
the news correctly, like dd() and sd() do.  Reported by Doug Wescott.

Initial Release.

Update for Make.sysdefs:1.9: Get rid of -DPOSIXSIGNALS.

Use NULL instead of 0 where appropriate.  No functional changes.

(nstr_comp): Interpret identifiers as strings only when the
other operand is a string-value selector.
(nstr_comp_val): Don't interpret identifiers as strings.
(nstr_resolve_id): New parameter string_ok.

(build_bridge,build_tower,retreat_ship1): Mark "X" on sector
where mines were found when retreating.  Remove "X" when the
bridge span or tower is built.  Closes #1149809.
The bmap is now updated when the bridge span or tower is built.
A bulletin was added for when mines were swept when
retreating.

(ship_ca): Selector mobquota clashes with mobil, rename to mquota.

(land_ca): There are two selectors rad: radioactive materials and
reaction range limit.  Rename the latter to rmax.

(MINES_MAX): Change to 32767.  MINES_MAX was not adjusted from 65535
when sct_mines was changed from unsigned short to short in rev. 1.15.

(servercmd): Parsed BTUs into nmin and minutes into nbtu.  Fix.  No
functional change.

(main,install_service) [_WIN32]: Move the construction of full path names
for program file and econfig file for installing a service to main()
instead of install_service(). This is because current directory
information is lost because of the use of chdir() to move the current
directory to the datadir.  This change also fixed the setting of a
relative directory for the econfig file.  The service starts in an
arbitrary directory so a relative path can not used by the service.

(ship_grow_scale, plane_grow_scale, land_grow_scale): Change from int
to float.  This allows more control of the max. efficiency gain, and
makes it similiar to ship_mob_scale etc.  Closes #1025600.

(prod): All sectors produce when efficiency >=60%, not only enlistment
centers.

(xdprval): Only use of prnf() that is not for telegram text.
Replace by pr() in preparation of output conversion of telegram text
for the UTF-8 option.

(): Improve the explaination of the options in wire command.
Make wire.t similiar to the read.t and incorporate the
improvements from #1010010.

Update for 4.2.21.

(wu,typed_wu): close() a sent telegram before waking up other threads.
Remove s_char and register on the way.

(turn): Check whether fclose() succeeds.

(): Remove -e from the list of no parameter options in the fairland
manual page.  There is not -e without parameter option.
There is -e with parameter option and it is properly
documented.

(rea): Set parameters for read to [yes|no|<CNUM/CNAME>].
Add parameters for read in the command list and the info pages.
Closes #1010010.

Document Windows usage.  Since this is useless and confusing on other
platforms, it is only included in if number register w is non-zero.

Rewrite.

More 4.2.20 changes.

Remove .UC; we're not be part of BSD.

(lnd_mar_one_sector): When units in different sectors march, only
units in the sector with the highest-numbered unit were interdicted.
A marching non-spy made all marching spies visible to interdiction
regardless of location.  Interdict sector by sector, similar to
shp_nav_one_sector().  Interdict spies only if they are together with
non-spies in the same sector.

(shp_nav_one_sector): When shp_interdict() removed the second ship in
a sector, interdiction loop followed dangling pointer and crashed.
Simplify & fix.
(mlist): Member done no longer used, remove.

Fix the previous rev.

[_WIN32]: Add #define WIN32 whereever winsock2.h is used. There is bug
in the VC7 version of winsock2.h.  In version VC7, the #pragma
pack(push) and pack(pop) have an #ifdef !WIN32 add to them.  However,
if WIN32 is not defined at the beginning, the push is done but by the
time the pop is reach something else has set WIN32 define so the pop
is not done, and it changes the default pack from /Zp2 to /Zp4, which
makes the data structures incompatible between objects with winsock2.h
and without winsock2.h.  By adding the WIN32 both the push and pop are
not done.

(nstr_parse_val): strtod() recognizes a few strings as numbers that we
want to recognize as identifiers, notably "inf".  Recognize
identifiers starting with letters before numbers.

Fix the previous rev.

(produce): When player->simulation, level production executed item
production code for item I_NONE.  This tries to put products into
vec[I_NONE].  Until item.h rev. 1.12, vec[I_NONE] was unused and zero,
so it worked.  Since then, it's a subscript out of bounds, clobbering
two bytes of stack.  The (unpredictable) stack contents could make the
code believe that there's not enough space for the `products', which
then reduced predicted production, typically to zero.

(guerrilla): Fix saturation of civilians (new in rev. 1.9).  It
created anti-uw, which made the next checksect() oops.

(_c_copyright_header, _ipglob_copyright_header, copyright_header):
_c_copyright_header and _ipglob_copyright_header are almost identical.
Replace by copyright_header, interpolate variable parts with printf.

(wrmakesrc, wrauth, wrgamesdef): Make generated file comments match
our usual style a bit more closely.

(wrgamesdef): Fix generated comment on overwrite.

Update to match current prod.c.