no longer need -Wno-error=implicit-fallthrough.
don't provide common symbols that are in libterminfo now days.

convert HAVE_GCC == 7 to HAVE_GCC >= 7.

apply -Wno-error=implicit-fallthrough on files that have fallthru
comments but are missed by the current checker.

branches: 1.9.62;
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)

branches: 1.8.18;
Use MKPRIVATELIB instead of NOLINKLIB/NOMAN/NOPIC/NOLINT.

some makefile de-linting

MKfoo=no -> NOfoo=

set MKLINT=no. noted by NAKAJIMA Yoshihiro <nakayosh@kcn.ne.jp>

convert from NOxxx= to MKxxx=no.
include <bsd.own.mk> if testing a MKxxx variable.

common_source -> common, and GC some garbage variables.

Don't install this library, we only need this during the build.

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

backgammon: fix lint warnings

branches: 1.8.42;
Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

branches: 1.6.42; 1.6.48;
KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

backgammon: fix lint warnings

branches: 1.23.4;
fix various typos in comments.

speed limit 80

buffnum is shared between subs.c and fancy.c

Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

branches: 1.18.2; 1.18.8;
Use __dead and __printflike instead of __attribute__.

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

sprinkle static

KNF and WARNS=3 fixes

Remove uses of __P.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

update after change to return value of tputs() third argument

branches: 1.11.2;
This patch adds to backgammon(6) a couple of noreturn attributes
missed in bin/6144.

Patch submitted in PR 8082 by Joseph Myers <jsm28@cam.ac.uk>

The patch below makes the game backgammon(6) use `const' where
appropriate. It also removes some obsolete, unused code to check the
number of users on the system (obsoleted by dm); removes a prototype
for an unused function that was removed earlier; and removes a control
character from one message that was I think intended to clear the
screen on some particular terminal (though I can't find the comment to
that effect in any of the BSD source trees I have handy) but no longer
serves any useful purpose.

From PR 6580 by Joseph Myers <jsm28@cam.ac.uk>.

Also contains two more patches, one in teachgammon/tutor.h (by Joseph)
and one in common_source/fancy.c (by me).

Remove unused function and mark non-returning ones(PR#6076, #6144 by Joseph Myers <jsm28@cam.ac.uk>)

define global vars with `extern' in "back.h", and only define once in
init.c... backgammon works again...

use <termcap.h>

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

Use POSIX tty semantics.

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.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

sprinkle static

KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

The patch below makes the game backgammon(6) use `const' where
appropriate. It also removes some obsolete, unused code to check the
number of users on the system (obsoleted by dm); removes a prototype
for an unused function that was removed earlier; and removes a control
character from one message that was I think intended to clear the
screen on some particular terminal (though I can't find the comment to
that effect in any of the BSD source trees I have handy) but no longer
serves any useful purpose.

From PR 6580 by Joseph Myers <jsm28@cam.ac.uk>.

Also contains two more patches, one in teachgammon/tutor.h (by Joseph)
and one in common_source/fancy.c (by me).

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

Use POSIX tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

add fallthrough

branches: 1.8.30;
Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

branches: 1.6.42; 1.6.48;
KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

backgammon: fix crash on small terminals. From Aran Clauson.

(add a small tidyup from me as well)

Closes PR 58862.

backgammon: fix lint warnings

branches: 1.18.2;
Fix various typos, mainly triple letters.

games: remove trailing whitespace in *.c and *.h

no longer need -Wno-error=implicit-fallthrough.
don't provide common symbols that are in libterminfo now days.

buffnum is shared between subs.c and fancy.c

sprinkle static

KNF and WARNS=3 fixes

s/the the/the/ (only in sources that aren't regularly imported from
elsewhere).

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

This adds a check for memory allocation failure to one place in
backgammon(6). The use of write(2) for the message may seem odd, but
is used in another place in this game. (Actually, a lot of
backgammon(6) could do with being substantially cleaned up.)

Patch submitted in PR 8080 by Joseph Myers <jsm28@cam.ac.uk>

tgetent() handles const correctly in current

The patch below makes the game backgammon(6) use `const' where
appropriate. It also removes some obsolete, unused code to check the
number of users on the system (obsoleted by dm); removes a prototype
for an unused function that was removed earlier; and removes a control
character from one message that was I think intended to clear the
screen on some particular terminal (though I can't find the comment to
that effect in any of the BSD source trees I have handy) but no longer
serves any useful purpose.

From PR 6580 by Joseph Myers <jsm28@cam.ac.uk>.

Also contains two more patches, one in teachgammon/tutor.h (by Joseph)
and one in common_source/fancy.c (by me).

