fix various typos in comments.

- add or adjust /* FALLTHROUGH */ where appropriate
- add __unreachable() after functions that can return but won't in
this case, and thus can't be marked __dead easily

branches: 1.23.46;
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.

sprinkle static

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

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

sprintf -> snprintf

ANSIfy function declarations. Object file diffs cross-checked.

if initscr() fails, exit with a message rather than crash in the
next curses call

branches: 1.16.20;
Coverity CID 3508: Fix file leak.

Redo how void gets init'd.
setup now just creates an empty file. When main starts and loads it in,
it stats and if zero size init's a new location and saves it back out.
Now games.tgz can be shared among MACHINE_ARCH's

Undef bool before including <curses.h>, since it tries to typedef
bool. Otherwise we get (at least on gcc 2.95.3) an empty declaration
warning from "typedef char char;" due to the #define of bool in
phantstruct.h.

Rework to use curses.h only on the target, not on the host.

branches: 1.12.4;
When cleaning up, only close files that have been opened successfully.
When reporting errors for failing to open files, give error text
rather than errno number. Patch from Joey Hess <joey@kitenet.net> for
Debian bug 187251 from Mikael Hedin <micce@debian.org>.

de-__P().

branches: 1.10.2;
Change deprecated curses function calls to new equivalents.

A couple more checks for file descriptor < 3 missed earlier.

Use the symbolic names `SEEK_SET' and `O_RDONLY' where appropriate in
the games.

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.

Make phrase to enter password again clearer

Add braces to make the new egcs happy.

WARNSify, KNFify (inc. removing extremely verbose & useless comments), ...

Various changes to make games compile w/o warnings on the alpha:
Include appropriate includes, delete bogus function declarations,
change sizes of variables and casts.

rcs id conventions

branches: 1.1.1;
Initial revision

- add or adjust /* FALLTHROUGH */ where appropriate
- add __unreachable() after functions that can return but won't in
this case, and thus can't be marked __dead easily

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.

sprinkle static

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

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

sprintf -> snprintf

ANSIfy function declarations. Object file diffs cross-checked.

if initscr() fails, exit with a message rather than crash in the
next curses call

branches: 1.16.20;
Coverity CID 3508: Fix file leak.

Redo how void gets init'd.
setup now just creates an empty file. When main starts and loads it in,
it stats and if zero size init's a new location and saves it back out.
Now games.tgz can be shared among MACHINE_ARCH's

Undef bool before including <curses.h>, since it tries to typedef
bool. Otherwise we get (at least on gcc 2.95.3) an empty declaration
warning from "typedef char char;" due to the #define of bool in
phantstruct.h.

Rework to use curses.h only on the target, not on the host.

branches: 1.12.4;
When cleaning up, only close files that have been opened successfully.
When reporting errors for failing to open files, give error text
rather than errno number. Patch from Joey Hess <joey@kitenet.net> for
Debian bug 187251 from Mikael Hedin <micce@debian.org>.

de-__P().

branches: 1.10.2;
Change deprecated curses function calls to new equivalents.

A couple more checks for file descriptor < 3 missed earlier.

Use the symbolic names `SEEK_SET' and `O_RDONLY' where appropriate in
the games.

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.

Make phrase to enter password again clearer

Add braces to make the new egcs happy.

WARNSify, KNFify (inc. removing extremely verbose & useless comments), ...

Various changes to make games compile w/o warnings on the alpha:
Include appropriate includes, delete bogus function declarations,
change sizes of variables and casts.

rcs id conventions

branches: 1.1.1;
Initial revision