Raised WARNS to 4.

* fixed <ctype> bugs
* removed ASCII dependency
* added input data validation
* added bounds checking
* added error checking
* handle short reads (from pipes or sockets) correctly
  when trying to decode an encoded stream
* added lint(1) directives

Otto Von Bismarck -> Otto von Bismarck.

It is Bismarck not Bismark.

Sort list, and add Nunavut, which has been a Canadian province since 1999.

Cyprus is part of Europe, not Asia.  Turkey is currently listed in both
Europe and Asia, I'm leaving it that way.

The current President of the United States is George Walker Bush, not George
Walker Bush, Jr. (his father was George Herbert Walker Bush).

Fixed wrong use of the <ctype.h> functions by adding an explicit conversion
to unsigned char. Approved by christos.

Fixed wrong use of datatypes. In wait_for(), a char was used together
with getchar(), in readchar(), a char was used to read input, which in
getuchar() is used as an argument to islower() and toupper(). Also removed
a condition which terminated the process if the user typed in character
255. Approved by christos.

Don't assume /usr/games; use the same path as what rot13 was found at
to run caesar.

A few minor spelling, grammar, and formatting fixes.

Fix a long-standing bug in wump(6) whereby hitting ^d to exit would first
print ``Care to play another game? (y-n)'' (but then exit anyway without
waiting for an answer).

Add -lcrypt where -lcrypto is specified.

Remove a incorrect semicolon at the end of an if().
Magic levels now appear after extensive play testing!

Don't presume phantasia's internal bool (char) is the same as that in
curses.h: define phbool and use it where necessary to avoid declaring
functions with one bool and defining them with the other.

Reviewed by <hubertf>.

Cast last argument of execl to (char *).

Reviewed by <hubertf>.

Avoid arrays of incomplete types (required to build with GCC 4).

Reviewed by <hubertf>.

A preprocessor macro named "srand" was defined which seeded the random
number generator.

Unfortunately, it conflicted with a function of the same defined in
stdlib.h. This was masked by the __P hack, which is now being purged.

I've renamed "srand" to "seedrand" to eliminate the conflict.

Correct typos in German fortune.  Reported by Charles Senger, csenger at UCSD.
Reviewed by <hubertf>.

Convert from libc/threadlib-style mutex protection to pthread style.

(XXX this game totally does not need a thread-safe malloc)

Redo how void gets init'd.
setup now just creates an empty file. When main starts and loads it in,
it stats and if zero size init's a new location and saves it back out.
Now games.tgz can be shared among MACHINE_ARCH's

Slimy worms are clean enough for level 3.

Add (unsigned char) cast to ctype functions

Don't try to catch SIGSTOP...

Indent header description (from jmc@openbsd).
Sort sections and remove a trailing space.

Some improvements from jmc@openbsd:
sort option descriptions, add argument to -m, mention strfile
and add some xrefs.

Replaced strncpy with strlcpy.  Thanks to Peter Postma who
pointed them our in PR #25762. Approved by christos@NetBSD.org.

- sleep a little in autobot mode so that we can see what is happening.
- use refresh() instead of wrefresh()

describe the algorithm better.

Fix typo

Grammar fix, from ray at raylai org via jmc@openbsd.

Had fun with a spell checker.
Also, try to canonize attributions.

- don't catch STOP, TSTP, QUIT; the first you cannot catch and the other two
  you should not.
- don't accept delay == 1000, because it turns to 0 and fix the number parsing
- misc KNF cleanups.

s/the the/the/ (only in sources that aren't regularly imported from
elsewhere).

Undef bool before including <curses.h>, since it tries to typedef
bool.  Otherwise we get (at least on gcc 2.95.3) an empty declaration
warning from "typedef char char;" due to the #define of bool in
phantstruct.h.

Rework to use curses.h only on the target, not on the host.

If the current player has no cards, skip to the other player.
While we're here, make this compile with -DDEBUG.
Fixes PR bin/12530 from Thomas Klausner.

Properly handle -d 1000, OK'd by jsm

It's "its".

Spell "political" correctly and capitalize a last name.

s/feal/feel/

s/venomenous/venomous/

becuase -> because. From Peter Postma.

Encode and decode more characters.  Don't put commas after every
character.  Don't encode whitespace as "...".  Don't fail decoding
across a 1024-byte boundary.  Mark end of message appropriately.  From
OpenBSD but with punctuation taken from ITU-T Recommendation F.1
(03/98).

Uppercase CPU, plural is CPUs.

|fmt; add single quotes around a plus.

Mark unused parameters, nonreturning functions and format functions in
the few cases not already marked.

Hide the cursor if possible.  From OpenBSD.

Fix race in parallel make when creating multiple targets in one operation.

