caesar: WARNS=6, strict bool mode

The rotation is validated to be nonnegative, therefore use unsigned int
for it.

Make the default WARNS for games 4. The only game that needs to set WARNS
to anything else now is rogue, so clear WARNS from the other makefiles.

branches: 1.7.10;
Raised WARNS to 4.

- Makefile cleanups

Use ${INSTALL}.

clean up import

specify man pages the new way.

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

Lots of minor fixes resulting from reading these man pages in detail.

Ispell.

add missing .Nm calls and MLINKS. from spz@serpens.swb.de

branches: 1.5.2;
WARNSify

clean up import

Update to use new -mandoc macros.

Add RCS indentifiers.

branches: 1.1.1;
Initial revision

caesar: remove lint boilerplate

Lint nowadays knows __dead functions, so it no longer needs the
NOTREACHED comments.

games/caesar: fix error handling for I/O

The '!= 0' is needed for an upcoming fix to lint's strict bool mode.

The additional text in the error messages is necessary because errx only
outputs the given string, with no details from errno. Since ferror does
not set errno, no such details are available, so just output a generic
message, which is still better than just "caesar: <stdout>".

caesar: WARNS=6, strict bool mode

The rotation is validated to be nonnegative, therefore use unsigned int
for it.

Remove the \n and tabs from the __COPYRIGHT() strings.

branches: 1.21.20;
Simplify error checking.

Replaced one instance of err() with errx(), as the value of errno might not
be correct at that time.

Removed the unnecessary #include <ctype.h>.

Added an empty line to conform to KNF.

Aligned the usage message from the program with the one mentioned in the
man page.

- caesar(6) is not a SETGIDGAME, so we don't need to revoke any privileges.
- replaced direct POSIX read/write with stdio, which made the code simpler.
- added even more error checking.
- restructured the code to make each function fit on one screen
(well, except one).
- now the code reflects the intended purpose of the program.
- return 0 instead of EXIT_FAILURE on success.

* 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

Remove uses of __P.

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

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

Use setgid(), not setregid().

Use err(1, NULL) and warn(NULL) for `out of memory' error messages.

Add use of `const' where appropriate to the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Some such changes were previously covered in PRs bin/6041, bin/6146,
bin/6148, bin/6150, bin/6151, bin/6580, bin/6660, bin/7993, bin/7994,
bin/8039, bin/8057 and bin/8093.

Drop setgid privs early, via OpenBSD
Reported in PR 5970 by Joseph Myers <jsm@octomino.demon.co.uk>

mark non-returning functions (PR#6144 by Joseph Myers <jsm28@cam.ac.uk>)

minor KNFify

WARNSify

Removed "extern int errno"; Added #include <errno.h> if not already present.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

While everyone's beating up on rot13 - use 'exec' to loose the shell
process.

$* -> "$@", not that it does anything. yet.

keep it simple.

Don't rely upon /usr/games being in PATH, include it explicitly.

Fixes PR 9968 by yyrkoo@imrryr.org.

clean up import

branches: 1.1.1;
Initial revision