Move scratch variables of main to main, instead of having them be
file-static.

Add strategic assertion, now passes -Wstrict-overflow.

Remove ifdefs for V7 (as opposed to older) unix.

Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

fix previous, thanks asau@

Don't use ++ to set flags to true. Now passes -Wstrict-overflow.

Pass -Wstrict-overflow.

if 0 is used for the dimensions, compute the maximum size.

Bump date for previous.

- improve wording and overall program description (notably for `-f');
- improve macro usage;
- bump date.

From Bug Hunting.

- correct `usage' message;
- improve wording.

From Bug Hunting.

Add a remarkably percipient fortune cookie.

Fix formatting of Churchill quote on declarations of war.

- improve wording
- sort option list
- add argument names in option list and descriptions and adjust table width
- improve punctuation
- bump date

From Bug Hunting.

src/games/fortune/fortune/fortune.6:
 * reference strfile(8) in `SEE ALSO' section.

src/games/fortune/strfile/strfile.8:
 * remove redundant argument to (successive) `.Nm' macros.

src/games/fortune/datfiles/fortunes2:
 * remove (the least complete variant of a) double fortune.

Patch submitted by Bug Hunting.

WARNS=5, and make WARNS=5 the default for src/games.

WARNS=5

WARNS=5

Typo. Reported by I-forget-who-but-they'd-probably-rather-not-be-credited,
during the -6 freeze, and then forgotten until now.

Remove unused fields

Consistently use major/minor for NetBSD version.

Add missing terminfo

rcsid, warns

Sort sections; remove trailing whitespace.

add colorbars

new colorbars program from Nathanial Sloss

Attribute the `Consequences, Schmonsequences...' quote to Daffy Duck.

use modern shell constructs

allow multiple '-f acronyms' arguments to be given, which makes this behave
more consistently with ACRONYMSDB, which may also point to multiple files

Add capfile(5) to describe the termcap format.
Adjust various man pages and other documentation to point to capfile(5)
instead of termcap(5).
Remove getcap(3) as curses hasn't been building it for a long time.
Punt wrterm.c as tset no longer uses it.

Fix various format string mismatches.

Man page and usage cleanups from Bug Hunting in PR 46103.

Fix conflicting variable definitions

PR/45842: Henning Petersen: compare fgets with NULL not 0

grumble.

Oops, I forgot to actually implement the checksumming code for the new
savefile format, so any savefiles generated yesterday can be tampered
with. Oh well. While here, tidy up the crc code.

Redo save file handling. The old save files were unportable, had no
magic number or versioning, relied on random(3) never changing to a
different implementation, and were also saving pointers to disk and
reading them back again. It *looks* as if the pointers thus loaded
were reset before being used, but it's not particularly clear as the
main loop of this thing is goto-based FORTRAN translated lightly to C.
I've changed the logic to null these pointers instead of saving and
loading them, and things seem to still work.

The new save files have a header, support versioning, write only sized
types in network byte order, and for the toy encryption to discourage
cheating do something self-contained instead of using random(3) as a
stream cipher.

Because between the original import from 4.4 until earlier today
trying to save would result in SIGSEGV on most platforms, it's
unlikely anyone has a save file, but just in case (since the pointer
issue appears to be nonlethal) I've kept compat code for old save
files.

Make this not crash on machines that are (a) 64 bit, or (b) have signed
chars by default (i.e., almost all machines). Makes it possible to save
the game. This has been broken since 4.4 and probably ever since the
FORTRAN -> C translation.

Crash reported by Petri Laakso in private mail.

Per [1] the speed of light in a vaccum is exactly 299792458 m/s.
Per [2] a furlong is 220 yards and a yard is exactly 0.9144 m.
Per [3] a fortnight is 14 days.

As I didn't find a good authority for what definition of a day a fortnight is
measured in, I'll assume here a day is 86400 SI seconds.

Thus, the speed of light in a vaccum is approximately
1.80*10^12 furlongs per fortnight.

1.80*10^12 = 299792458*86400*14/(220*0.9144)

[1] Resolution 1 of the 17th meeting of the CGPM (1983)
http://www.bipm.org/en/CGPM/db/17/1/
[2] Weights and Measures Act 1985
http://www.legislation.gov.uk/ukpga/1985/72
[3] The Concise Oxford Dictionary, 5th Edition, 1964, p. 480

handle EOF in input

grammar fix

remove duplicated #defines (in a usually unused part of the code)

Use tiparm instead of vtparm.

reinstate NULL cast by request, where the NULL was being passed as a vararg

NULL does not need a cast

No need to cast double to ubig (aka uintmax_t) through int.

This change prevents a modulo by zero in a invocation such as:
primes 18446744073709551000 18446744073709551615
on a LP64 machine.

Use __dead

Mark error() as dead, so that validation for setclipped works correctly.

Avoid non-literal format strings.

Use __dead and __printflike instead of __attribute__.

add printf attributes.

Uses non-literal format strings

-Wno-shadow is no longer needed for HAVE_PCC, here

use const char [] for format.

Remove redundant set of prototypes for local functions, exposed by
previous.

Move some things around so declarations appear in a reasonable order.

Use __dead instead of __attribute__((__noreturn__)).

document non-literal string formats

avoid non-literal format string

eliminate/document non-literal format strings

eliminate non-literal format strings

avoid non-literal format strings.

kill homebrew error functions

document non-literal format strings

- document non-literal format strings
- avoid strict aliasing violations, but adding an intermediate function.

Fix up some lint.

Fix up still more casts; use NULL instead of (char *)0.

Remove *another* set of useless casts. sheesh

Remove more silly casts.

drop private (and wrong) definition of NULL

Use the right type for the malloc wrapper function, and don't cast the
return value.

(XXX: Except for a pile of allocation macros that produce typed pointer
results; there the typechecking of the result assignment is more valuable
than the warning if the alloc function isn't declared properly. These
macros should go away.)

Remove silly casts.

<sys/wait.h> is standard, so just use it instead of fiddling about with
ifdefs for long-dead systems. While here, remove union wait.

abolish references to index/rindex

Remove the PYRAMID_BUG code. I don't think Pyramids are coming back, nor
do we care if their compiler vomits trying to decrement a bitfield.

Don't refer to index() in comments; use strchr().

Include system headers before private headers. Avoids various possible
symbol conflicts.

apply some -O0 with gcc 4.5 and vax.

use DBL_MIN from <float.h> instead of a hard coded small value (that
becomes zero for vax fp.)

include sys/time.h earlier (rather than implicitly via fcntl.h) so that
the "u" variable doesn't shadow stuff, found later in time.h inlines.

kind of a hack, but i don't want to modify time.h either.

XXX: seems kind of annoying

rename a variable from "time" to "stime" (it's a string)

apply some -Wno-error and/or -fno-strict-aliasing.

all of this should be looked at closer, but some of them are not
very trivial.

Use consistent sign in compare, the function is only called with
constants as third argument anyway.

Map RCSID to __RCSID

ANSIfy

Don't use variables as format string.

Kill some pointer indirections. Don't use variables as format strings.

ANSIfy

#if 0 some unused code.

Don't use variables as format string.

Correctly print variables as strings, not as format string.

Properly print string.

Don't use variables as format strings.

ANSIfy

Don't use strings from arrays as format string.

Don't use return value of a function as format string.

Bug fix: in a game with 26 planes, the last one to be allocated wouldn't
be allocated if it was the only eligible one.
From Jonathan David Amery via Debian Bug report 214626.

Fix spelling: it's *Jef* Raskin, not Jeff Raskin.

avoid unnecessary cast (hi wiz)