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.)

Remove description of -a option and optional log file name, since
neither has done anything in a long long time. Add previously
undocumented -s/-f (slow vs. fast) options, although I question the
utility thereof and suspect they should just be removed.

Bump date (first time since 1993)

sprintf -> snprintf

sprintf -> snprintf.

"abbreviation" has two Bs.

Use random() instead of rand(), so we get something like random
numbers out.

This changes the "tournament codes"; that is, the same code will give
you a different game now from what it used to. (This is because the
codes are basically random seeds.) I really really doubt anyone cares
about this, especially since the tournament feature appears to be
undocumented.

Remove obviously botched test for "fast mode" based on terminal speed.
Default to "fast mode" as ~nobody has a 300 baud terminal any more.
("Fast mode" apparently controls whether short-range scans are printed
by default at certain times.)

Use getopt to handle the remaining (all currently undocumented) options.

Remove last traces of undocumented -p option (apparently related to
process priority) and documented -a option, neither of which have
done anything since CSRG days.

Don't give special privileges to uid 13107.

__attribute__((__noreturn__)) -> __dead

Null for pointers, not 0.

Fix score printing so columns line up.

Abolish cgetc(). It contained one line of code, which was wrong.
Call getchar() directly, and handle EOF properly instead of looping
(in some cases) or pretending that EOF is 0 (which it isn't).

Remove unnecessary initialization that silenced a compiler warning in 1997.

Sprinkle some blank lines for readability.

Split up lines > 80 chars. Object files unchanged.

Don't use literal ^G's in string constants. Use \a.

KNF: brace and comment placement. Object files identical.

whitespace

KNF: fix formatting of preprocessor directives

ANSIfy function declarations. No object file diffs except for two functions
whose K&R declaration didn't fully match the prototype (char vs. int) and
the diff of the disassembly of those looks plausible.

KNF. No change to .o file.

Change getdate() to getdatestr() since the former maybe present on systems
supporting X/Open System Interfaces Extension (XSI) getdate() in <time.h>.
This prevents a function prototype conflict.

Remove trailing whitespace.

define YY_NO_INPUT where appropriate, from Kurt J. Lidl per PR misc/41160

Sort sections.

Don't nest displays.

Convert to mdoc markup. With input from wiz.

\@ is not a valid escape sequence.

Avoid hardcoded "too large" constant that caused FPE on the vax.
XXX: Should pullup to 5.0

Simplify previous; suggested by Paul Goyette

Fix sign-compare problem that crept in due to working from an old tree.
Whoops. won't happen again...

grammar fix

Rework curses handling.

The most visible change is that the scrolling buffer of messages now
actually scrolls and keeps history, and you type on the bottom line only.
For now you can page through the history with ^P/^N; that's not ideal but
one can't fix everything all at once.

Use resizeable array code (roughly the same that I floated a while ago
in connection with make) to hold the scroll.

Move newturn() from pl_7 to pl_2.
pl_7 was mostly curses/display stuff, is going to be all display stuff.
pl_2 is where the main loop lives.

Move all curses bits to pl_7.c.

Fix usage message

chdir to /var/games/sail while running, to simplify name handling.

Don't crosswire the two kinds of path names, and don't croak on
scenario numbers > 99.

Turns out the W_FILE message wasn't ever used.

Create some abstraction for sending messages.
Make a send and receive function for each possible message.
Make these have useful argument signatures.
Hide the list of message codes inside sync.c.

Don't use perror()

constify the version string

Sprinkle some "bool". And don't use '\0' to mean 'false'.

Use \a instead of \7 for BEL.

Prune a comment referring to objects that (AFAICT) didn't even exist in -r1.1.

Don't cast the return value of calloc().

Lines <= 80 chars, remove trailing whitespace, other formatting adjustments.
No functional change.

Remove some unnecessary casts.

String buffer cleanup. sprintf bad.

Don't use char for random state variables; use int or bool.

Fix list markup. Use .Ic for key as suggested by wiz.

Fix preamble

Remove redundant .br

Fix markup

Fix preamble

Fix markup

sprintf -> snprintf

Move HISTORY to proper place.

Edit. Sync with reality, at least partway. Bump date.

Sort options. End sentence with a dot.

mdocify. Partly merged from OpenBSD. Not by any means perfect yet.

More fixes from OpenBSD: use err instead of perror, don't hork on EOF
during the initial setup questions.

Check for failure of fgets when reading the captain's name, and don't
allow it to be empty. From OpenBSD.

Use errx() to send fatal error messages. From OpenBSD.

Use errx/warnx instead of printing to stderr. From OpenBSD.

Use err(3) and errx(3), not perror and fprintf to stderr. From OpenBSD.

Braces around a nested if, for clarity; suggested by diffing against OpenBSD.

Fix bogus conditional caught by today's make. It looks as if MKHIDEGAME
has never worked since it was introduced in 2002... clearly a lot of
people care about that.

It should work now, but because it's evidently never been tested it
may also require setlist adjustments.

use %zu to print size_t

fix -Wsign-compare issues

Fix fiance/fiancee confusion. Fix another typo.
Jason McIntyre <jmc@kerhand.co.uk>.

Fix insure -> ensure confusion. From Jason McIntyre <jmc@kerhand.co.uk>.

PR/39923: Valery Ushakov: trek(6) can spin, consuming 100% CPU on powerpc
On systems where char is unsigned the number of black holes can end up being
very large (instead of negative).

more MKSHARE == no fixes:
- move installation of atf pkg-config files out of share/atf/
- don't install spell or ching share files
- update some directory entries to not have 'share' tag

now my MKLINT=no MKSHARE=no MKPROFILE=no builds work again.

Use uint32_t and include <stdint.h>, instead of using u_int32_t
and including <sys/types.h>.

Make this build on Mac OS X.

Install unstr:
* in games/fortune/Makefile, add unstr to SUBDIR;
* in games/fortune/unstr/Makefile, include ../../Makefile.inc;
* in games/fortune/strfile/Makefile, add MLINKS for unstr man page;
* in distrib/sets/lists/games/mi, add new files.

Build strfile both as a host tool and as an installed program:
* in games/fortune/strfile/Makefile, build strfile as a
  regular program instead of as a host tool;
* add tools/strfile directory to build strfile as a host tool;
* in tools/Makefile, add strfile to SUBDIR list;
* in BSD.*.mk, define TOOL_STRFILE variable;
* in games/fortune/datfiles/Makefile, use TOOL_STRFILE when creating
  databases at build time;
* in distrib/sets/lists/games/mi, mention usr/games/strfile.

Make this pass WARNS=4:
* u_int32_t -> uint32_t, and remove ifdef test.
* add function prototypes.

Both legs of the conditional operator must return the same type.

Wrap compiler-specific flags with HAVE_GCC and HAVE_PCC as necessary. Add a few flags for PCC.