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

sprinkle static and prune some dead code

sprinkle static

sprinkle static

sprinkle static

sprinkle static

sprinkle static

sprinkle static and prune some dead code

sprinkle static

looks as if "ospeed" here was meant to be extern.

sprinkle static

sprinkle static

Don't use a FILE * after fclose(). PR 41832 from Henning Petersen.

don't use char values for functions that can return -1; chars are not always
signed.

Use getopt instead of hand-rolled options code. Document all the arguments
and options. Don't allow the previously undocumented method to change the
maximum number of scores kept per user to be used on the system-wide high
score file. Sort options list in the man page. Bump its date.

don't ignore errors from read(); found by lint

Remove the need for -Wno-pointer-sign. Object file unchanged.

Remove the need for -Wno-pointer-sign. Object diffs checked.

Use random() instead of rand(), and seed with time instead of pid.

Assorted minor cleanups, no functional change:
   - u_int* -> uint*
   - don't make private typedefs of system structures
   - use curses TRUE and FALSE only with curses booleans, otherwise
     true and false;
   - includes cleanup
   - group globals in extern.c by functionality

Object file diffs inspected.

A bit more whitespace.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

ANSIfy. Also, we now have RANDOM_MAX; use it. Object diffs checked...

Rename internal getline() function to get_line() so it does
conflict with the soon to be added getline(3) libc function.

Remove config options for 4.2BSD and 4.3BSD, and associated code. Highly
obsolete, and I doubt most of it still worked anyway.

Remove the non-curses screen handling, since it relies on knowing
about various internals of SVR4 curses, old BSD curses, and/or other
unclean things. (Yes, the non-curses handling still used bits of
curses. Fie.)

Remove config for the return type of signal handlers, which is also a
long-dead issue.

Remove references to config #defines for the thoroughly obsolete
4.2BSD syslog API.

more whitespace

Various minor fixes, mostly from OpenBSD.

Notes on the protocol used by hunt, from OpenBSD.

Whitespace.

Whitespace.

Sigh, more of the same (missed two)

__attribute__((__noreturn__)) -> __dead

ANSIfy function declarations.

Remove two more bogus casts. These *do* create object diffs on amd64,
for no clear reason, but it looks like the diffs are harmless.

Remove bogus casts. Two object file diffs inspected and found to be harmless.

Remove a pile of bogus casts. One object file diff, crosschecked and
found harmless.

Fix two serious string-handling bugs (one exploitable, one probably
exploitable) and also add proper checking/paranoia in several other
places.

sprintf -> snprintf

Fix possible remote DoS of a running hunt game, and prevent a possible
theoretical attack involving >= 1 billion ammo.

a bit more const

sprinkle some more const

remove unnecessary casts

sprintf -> snprintf, plus some use of strlcpy/strlcat where appropriate
XXX: there's still one sprintf left which will take some hacking to expunge.

ANSIfy function declarations. All object file diffs inspected.

Rectify non-compiling code that appears when DEBUG is defined.

Set WARNS=4 here. This got missed on my previous WARNS sweep in games.
Everything appears to compile ok with WARNS=4 anyway though.

Remove global scratch string buffer. Don't zoom off the end while reading
user input, either.

Remove remaining references to sprintf.

Make a couple of the logging/printing functions printf-alikes. This removes
most of the calls to sprintf.

attribute noreturn -> __dead

sprintf -> snprintf

Increase spending on vowels. No object file diffs.

ANSIfy function declarations.x

sprintf -> snprintf

Don't use a variable-size array here. There's not really that much point.

Rearrange where the alternate definition of __dead goes.
It can't be before "include.h" as that includes a bunch of system headers,
but it can't be after either as it also includes a bunch of local headers.
Therefore, it needs to be *in* include.h.

Edit some.

Assorted fixes (mostly grammar, some markup) from OpenBSD.

Paranoia about lengths of login names. From pjanzen of OpenBSD.

Avoid SIGSEGV on users not in password file. From pjanzen of OpenBSD.

sprintf -> snprintf

attribute noreturn -> __dead

We don't need a private #define "unsgn" for "unsigned", especially since
it was used exactly once.

Use the time to call srandom(), not getpid().

KNF/whitespace: format preprocessor directives correctly.

ANSIfy function declarations. Object file diffs checked.

sprintf -> snprintf

ANSIfy function declarations. Object file diffs cross-checked.

attribute noreturn -> __dead

Clean up scorefile handling.

This was writing time_t to disk. Worse, the time_t member was at an
unaligned offset in the structure in question, so after the time_t
change the structure layout depended on compiler-inserted structure
padding. This gives three legacy formats: one with 32-bit time, one
with 64-bit time, and one with 64-bit time and 4 bytes of structure
padding.

And of course the file didn't have a header or version coding or
anything.

The new code writes a structure of well-defined size that should not
receive unexpected padding, and gives the file a header and version
number. It reads that format and any of the three legacy formats,
figuring out which one it's dealing with by inspecting the file
contents. For good measure, it also now handles opposite-endian files,
doesn't bail out unceremoniously unless necessary, and won't croak if
the file is corrupt and e.g. contains unterminated strings.

(Was it worth going to this length? Maybe not. But it didn't seem
right to just leave it, and it's not clear where to stop halfway.)

ANSIfy function declarations.

Some object file diffs, but they are harmless. (Mostly they seem to
come from internal counters in gcc... and in one case the order of two
instructions was harmlessly swapped, which is odd and annoying.)