Remove unused function (PR#6076 by Joseph Myers <jsm28@cam.ac.uk>)

use <termcap.h>

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

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.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

backgammon: fix lint warnings

branches: 1.10.42;
Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

branches: 1.8.54; 1.8.60;
Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

The patch below makes the game backgammon(6) use `const' where
appropriate. It also removes some obsolete, unused code to check the
number of users on the system (obsoleted by dm); removes a prototype
for an unused function that was removed earlier; and removes a control
character from one message that was I think intended to clear the
screen on some particular terminal (though I can't find the comment to
that effect in any of the BSD source trees I have handy) but no longer
serves any useful purpose.

From PR 6580 by Joseph Myers <jsm28@cam.ac.uk>.

Also contains two more patches, one in teachgammon/tutor.h (by Joseph)
and one in common_source/fancy.c (by me).

define global vars with `extern' in "back.h", and only define once in
init.c... backgammon works again...

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

Use POSIX tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Coverity CID 558: && should be obviously ||

KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

branches: 1.7.6; 1.7.12;
sprinkle static

KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

branches: 1.14.2; 1.14.8;
Use __dead and __printflike instead of __attribute__.

sprinkle static

Coverity CID 1230: Remove spurious close().

KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

This patch fixes a remaining relic of the backgammon(6) "execute
saved game to continue playing" problem described in bin/6077.

Patch contributed by Joseph Myers <jsm28@cam.ac.uk> in PR 8092.

The patch below makes the game backgammon(6) use `const' where
appropriate. It also removes some obsolete, unused code to check the
number of users on the system (obsoleted by dm); removes a prototype
for an unused function that was removed earlier; and removes a control
character from one message that was I think intended to clear the
screen on some particular terminal (though I can't find the comment to
that effect in any of the BSD source trees I have handy) but no longer
serves any useful purpose.

From PR 6580 by Joseph Myers <jsm28@cam.ac.uk>.

Also contains two more patches, one in teachgammon/tutor.h (by Joseph)
and one in common_source/fancy.c (by me).

remove x bit from save files (PR#6077)

Use symbolic names for signals(!), open(2)-constants, ...
Fix applied as per PR 6058 by Joseph Myers <jsm28@cam.ac.uk>

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

Use POSIX tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Fix DEBUG build.

GC descr and helpm.

Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

branches: 1.17.6; 1.17.12;
sprinkle static

convert __attribute__s to applicable cdefs.h macros

branches: 1.15.10;
KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

update after change to return value of tputs() third argument

branches: 1.12.2;
Add `__noreturn__' and `__unused__' attributes 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).

Most noreturn attributes were previously added in bin/6144, with some
others that were missed then in bin/8082. Previous `unused'
attributes were covered in bin/6557, bin/8058 and other PRs (all these
PRs have already been handled and closed).

This patch makes backgammon(6) check that, if you use the -s option to
recover a saved game, then you specify a file name.

Patch submitted in PR 8081 by Joseph Myers <jsm28@cam.ac.uk>

The patch below makes the game backgammon(6) use `const' where
appropriate. It also removes some obsolete, unused code to check the
number of users on the system (obsoleted by dm); removes a prototype
for an unused function that was removed earlier; and removes a control
character from one message that was I think intended to clear the
screen on some particular terminal (though I can't find the comment to
that effect in any of the BSD source trees I have handy) but no longer
serves any useful purpose.

From PR 6580 by Joseph Myers <jsm28@cam.ac.uk>.

Also contains two more patches, one in teachgammon/tutor.h (by Joseph)
and one in common_source/fancy.c (by me).

Fix args processing of backgammon and teachgammon, closes pr 5345 by
David Brownlee.

Remove unused function (PR#6076 by Joseph Myers <jsm28@cam.ac.uk>)

fix typo, as per PR#5867

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

Use POSIX tty semantics.

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.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Pass the move info around instead of using it as a global.

Crack down on global variables with single-letter names.

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

sprinkle static

Coverity CID 1294: Avoid negative variable used as index.

KNF and WARNS=3 fixes

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

The patch below makes the game backgammon(6) use `const' where
appropriate. It also removes some obsolete, unused code to check the
number of users on the system (obsoleted by dm); removes a prototype
for an unused function that was removed earlier; and removes a control
character from one message that was I think intended to clear the
screen on some particular terminal (though I can't find the comment to
that effect in any of the BSD source trees I have handy) but no longer
serves any useful purpose.

From PR 6580 by Joseph Myers <jsm28@cam.ac.uk>.

Also contains two more patches, one in teachgammon/tutor.h (by Joseph)
and one in common_source/fancy.c (by me).

KNF & WARNSify, and only compile common_source once/* (compiles 25% faster)

Use POSIX tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision