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.

Spell "versus" right.

Allow saved game filenames up to PATH_MAX. From OpenBSD.

fix quote markup, spotted in openbsd diffs

Fix typo. From OpenBSD.

Don't exit(0) on failure. Use errx() instead of fprintf.

rename text() to wrtext(), aligning it with most other output routines,
and avoiding a compile error on powerpc64.

XXX: this tries to make a ".text" symbol, which already exists...

Latin police: use proper conjugation.
(whether the rest of the affected fortune is valid either is debatable)

When using -lcurses, you also need -lterminfo.
This fixes the build for sun2, and also builds with LDSTATIC=-static,
since archive libraries don't record inter-library dependencies.

Userland now builds and uses terminfo instead of termcap.

OK: core@, jdc@

Close file when finished with it. Found by cppcheck.

Simplify alloc() to avoid ifdef(LINT) workaround.

Close two file descriptors after using them. Found by cppcheck.

The default output file extension is '.dat', not '.out'.
From Lars Nooden via OpenBSD.

Use .%U instead of .%O for URLs.

Christos points out that usage should not use errx, but should use
getprogname.

New sentence, new line.

Adjust SYNOPSIS for cfscores to better match reality. Note in BUGS
that the score file isn't portable. Bump date (first time since 1993,
and first in 2010...)

Split struct betinfo into its own header file so it can be shared
between canfield and cfscores, instead of copy-pasted.

Use NULL instead of 0 for pointer tests. Remove an unnecessary cast.

Send error messages to stderr. Use errx/warnx, not printf.

Correct usage message.

Fix math notation, from PR 41547. While here, fix some potential tab
damage in the same fortune.

Hide the cursor during the game.

Fix bad merge.  (SUBDIR= != SUBDIR +=)

Conditionalize some build directories by ${MKCXX}.

 was disappointed that someone wrote disappointed with one 'p', so I fixed it.

I was disappointed that someone wrote disappointed with two 's', so I fixed it.

use %option noinput nounput

Another spelling mistake.

Whitespace.

Fix a bunch of typos and spelling errors.

Kill two duplicates.

"Ghandi" -> "Gandhi"

Remove 3rd and 4th clauses in christos' license.  OK christos.

Fix '=' for '==' in a test. From NAKAJIMA Yoshihiro in PR 42177.

Disable writing scores when built for /rescue. PR 42009

Dumping all system includes anything uses in one big header file is so 1986.

XXX: Removing unused declarations (thus, unused headers can trigger
XXX: it) sometimes causes gcc to generate substantially different
XXX: code. Dunno why but it looks rather like a gcc bug.

Don't need -DHAVE_REGCOMP any more

Instead of trying to cons up a regexp that ignores case, use REG_ICASE.
This makes fortune -im work; it's been broken since whenever.

Tidy up regexp handling further.

Prune conditional compilation for using obsolete regexp libraries.

Whitespace.

Prevent nasal demons in a case I spotted while preparing the previous.

ANSIfy function definitions.

Sprinkle some size_t in places where it's better than int.
(Reduces casts.)

remove more unnecessary casts

Use correct type for malloc sizes (size_t, not unsigned)

remove unnecessary casts

u_int32_t -> uint32_t

ANSIfy function definitions

Use standard C: 'unsigned', not 'u_int'

remove unnecessary cast

remove unnecessary casts

remove unnecessary cast

Use __dead.

Whn ths cd ws wrttn, thr ws bt shrtg nd vwls wr xtrml xpnsv. Nowadays,
however, we have an ample vowel budget, and bit shortages are a thing
of the past (even in a down economy) so spend a bit to improve
readability.

ANSIfy a leftover function.
Also note some unportable code with a comment. Can't change it because
it'd break save files, though.

sprinkle static

sprinkle static

sprinkle static

sprinkle static