KNF and WARNS=3

Remove the -t flag from the usage.

KNF and WARNS=3

Whitespace fix

KNF and WARNS=3 fixes

KNF and WARNS=3 fixes

KNF and WARNS=3

add ching, and also break up the SUBDIR line a bit more to make future
additions easier.

now if we could only get "dungeon"...

Initialize lnum and unum to avoid a "might be used uninitialized"
warning.

missing paragraph break from the original

ching(6) -- The Book of Changes, not seen in BSD since 4.4BSD.

The C code is a complete rewrite done by Guy Harris for 4.4BSD and is
not the original from v7 at all. Unfortunately, the data file itself,
without which the rest isn't useful, was not freed until Caldera
released all of 32v a few years ago, so it was not in 4.4lite.

The data file, the ching nroff macros, and the driver script are under
Caldera or Caldera+Berkeley copyright.

I've partially redone the driver script from 4.4 (there was no point
in using a temporary file for the hexagram), which was already
partially redone from 32v. (As an aside, it is nutty that the script
needs a copyright so many times longer than the code.)

I've renamed "cno" to "castching", "phx" to "printching", (the
traditional names were opaque), and put them in /usr/libexec/ching
instead of the v7 /usr/games/ching.d. The data file and nroff macros
are in /usr/share/games/ching

Fixed error on a nonexistent mail spool file, when countmail can't find it\n\nCloses PR misc/29526\n\n\(Code submitted in PR misc/29526 by Andreas Eder)

Fixed a comment that said the factors in the output would be strictly
ascending.

Changed the type of data_table to be unsigned char[] instead of char[],
because it is initialized with values >= SCHAR_MAX. Also removed two
unnecessary bit mask operations. These changes do not affect the actual
behavior. Also replaced an "#ifdef notdef" with "#if 0". All these
changes are mainly for reducing lint(1) warnings.

Fixed buffer overflow with long command line arguments.

Drop trailing space.

 * Change program logic to not nest needlessly
 * Search all acronyms databases, and don't force the user to
   know in which category to look (-t is gone)
 * If an acronym is not found in the database or by whatis(1),
   also check pkg_info(1). Per PR bin/30539 by Geert Hendrickx
   (geert.hendrickx@ua.ac.be)

OK'd by Julio M. Merino Vidal <jmmv>

Change marker from "pmppc" to "powerpc" because a NetBSD-macppc build
is also affected.

Add an initialization to placate -Wuninitialized.
Marked with XXXGCC for pmppc (found while compiling for it).

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.