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.

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

Remove the \n and tabs from the __COPYRIGHT() strings.

Remove the \n and tabs from the __COPYRIGHT() strings.

Clarify how one is supposed to use the pointers returned by getutentries()
(the utmpentry.c code), specifically with respect to who owns them and
when to free them. Now they're owned by utmpentry.c, only. Abolish the
freeutentries() function, which was the wrong abstraction; add instead
endutentries(), which flushes out the internally managed memory.

Update callers as necessary. Some (e.g. talkd) had been leaking memory;
others (e.g. syslogd) had been accidentally freeing and reloading utmp
more often than necessary. There are a couple untidy bits in users and
rwhod that someone should look after sometime, maybe.

Fixes PR bin/35131, which was about talkd's memory leak.

Add test for HAVE_PCC with HAVE_GCC which turn off compiler warning flags.

remove clause #3 from my license where there are no other
copyright holders involved.

2-clause my license.

Rename MKPRIVATELIB to LIBISPRIVATE, to make it clearer that this is
a variable that is used by in-tree Makefiles to control behaviour.
(MKsomevar variables are generally intended to be controlled by the end-user)

Remove clause 3 and 4 from TNF licenses

Fix a tip: use pkg_info, not pkg_add to find out which executables
belong to a package.

Add a tip explaining how to run the tests in /usr/tests.

In example, change dd count size and also chmod so others can't read swap.

Add five more tips.

Add "netbsd-tips" fortune database with 23 initial fortunes --
actually hints or suggestions or tips on using NetBSD.
Please share your brief NetBSD usage tips.
If you want to discuss or share suggestions, have a look
at http://wiki.netbsd.se/netbsd-tips wiki page.

Don't crash if we're asked to restore from /dev/null. (woops)

Fix a bit more lint.

Fix some more lint. (There's plenty to go around.)

If we try to load a game and it bombs, keep prompting for another one,
because if we try to continue with a half-loaded game we'll probably
end up in trouble.

Don't cast the return value of calloc.

fix some lint

need errno.h

Update for today's changes: 1-player games are not allowed, and cards.pck
no longer appears.

Fix the OpenBSD housing shortage code.

ret_card(), which I moved earlier, was consequently filed under the wrong
source file name. fix.

We do not need special code for pdp11. Also, because random() returns all
good bits (unlike rand()) we can simplify this.

Typo patrol

This file is also no longer used.

Abolish the initdeck program and the weird little binary file it generates.
The card deck data is now compiled in. (And it always should have been,
even back when memory usage counted. It's small.)

This changes the save format slightly. (I am riding the previous
breakage an hour ago and not providing compat. There should be no
further breakage.)

Source files should be named *.c or *.h, not random other things.

Fold monop.ext (which was extern data decls) into monop.h and deck.h.
Fold monop.def (which had definitions for the extern data) into monop.c.
Fold brd.dat/mon.dat/prop.dat (which were array initializers for some of
the data) into monop.c.

Rearrange includes accordingly. No functional change.

Remove last traces of old save/load code. We no longer need a private
copy of malloc.

Sort out the memory leak fix properly.

New save/load code. Related to (but not the same as) the new save/load
code in OpenBSD.

Note: this breaks compatibility with old save files. However, the old
save files were missing critical information, had a completely insane
format, and didn't work anyway.

I will be riding this breakage with another breakage within a day or
two when I fix the insanity in the card deck handling.

Fixes from OpenBSD.
   - If a player goes bankrupt buying a property (which shouldn't be
allowed, but I guess it is), don't cause them to resign to themselves.
   - Use "NULL", not "0", for pointers.
   - Avoid undefined behavior adjusting the current player after resigning.
   - Don't leak the memory used to store player names.

The last of these is disabled for the moment because we have the pointers
in question tagged const. Will see to this when I'm finished merging.

Fix from OpenBSD: don't say "1 houses".

Bug fixes from OpenBSD:
   - don't truncate unreasonably long player names when bidding;
   - don't allow players to bid more cash than they have.

Bug fix: having $0 does not mean you need to mortgage something. Plus some
tidyup.
From OpenBSD.

More fixes from OpenBSD:
  - initialize in the proper order
  - don't allow 1-player games
  - make the check for players named "done" work properly
  - a couple other minor things

Sanity fixes for input handling. Suggested by OpenBSD, but different code.

A first stab at supporting the housing shortage rules. From OpenBSD.
(note: this is not fully correct - I will fix it when I finish merging)

Don't increment booleans. From OpenBSD

Sanity fixes for input buffer handling. From OpenBSD

Improve the behavior of get-out-of-jail-free cards by returning them to
the bottom of the deck when used. There are still a few other problems.
From OpenBSD.

Rename ->last_card to ->top_card to match OpenBSD (and also to make
somewhat more sense...)

Move ret_card() to cards.c, where it more rationally belongs. From OpenBSD.

A few more whitespace/KNF. I keep hoping I've found them all...

Add the sizes to the extern array declarations. This may not be
necessary, but it's safe and reduces unnecessary divergence.
From OpenBSD.

Use random() rather than rand(). Suggested by diffing against OpenBSD.

Remove unused swap() macro.

Don't misuse random strings as printf formats, even where it's harmless.
Found by diffing against OpenBSD.

One more (hopefully the last) whitespace fix.

getinp() no longer magically treats the prompt arg as a printf format, so
don't escape % as %%. From OpenBSD.

Move function heading comments to something vaguely like the right place.

KNF/whitespace nits. No functional changes.

Fix bugs in debug printouts. From OpenBSD.

Don't use sprintf; use snprintf.

Don't use perror(); use err/warn instead.
Suggested by diffing against OpenBSD.

Fix capitalization of street names. from pjanzen@openbsd

More typo fixes

Fix formatting bug apparently introduced when the page was mdoc'd. (Had
.Nm, should have been .Pp.)

Also bump date; for some reason this hadn't been done since the original
import.