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)

Fix fd leak in error cases. Found by cppcheck.

Fix realloc error. Found by cppcheck.

Don't use the same name for two parameters.

Use __c99inline from sys/cdefs.h

Fix a few of the games to use getprogname() in their usage() functions.

Suggested by Joerg.

XXX There are probably many more that have their progname hard-coded, but
XXX I'm not up for a wholesale roto-till at the moment.

Since this game is installed as 'boggle', fix the usage() text to match.

add one i saw on irc today

Avoid undefined behavior, from Chris Spiegel in PR 43860.

Rework the game startup so it uses curses nicely. There are now menus
and stuff for picking scenarios and ships and all that.

Don't define esc-[ as a custom key sequence because it steps on vt
function keys.

Fix some additional typos/grammar slips.

Fix dog-Latin. The plural of wumpus could be wumpi, wumpus, or even wumpora,
but not 'wumpii'. Where does this -ii crap come from?

Follow the Fundamental Theory of Algebra. Disallow factorising of
numbers less than 2 as it is not
- naturally unique (negative numbers)
- finite (0)
- non-empty (1)

Discussed with the kristaps and wiz

\\ -> \e

cast isblank(3)'s argument to unsigned char.

rename pollard_pminus1->pollard_rho for consistency

-Fix an old bug in the "pollard" code: it gets its argument passed
 by reference, and changes the value behind the pointer under some
 circumstances (basically if it finds more than 2 different factors).
 It also calls itself if it finds a factor which is not considered prime
 (by openssl's miller-rabin check) and uses the call argument afterwards.
 This doesn't work -- we need to copy the argument into its own storage.
-Modify the code to do the "rho" algorithm as was initially announced.
 It takes somewhat longer in rare cases, but still works in cases where
 the "p-1" algorithm is unusable. This might fix PR misc/43192
 by Luiz Henrique de Figueiredo.
-Add some optional debug support, minor cleanup.

Document the options that actually exist, instead of some other
completely unrelated set. Bump date. (first time since 1993!)

Handle the "diedtime" field of the player log (which is not the high
scores file, the append-only log of all games) as int32_t instead of
time_t. Log files from before the 64-bit time_t change can now be read
again; however, log files from the last year of -current are hosed.
All none of you who play larn, take note...

fflush the score file while signals are blocked to decrease the chance of
it getting corrupted.

fix an obvious flaw in bounds check: the array of precomputed primes
could be overrun if its last entry (65537) was a factor of the input
(this does not affect PR misc/43192 -- the factors are much larger
here: 7742394596501*159455563099482401)

Add backslash before a dash to get a minus.

Sync with reality. Bump date.

handle ctime returning NULL.

ctime can return NULL.

fix typo.

handle ctime returning NULL.

remove #ifdef SVR4

Some improvements to the new UI.

reorg for clarity and exposition.

Better user interface. From OpenBSD, written by Paul Janzen quite a
long time ago. A few minor adjustments by yours truly.