Separate primes manpage from that of factor.  Correct DIAGNOSTICS
information.

Check large factor for being prime before applying Pollard's
algorithm; fixes "factor 2147483647111311".  Correct comment;
algorithm is Pollard p-1, not Pollard rho.  Increase base if p-1
algorithm reaches 1; fixes "factor 99999999999991".  Testcases from
David A Bagley <bagleyd@tux.org>.

Control -d by #ifdef DEBUG; adjust usage message accordingly.  Bug
reported by Malcolm Parsons <malcolm@ivywell.screaming.net> in Debian
bug 84852.

Use standard C abs function rather than local macro.

Mark variables changed between setjmp and longjmp as volatile rather
than just taking their addresses.

Fix grammar (bug reported by Thomas Bushnell BSG <tb@MIT.EDU> in
Debian bug 61846).

Remove uses of __P.

Include <sys/stat.h>.

Include <sys/types.h>.

Include <sys/endian.h>.

Include <limits.h> instead of <machine/limits.h>.

Include <sys/endian.h>.

Include <sys/endian.h> instead of <machine/endian.h>.  Include
<stdlib.h>.

Include <sys/endian.h>.

Use int for wait status.

Rename yn to yncoms to avoid conflict with GCC 3.4 built-in XSI
function.

Rename rint to receive_intr to avoid conflict with GCC 3.4 built-in C99
function.

Use PROG_CXX so we don't need to include -lstdc++ explicitly. From Nick Hudson.

Move AUTHORS section directly before BUGS section.
Suggested by jmc@openbsd.

Typo fixes.

Bump date for previous. Remove superfluous .Pp.

Score is in upper right corner, not upper left corner.  Patch from
Robert Hunter <roh@cryptocracy.hn.org> in Debian bug 209034.

Fix typo.  Patch from Joey Hess <joeyh@debian.org>.

Don't use -fwritable-strings.  Add one more const.

When cleaning up, only close files that have been opened successfully.
When reporting errors for failing to open files, give error text
rather than errno number.  Patch from Joey Hess <joey@kitenet.net> for
Debian bug 187251 from Mikael Hedin <micce@debian.org>.

Refer to 'O' rather than 'o' as command to set options.  Bug reported
by Christian Garbs <debian@cgarbs.de> in Debian bug 191452.

Return proper fortune length from fortlen for fortune files without
STR_RANDOM or STR_ORDERED rather than comparing it to SLEN.  Bug
reported by Simon Williams <simon@no-dns-yet.org.uk>.

Show residual probability rather than non-residual in "No place to put
residual probability" error message.  Distribute residual probability
correctly rather than all to one file.  Patch from Mike Castle
<dalgoda@ix.netcom.com>.

Make dot and winnerdot of type double.  Patch from Jan Vornberger
<jan@uos.de> in Debian bug 197816.

Only score as a Gammon or a Backgammon if the opponent hasn't removed
any pieces, not if the opponent has removed pieces but has had a piece
knocked back to the bar afterwards.  Fixes Debian bug 185101 from
Konstantin Seiler <list@kseiler.de>.

Make tutorial text match actual moves.  From FreeBSD via OpenBSD.
Fixes Debian bug 212478 from Jim W. Jaszewski <grok@sprint.ca>.

Fix capitalisation of compass directions.

Document that pressing return allows you to "fast forward" the game
clock.  Patch from Peter Maydell <pmaydell@chiark.greenend.org.uk> in
Debian bug 216980.

Const the player type variable.

Fix RCS IDs.

Format size_t values using %zd instead of %d, for archs where size_t != int.

Rename .C files to .cc

NetBSD rcsid.

Add dots+boxes

import dots+boxes games.

Various typo fixes from Jonathon Gray via jmc@openbsd.

Improve how various "simple" host tools are built and invoked.

Hobbes was misspelled. PR#23301

Rework how MAKEVERBOSE operates:

    * Don't bother prefixing commands with a line of  ${_MKCMD}\
and instead rely upon "make -s".  This is less intrusive on
all the Makefiles than the former.  Idea from David Laight.

    * Rename the variables use to print messages.  The scheme now is:
    _MKMSG_FOO Run  _MKMSG 'foo'
    _MKTARGET_FOO Run  _MKMSG_FOO ${.TARGET}
From discussion with Alistair Crooks.

Don't redefine NULL

rework to use the newer _MKMSGCREATE (et al) macros

Support MAKEVERBOSE.  (Some host tools still generate verbose runtime messages)

rework to use BUILDSYMLINKS and the <bsd.sys.mk> rule for .c.lo

remove unnecessary /

sort recently added entries
pointed out by Valeriy E. Ushakov on source-changes

add mine:
``NetBSD - no propaganda in the default system, in more than 10 years!''