Add testpat(6) to games.

Testpat displays a test pattern in curses(3).

Integrate Warp Kit into the NetBSD build

Fix the build and make it install and run with minimal required changes.

hals_end(6) outputs the last words of the supercomputer HAL 9000 aboard
the spaceship in Stanley Kubrick's famous film "2001 - A Space Odissey".
The source code and output of this program were used to illustrate
an article in the book "Total Interaction". How this looks in print can
be at http://www.netbsd.org/~mbalmer/hals_end.jpg.

hook cgram to the build

branches: 1.27.2;
add colorbars

branches: 1.26.6;
Fix bad merge. (SUBDIR= != SUBDIR +=)

Conditionalize some build directories by ${MKCXX}.

branches: 1.24.32;
moduli utils moved from games to usr.bin.

Descend to moduli too.

add ching, and also break up the SUBDIR line a bit more to make future
additions easier.

now if we could only get "dungeon"...

Add dots+boxes

make factor work with and without openssl.

Include <bsd.own.mk>, to make MKCRYPTO available.

Only build factor(6) if ${MKCRYPTO} != no.

add wtf

hunt isn't "Missing:" anymore ;-)

add hunt.

add countmail.

Sync to 4.4BSD-Lite2

add gomoku game, from 4.4BSD-Lite2

banner, the game, really does belong here, contrary to what CSRG says.

clean up import

Added phantasia and random

I didn't reealize that ching(6) was incomplete

Added adventure, boggle, and ching

run and hide

add pig

Add RCS identifiers.

moved chess into /usr/src/gnugames, fixed "Missing" -- shold have been
comment

added banner and quiz from net-2

branches: 1.1.1;
Initial revision

Add note cautioning against bothering with WARNS=6 until gcc improves
(or -Wconversion is removed from WARNS=6) as it produces loads of false
positives. The most entertaining of these that I've seen this afternoon:

games/hack/hack.apply.c:143:22: error: conversion to 'unsigned char:1' from 'int' may alter its value [-Werror=conversion]
flags.move = multi = 0;
^

branches: 1.15.2;
WARNS=5, and make WARNS=5 the default for src/games.

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

Make the default WARNS for games 4. The only game that needs to set WARNS
to anything else now is rogue, so clear WARNS from the other makefiles.

branches: 1.12.4;
Add new Makefile knob, USE_FORT, which extends USE_SSP by turning on the
FORTIFY_SOURCE feature of libssp, thus checking the size of arguments to
various string and memory copy and set functions (as well as a few system
calls and other miscellany) where known at function entry. RedHat has
evidently built all "core system packages" with this option for some time.

This option should be used at the top of Makefiles (or Makefile.inc where
this is used for subdirectories) but after any setting of LIB.

This is only useful for userland code, and cannot be used in libc or in
any code which includes the libc internals, because it overrides certain
libc functions with macros. Some effort has been made to make USE_FORT=yes
work correctly for a full-system build by having the bsd.sys.mk logic
disable the feature where it should not be used (libc, libssp iteself,
the kernel) but no attempt has been made to build the entire system with
USE_FORT and doing so will doubtless expose numerous bugs and misfeatures.

Adjust the system build so that all programs and libraries that are setuid,
directly handle network data (including serial comm data), perform
authentication, or appear likely to have (or have a history of having)
data-driven bugs (e.g. file(1)) are built with USE_FORT=yes by default,
with the exception of libc, which cannot use USE_FORT and thus uses
only USE_SSP by default. Tested on i386 with no ill results; USE_FORT=no
per-directory or in a system build will disable if desired.

Temporarily add a "MKHIDEGAME" variable that by default blocks the
"install some games into /usr/games/hide" dance. This is ?='ed to "no"
at the top of the file.

XXX What we should do ultimately is either add this to bsd.own.mk or
rename it to MKDM and also have it control whether dm(8) is built at all.

Make this work if HIDEGAME is not set but SETGIDGAME is set.
Also, fix SETGIDGAME -- it was breaking for anything that included
bsd.own.mk

install games that need it setgid.

make hidden games mode 550, group games.

branches: 1.7.2;
enable WARNS?=1 by default, & temporarily disable for the few unclean programs

Do the previous a little differently.

Only use SYMLINKS when PROG is defined.

Do the symlink for HIDEGAME here, via SYMLINKS.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Workaround a toolchain issue by renaming the 'setup' host tool to 'mkdata' to make UAC happy when building with Cygwin. As described here -- http://msdn.microsoft.com/en-us/library/windows/desktop/bb756960.aspx -- 32-bit executables whose filenames include the strings 'install', 'setup', 'update', 'patch', etc. by default will request admin privileges unless an application manifest is provided.

branches: 1.13.54; 1.13.60;
Improve how various "simple" host tools are built and invoked.

Rework how MAKEVERBOSE operates:

* Don't bother prefixing commands with a line of ${_MKCMD}\
and instead rely upon "make -s". This is less intrusive on
all the Makefiles than the former. Idea from David Laight.

* Rename the variables use to print messages. The scheme now is:
_MKMSG_FOO Run _MKMSG 'foo'
_MKTARGET_FOO Run _MKMSG_FOO ${.TARGET}
From discussion with Alistair Crooks.

rework to use the newer _MKMSGCREATE (et al) macros

Support MAKEVERBOSE. (Some host tools still generate verbose runtime messages)

No need to reference .OBJDIR here.

HOST_CC -> HOST_LINK.c

Find setup-program even w/ obj dirs.

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

Allow this to be cross-compiled.

clean up RCS ids

specify man pages the new way.

branches: 1.1.1;
Initial revision

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

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

deprecate register, fixup .Nm usage

clean up RCS ids

branches: 1.1.1;
Initial revision

Oops, I forgot to actually implement the checksumming code for the new
savefile format, so any savefiles generated yesterday can be tampered
with. Oh well. While here, tidy up the crc code.

Make this not crash on machines that are (a) 64 bit, or (b) have signed
chars by default (i.e., almost all machines). Makes it possible to save
the game. This has been broken since 4.4 and probably ever since the
FORTRAN -> C translation.

Crash reported by Petri Laakso in private mail.

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

KNF and WARNS=3

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 does some minor cleanup of adventure(6): an additional
use of const I missed in bin/6041; avoiding a signed/unsigned warning;
marking an unused parameter as such; revoking setgid privileges
(including the saved gid) rather than setuid ones; includes and
function prototypes in setup.c; the string passed to err() should not
end with a `.'. Together with the patch sent concerning adventure's
EOF handling, this synchronises adventure(6) in NetBSD with the Linux
port of the NetBSD games.

Via PR 6557 by Joseph Myers <jsm28@cam.ac.uk>

constify, per PR 6041 by Joseph Myers <jsm28@cam.ac.uk>

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

clean up RCS ids

branches: 1.1.1;
Initial revision

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.

KNF and WARNS=3

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

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

More include additions for exit, abs, strcmp, etc.

remove unused function return values (PR#6079 by Joseph Myers <jsm28@cam.ac.uk>)

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

clean up RCS ids

branches: 1.1.1;
Initial revision

grumble.

Oops, I forgot to actually implement the checksumming code for the new
savefile format, so any savefiles generated yesterday can be tampered
with. Oh well. While here, tidy up the crc code.

branches: 1.14.6;
Remove 3rd and 4th clauses in christos' license. OK christos.

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.

sprinkle static

Remove uses of __P.

remove unused function return values (PR#6079 by Joseph Myers <jsm28@cam.ac.uk>)

constify, per PR 6041 by Joseph Myers <jsm28@cam.ac.uk>

remove main() prototype, as per PR 6867

Remove unused function and unnecessary prototypes, as per PR 6050 by
Joseph Myers <jsm28@cam.ac.uk>

Replace some homegrown string-functions with standard-<string.h>-macros.
Reported by Joseph Myers <jsm28@cam.ac.uk> in PR 6028.

Kill unused parameters, per PR 6023 by Joseph Myers <jsm28@cam.ac.uk>.

KNFify again

KNFify

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

Fix capitalisation of compass directions.

branches: 1.1.1;
Initial revision

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.

sprinkle static

KNF and WARNS=3

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

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

eliminate redundant decls

This patch fixes adventure(6) to use `extern' on declarations of
objects in its header file, and to add corresponding definitions to
init.c accordingly. (See the C standard - relying on linker commons
where there are multiple uninitialised declarations of an object in
the program traditionally works on Unix, but is not standard C.)

The patch also removes a bit-rotten code fragment under #ifdef
OLDSTUFF.

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

This patch cleans up the handling of the variable `saved' in
adventure(6). The handling of this variable is somewhat confusing,
since it is used for two different purposes (controlling the time
required before a saved game can be restored, and controlling various
aspects of dwarf behaviour); in fact, it is also declared twice in
hdr.h. Except possibly when saving a game fails, these uses can never
interfere; when used for controlling dwarf behaviour, we always have
saved == -1. This can be better understood with reference to the
original PDP-10 FORTRAN source (URL in patch, since hdr.h references
the comments of the FORTRAN as still relevant to this version) of
which the C version is a direct translation: the wrong value for
`saved' meant that someone was cheating and had bypassed normal
initialisation. Saving was done by halting and telling the user to
save their core image, so the question of carrying on after saving
failed to open the output file did not arise.

This patch separates the uses of `saved' into uses of two separate
variables.

Closes PR 8005 by Joseph Myers <jsm28@cam.ac.uk>

The game adventure(6) handles EOF on standard input rather
ungracefully. The patch, derived from OpenBSD, improves this
handling.

Sent in in PR 6556 by Joseph Myers <jsm28@cam.ac.uk>.

Improved signal handling as per PR 6051 by
Joseph Myers <jsm@octomino.demon.co.uk>

KNFify

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

clean up RCS ids

branches: 1.1.1;
Initial revision

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

tag decl() __noinline, as inlining it would defeat its purpose
(which is to not have interesting string literals appear in the
compiled binary)

branches: 1.20.2; 1.20.8;
NULL does not need a cast

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.

sprinkle static

convert __attribute__s to applicable cdefs.h macros

branches: 1.16.10;
don't access out of bounds element of static alloc array.
found by Coverity, CID 1519 / Run 5

XXX possible that intention was to start with i = 0.

KNF and WARNS=3

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

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

eliminate redundant decls

This patch fixes adventure(6) to use `extern' on declarations of
objects in its header file, and to add corresponding definitions to
init.c accordingly. (See the C standard - relying on linker commons
where there are multiple uninitialised declarations of an object in
the program traditionally works on Unix, but is not standard C.)

The patch also removes a bit-rotten code fragment under #ifdef
OLDSTUFF.

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

Add __attribute__() to a variable. From PR 6557, after discussion with
Joseph S. Myers <jsm28@cam.ac.uk>

Improved signal handling as per PR 6051 by
Joseph Myers <jsm@octomino.demon.co.uk>

Kill unused parameters, per PR 6023 by Joseph Myers <jsm28@cam.ac.uk>.

change includes to fix compiler warning

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

oops, use <signal.h> not <sys/signal.h> (tholo@sigmasoft.com)

give up setuid-ness. we don't need it. while i'm here, include
<sys/signal.h> and use SIGINT rather than 2.

clean up RCS ids

branches: 1.1.1;
Initial revision

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

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.

sprinkle static

don't try to switch a char and have a case of EOF - use an int.

Coverity CID 2737: Handle linked-lists properly. Use calloc instead of malloc
so that we don't end up storing garbage accidentally and the next pointer is
initialized. If there is an inconsistency in the file abort instead
of dereferencing NULL.

Coverity CID 1195: Check for possible negative index.

Coverity CID 777: Fix NULL pointer deref and de-obfuscate code.

KNF and WARNS=3

off-by-one. aaron@openbsd

off-by-one. aaron@openbsd

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

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

Use err(1, NULL) and warn(NULL) for `out of memory' error messages.

The game adventure(6) handles EOF on standard input rather
ungracefully. The patch, derived from OpenBSD, improves this
handling.

Sent in in PR 6556 by Joseph Myers <jsm28@cam.ac.uk>.

return error on errors (PR#6147 by Joseph Myers <jsm28@cam.ac.uk>)

check for malloc failures (PR#6136 by Joseph Myers <jsm28@cam.ac.uk>)

constify, per PR 6041 by Joseph Myers <jsm28@cam.ac.uk>

Remove unused function and unnecessary prototypes, as per PR 6050 by
Joseph Myers <jsm28@cam.ac.uk>

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

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 RCS ids

branches: 1.1.1;
Initial revision

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

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.

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

branches: 1.19.20;
KNF and WARNS=3

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.

More include additions for exit, abs, strcmp, etc.

Use setgid(), not setregid().

This patch cleans up the handling of the variable `saved' in
adventure(6). The handling of this variable is somewhat confusing,
since it is used for two different purposes (controlling the time
required before a saved game can be restored, and controlling various
aspects of dwarf behaviour); in fact, it is also declared twice in
hdr.h. Except possibly when saving a game fails, these uses can never
interfere; when used for controlling dwarf behaviour, we always have
saved == -1. This can be better understood with reference to the
original PDP-10 FORTRAN source (URL in patch, since hdr.h references
the comments of the FORTRAN as still relevant to this version) of
which the C version is a direct translation: the wrong value for
`saved' meant that someone was cheating and had bypassed normal
initialisation. Saving was done by halting and telling the user to
save their core image, so the question of carrying on after saving
failed to open the output file did not arise.

This patch separates the uses of `saved' into uses of two separate
variables.

Closes PR 8005 by Joseph Myers <jsm28@cam.ac.uk>

The patch below does some minor cleanup of adventure(6): an additional
use of const I missed in bin/6041; avoiding a signed/unsigned warning;
marking an unused parameter as such; revoking setgid privileges
(including the saved gid) rather than setuid ones; includes and
function prototypes in setup.c; the string passed to err() should not
end with a `.'. Together with the patch sent concerning adventure's
EOF handling, this synchronises adventure(6) in NetBSD with the Linux
port of the NetBSD games.

Via PR 6557 by Joseph Myers <jsm28@cam.ac.uk>

return error on errors (PR#6147 by Joseph Myers <jsm28@cam.ac.uk>)

remove unused function return values (PR#6079 by Joseph Myers <jsm28@cam.ac.uk>)

add main() prototype

Kill unused parameters, per PR 6023 by Joseph Myers <jsm28@cam.ac.uk>.

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

oops, use <signal.h> not <sys/signal.h> (tholo@sigmasoft.com)

give up setuid-ness. we don't need it. while i'm here, include
<sys/signal.h> and use SIGINT rather than 2.

Removed "extern int errno"; Added #include <errno.h> if not already present.

clean up RCS ids

branches: 1.1.1;
Initial revision

games: Do not assume <sys/endian.h> is transcluded from <sys/types.h>

prompted by PR pkg/59839

avoid leaving off the trailing nul in a literal string.

Minor tidyup.

branches: 1.13.6;
Oops, I forgot to actually implement the checksumming code for the new
savefile format, so any savefiles generated yesterday can be tampered
with. Oh well. While here, tidy up the crc code.

Redo save file handling. The old save files were unportable, had no
magic number or versioning, relied on random(3) never changing to a
different implementation, and were also saving pointers to disk and
reading them back again. It *looks* as if the pointers thus loaded
were reset before being used, but it's not particularly clear as the
main loop of this thing is goto-based FORTRAN translated lightly to C.
I've changed the logic to null these pointers instead of saving and
loading them, and things seem to still work.

The new save files have a header, support versioning, write only sized
types in network byte order, and for the toy encryption to discourage
cheating do something self-contained instead of using random(3) as a
stream cipher.

Because between the original import from 4.4 until earlier today
trying to save would result in SIGSEGV on most platforms, it's
unlikely anyone has a save file, but just in case (since the pointer
issue appears to be nonlethal) I've kept compat code for old save
files.

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

sprinkle static

KNF and WARNS=3

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

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

Adventure(6) should check for errors (e.g. disk full or quota
exceeded) when writing out saved games. Do so!

Patch supplied by Joseph Myers <jsm28@cam.ac.uk> in PR 8016

This patch cleans up the handling of the variable `saved' in
adventure(6). The handling of this variable is somewhat confusing,
since it is used for two different purposes (controlling the time
required before a saved game can be restored, and controlling various
aspects of dwarf behaviour); in fact, it is also declared twice in
hdr.h. Except possibly when saving a game fails, these uses can never
interfere; when used for controlling dwarf behaviour, we always have
saved == -1. This can be better understood with reference to the
original PDP-10 FORTRAN source (URL in patch, since hdr.h references
the comments of the FORTRAN as still relevant to this version) of
which the C version is a direct translation: the wrong value for
`saved' meant that someone was cheating and had bypassed normal
initialisation. Saving was done by halting and telling the user to
save their core image, so the question of carrying on after saving
failed to open the output file did not arise.

This patch separates the uses of `saved' into uses of two separate
variables.

Closes PR 8005 by Joseph Myers <jsm28@cam.ac.uk>

constify, per PR 6041 by Joseph Myers <jsm28@cam.ac.uk>

KNFify

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

clean up RCS ids

branches: 1.1.1;
Initial revision

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

KNF and WARNS=3

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

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

Ensure trailing newline is present on error and warning messages (lost
in previous conversion from <err.h> functions to ISO C ones).

Don't use <sys/cdefs.h> __COPYRIGHT/__RCSID macros or <err.h> err/errx
functions for host programs - use portable ANSI constructs instead.

Patches from Chris Demetriou.

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

games/adventure/setup.c fails to check for errors when writing its
output. This means that, if the disk fills up at this point during a
build, it would nevertheless fail to return an error status.

The patch below does some minor cleanup of adventure(6): an additional
use of const I missed in bin/6041; avoiding a signed/unsigned warning;
marking an unused parameter as such; revoking setgid privileges
(including the saved gid) rather than setuid ones; includes and
function prototypes in setup.c; the string passed to err() should not
end with a `.'. Together with the patch sent concerning adventure's
EOF handling, this synchronises adventure(6) in NetBSD with the Linux
port of the NetBSD games.

Via PR 6557 by Joseph Myers <jsm28@cam.ac.uk>

some cleanups as per PR#6082 by Joseph Myers <jsm28@cam.ac.uk>

KNFify again

KNFify

clean up RCS ids

branches: 1.1.1;
Initial revision

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

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.

sprinkle static

KNF and WARNS=3

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

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

More include additions for exit, abs, strcmp, etc.

return error on errors (PR#6147 by Joseph Myers <jsm28@cam.ac.uk>)

remove unused function return values (PR#6079 by Joseph Myers <jsm28@cam.ac.uk>)

Kill unused parameters, per PR 6023 by Joseph Myers <jsm28@cam.ac.uk>.

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

clean up RCS ids

branches: 1.1.1;
Initial revision

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

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.

sprinkle static

Coverity CID 570: Remove impossible check.

KNF and WARNS=3

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

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

Use err(1, NULL) and warn(NULL) for `out of memory' error messages.

return error on errors (PR#6147 by Joseph Myers <jsm28@cam.ac.uk>)

check for malloc failures (PR#6136 by Joseph Myers <jsm28@cam.ac.uk>)

constify, per PR 6041 by Joseph Myers <jsm28@cam.ac.uk>

Replace some homegrown string-functions with standard-<string.h>-macros.
Reported by Joseph Myers <jsm28@cam.ac.uk> in PR 6028.

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

clean up RCS ids

branches: 1.1.1;
Initial revision

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

fix previous, thanks asau@

Pass -Wstrict-overflow.

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

sprinkle static

KNF and WARNS=3

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

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

This patch cleans up the handling of the variable `saved' in
adventure(6). The handling of this variable is somewhat confusing,
since it is used for two different purposes (controlling the time
required before a saved game can be restored, and controlling various
aspects of dwarf behaviour); in fact, it is also declared twice in
hdr.h. Except possibly when saving a game fails, these uses can never
interfere; when used for controlling dwarf behaviour, we always have
saved == -1. This can be better understood with reference to the
original PDP-10 FORTRAN source (URL in patch, since hdr.h references
the comments of the FORTRAN as still relevant to this version) of
which the C version is a direct translation: the wrong value for
`saved' meant that someone was cheating and had bypassed normal
initialisation. Saving was done by halting and telling the user to
save their core image, so the question of carrying on after saving
failed to open the output file did not arise.

This patch separates the uses of `saved' into uses of two separate
variables.

Closes PR 8005 by Joseph Myers <jsm28@cam.ac.uk>

The game adventure(6) handles EOF on standard input rather
ungracefully. The patch, derived from OpenBSD, improves this
handling.

Sent in in PR 6556 by Joseph Myers <jsm28@cam.ac.uk>.

Kill unused parameters, per PR 6023 by Joseph Myers <jsm28@cam.ac.uk>.

Consider leap years, as per PR 5830 by Joseph Myers <jsm28@cam.ac.uk>

KNFify

deprecate register, fixup .Nm usage

PR/3964: Eric Fischer: Remove dependencies to traditional cpp...
While I was there add WARNS?= too.

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 RCS ids

branches: 1.1.1;
Initial revision

clean up import.

specify man pages the new way.

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Fix markup

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

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

Use .Nm instead of "arithmetic" in one place. From jmc@openbsd.

Lots of minor fixes resulting from reading these man pages in detail.

Whitespace and/or punctuation fixes.

More and more .Os cleanups. .Os is defined in the tmac.doc-common file,
so we shouldn't override it with versions in the manpages. Many more to
come.

Fix man to mdoc conversion mistakes, mostly of the form ^\.[A-Z][A-Z]

KNFify, WARNSify, use errx(), clean up .Nm in manpage

clean up import.

Update to use new -mandoc macros.

Add RCS indentifiers.

branches: 1.1.1;
Initial revision

WARNS=5, and make WARNS=5 the default for src/games.

branches: 1.26.2;
NULL does not need a cast

remove unnecessary casts

sprinkle static

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

branches: 1.22.6;
convert __attribute__s to applicable cdefs.h macros

branches: 1.21.16;
Add (unsigned char) cast to ctype functions

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.

The game would show the statistic info and immediately prompt you with a new
question, so you lose time, etc. Added a few lines so that it prompts to
press the RETURN key to continue.

Code contributed by Vic Hari <vkhr@rawsocket.net> in PR 16135

convert to use getprogname()

fix nested extern

Use setgid(), not setregid().

Don't declare 'extern opt*' getopt variables.

This patch (a) makes arithmetic(6) revoke any setgid privileges it got
from dm; (b) marks an unused parameter as such.

Submitted in PR 8014 by Joseph Myers <jsm28@cam.ac.uk>

constify, as per PR#6136 by Joseph Myers <jsm28@cam.ac.uk>

mark non-returning functions (PR#6144 by Joseph Myers <jsm28@cam.ac.uk>)

change includes to fix compiler warning

Make compiler happy.

getopt returns -1 not EOF

KNFify, WARNSify, use errx(), clean up .Nm in manpage

Be pedantic, main() returns int in Standard C.

clean up import.

change <sys/signal.h> to <signal.h>

Add RCS identifiers.

Cleanup for GCC 2.

branches: 1.1.1;
Initial revision

initial import of 386bsd-0.1 sources

document non-literal format strings

When using -lcurses, you also need -lterminfo.
This fixes the build for sun2, and also builds with LDSTATIC=-static,
since archive libraries don't record inter-library dependencies.

use %option noinput nounput

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

branches: 1.26.12;
Make the default WARNS for games 4. The only game that needs to set WARNS
to anything else now is rogue, so clear WARNS from the other makefiles.

branches: 1.25.10;
WARNS=4

Enable WARNS=3.

Rework how dependency generation is performed:

* DPSRCS contains extra dependencies, but is _NOT_ added to CLEANFILES.
This is a change of behaviour. If a Makefile wants the clean semantics
it must specifically append to CLEANFILES.
Resolves PR toolchain/5204.

* To recap: .d (depend) files are generated for all files in SRCS and DPSRCS
that have a suffix of: .c .m .s .S .C .cc .cpp .cxx

* If YHEADER is set, automatically add the .y->.h to DPSRCS & CLEANFILES

* Ensure that ${OBJS} ${POBJS} ${LOBJS} ${SOBJS} *.d depend upon ${DPSRCS}

* Deprecate the (short lived) DEPENDSRCS


Update the various Makefiles to these new semantics; generally either
adding to CLEANFILES (because DPSRCS doesn't do that anymore), or replacing
specific .o dependencies with DPSRCS entries.

Tested with "make -j 8 distribution" and "make distribution".

The patch below improves the security of the game atc(6), by having it
open the score file at the start and then drop all setgid privileges
while keeping a (close-on-exec) file descriptor open to it. In order
to allow this the static data files have to be made world readable.
In addition a potential buffer overrun with corrupted score files is
avoided by more careful use of scanf (note that SCORE_SCANF_FMT is
defined alongside the definition of the relevant structure).

Submitted in PR 8015 by Joseph Myers <jsm28@cam.ac.uk>

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

don't install in /usr/share if NOSHARE is defined

remove unused files (PR#6083 by Joseph Myers <jsm28@cam.ac.uk>)

y.tab.h is now called grammar.h (and there's an implied .y.h rule now too)
Noticed by veego.

.y.c <sys.mk> rule fixes. Don't create a y.tab.h file unless asked for,
and use smarter creation of the header file.

Simply include -lcurses instead of -lcurses -ltermcap

We need a FILESMODE=440 for dm which is now setgid games.

install games that need it setgid.

use CPPFLAGS instead of CFLAGS, fix dependancies for parallel compile

branches: 1.12.2;
WARNSify

- Makefile cleanups

Use ${INSTALL}.

clean up import

MAN6 => MAN.

adding 10 more games collected from ???? and some I wrote.

specify man pages the new way.

All programs that use lex must link with -ll.

Add RCS identifiers.

make sure directory to install stuff exists before installing

got rid of unnecessary -lcompat and associated dependency

branches: 1.1.1;
Initial revision

Use An in AUTHORS section.

Sync descriptive text with actual game definition.

Use standard AUTHORS section header. From YOMURA Masanori in private mail
Sort sections if necessary. Use more/better markup.

Bump date for previous. Remove superfluous .Pp.

Document that pressing return allows you to "fast forward" the game
clock. Patch from Peter Maydell <pmaydell@chiark.greenend.org.uk> in
Debian bug 216980.

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

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

Use only one of -literal and -unfilled with Bd.

Grammar fixes, from jmc@openbsd.

.Nm does not need a dummy argument ("") before punctuation or
for correct formatting of the SYNOPSIS any longer.

Lots of minor fixes resulting from reading these man pages in detail.

Mdoc nits -- unneeded ".

New sentences begin on new lines.
Patch from Richard Elz, slightly improved by yours truly.

Generate <>& symbolically. I'm avoiding .../dist/... directories for now.

No argument for .Os.

Whitespace and/or punctuation fixes.

remove unnecessary command name after .Nm

use lowercase with .Nm to match actual command name

Correct descriptions of "marked" vs. "ignored" -- they were swapped.

Add two .Els that nroff was complaining to miss.

man page rewritten into -mandoc by dholland@eecs.harvard.edu; from PR7861

A collision is adjacency in all, not any of, three dimensions.

clean up import

Add RCS indentifiers.

branches: 1.1.1;
Initial revision

Make the list of place types an enumeration. Use unsigned values for
place numbers.

branches: 1.6.54; 1.6.60;
Move UCB-licensed code from 4-clause to 3-clause licence.

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

Fix uses of namespaces reserved by ISO C or POSIX.1.

branches: 1.4.4;
better define PI, plus improve prototype (PR#5867)

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

sprinkle static

Fix y2038 bug: time_t != int.

branches: 1.9.10;
Pass lint(1).

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.

constify, per PR 6150 by Joseph Myers <jsm28@cam.ac.uk>

WARNSify

Use POSIX signal and tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

Move game logic out of the file with the curses code.

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

Make the list of place types an enumeration. Use unsigned values for
place numbers.

branches: 1.16.6; 1.16.12;
sprinkle static

attribute noreturn -> __dead

Fix y2038 bug: time_t != int.

branches: 1.13.10;
Pass lint(1).

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.

fix redundant decls and nested externs

The patch below improves the security of the game atc(6), by having it
open the score file at the start and then drop all setgid privileges
while keeping a (close-on-exec) file descriptor open to it. In order
to allow this the static data files have to be made world readable.
In addition a potential buffer overrun with corrupted score files is
avoided by more careful use of scanf (note that SCORE_SCANF_FMT is
defined alongside the definition of the relevant structure).

Submitted in PR 8015 by Joseph Myers <jsm28@cam.ac.uk>

constify, per PR 6150 by Joseph Myers <jsm28@cam.ac.uk>

mark non-returning functions (PR#6144 by Joseph Myers <jsm28@cam.ac.uk>)

WARNSify

Use POSIX signal and tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

atc(6): simplify reallocation logic

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

Don't cast the return value of malloc.

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

Remove empty functin check_adir() and references to it.

This patch marks unused parameters in atc(6) as such.

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

When atc(6) parses its game definition files, the check for `width'
being defined more than once incorrectly checks for `height' having
been previously defined instead.

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

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

Move game logic out of the file with the curses code.

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

woops, more of previous.

Make some things unsigned that should be (why would we need e.g. a
negative number of airports?) and remove some related bogus casts.

branches: 1.16.6; 1.16.12;
sprinkle static

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

branches: 1.14.6;
convert __attribute__s to applicable cdefs.h macros

branches: 1.13.10;
bracket else block

Pass lint(1).

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.

Compare getchar() against EOF, not -1.

In atc(6), the function getAChar() has BSD and SYSV variants to deal
with variations in EINTR behaviour, but the optimisation of using the
BSD version where the SYSV version isn't needed is insignificant.
This patch therefore simplifies the code by making there be just one
version, a more paranoid (about EOF when errno is already EINTR)
version of the SYSV code. Since the BSD/SYSV defines are mainly used
to control whether BSD timers are used, this helps where BSD timers
but SYSV EINTR handling are wanted.

Patch supplied in PR 8091 by Joseph Myers <jsm28@cam.ac.uk>

This patch marks unused parameters in atc(6) as such.

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

This patch makes atc(6) check for memory allocation failure.

Patch submitted by Joseph Myers <jsm28@cam.ac.uk> in PR 8056,
with small enhancement to make clearer when loser() is called
with NULL.

constify, per PR 6150 by Joseph Myers <jsm28@cam.ac.uk>

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

prevent "stack" overflow

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

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

Include <time.h> for declarations the Single Unix Specification says
are there but not in <sys/time.h>.

WARNSify

Use POSIX signal and 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

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

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

Make the list of place types an enumeration. Use unsigned values for
place numbers.

use a more suitable type

use more suitable types

Make some things unsigned that should be (why would we need e.g. a
negative number of airports?) and remove some related bogus casts.

branches: 1.24.6; 1.24.12;
sprinkle static

sprintf -> snprintf

convert __attribute__s to applicable cdefs.h macros

branches: 1.21.8;
Input validation, from David A. Holland on #NetBSD-code

prevent "stack" overflow

Pass lint(1).

KNF and WARNS=3 fixes

Cast last argument of execl to (char *).

Reviewed by <hubertf>.

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.

comment or delete text after CPP directives.

This patch marks unused parameters in atc(6) as such.

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

The patch below improves the security of the game atc(6), by having it
open the score file at the start and then drop all setgid privileges
while keeping a (close-on-exec) file descriptor open to it. In order
to allow this the static data files have to be made world readable.
In addition a potential buffer overrun with corrupted score files is
avoided by more careful use of scanf (note that SCORE_SCANF_FMT is
defined alongside the definition of the relevant structure).

Submitted in PR 8015 by Joseph Myers <jsm28@cam.ac.uk>

constify, per PR 6150 by Joseph Myers <jsm28@cam.ac.uk>

Replace special handling of \r by using ICRNL (PR#6078 by Joseph Myers <jsm28@cam.ac.uk>)

An airplane on a heading of 315, given instruction to turn
right 45 degrees should turn to direction 0, not circle.

An airplane on a heading of 270, given instruction to turn
right 90 degrees should turn to direction 0, not circle.

- don't use char as an array index

WARNSify

bfunctions -> memfunctions

Use POSIX signal and tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

use %option noinput nounput

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

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

.y.c <sys.mk> rule fixes. Don't create a y.tab.h file unless asked for,
and use smarter creation of the header file.

XXX: work around lex(1) vs termios lameness

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

Don't cast the return value of malloc.

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

`existent', not `existant'

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

Report the users score _after_ the game has ended and the screen has
been restored to pre-game state. Otherwise, the user gets a message
that their score did (or more likely, did not) improve over their
previous score, yet the user has no indication of what the current
score actually was!

branches: 1.23.14;
ned stat.h

branches: 1.22.2;
Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

sprinkle static

sprintf -> snprintf

convert __attribute__s to applicable cdefs.h macros

branches: 1.18.10;
Don't follow symlinks on systems that have O_NOFOLLOW.
Don't bother looping with lockf() since first iteration would return.
Prevent two buffer overflows.

Added a missing '-' in score list output.

Pass lint(1).

Fixed an output bug where a new-line was added due the usage of puts(3).

KNF and WARNS=3 fixes

Replaced strncpy with strlcpy. Thanks to Peter Postma who
pointed them our in PR #25762. Approved by christos@NetBSD.org.

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

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

comment or delete text after CPP directives.

This patch marks unused parameters in atc(6) as such.

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

The patch below improves the security of the game atc(6), by having it
open the score file at the start and then drop all setgid privileges
while keeping a (close-on-exec) file descriptor open to it. In order
to allow this the static data files have to be made world readable.
In addition a potential buffer overrun with corrupted score files is
avoided by more careful use of scanf (note that SCORE_SCANF_FMT is
defined alongside the definition of the relevant structure).

Submitted in PR 8015 by Joseph Myers <jsm28@cam.ac.uk>

constify, per PR 6150 by Joseph Myers <jsm28@cam.ac.uk>

Log hostname, not OS - PR 5831 by Joseph Myers <jsm28@cam.ac.uk>

use warn instead of perror/fprintf/...

WARNSify

bfunctions -> memfunctions

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

Move game logic out of the file with the curses code.

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

Make the list of place types an enumeration. Use unsigned values for
place numbers.

Don't use ++ to set flags to true. Now passes -Wstrict-overflow.

branches: 1.20.6; 1.20.12;
sprinkle static

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

branches: 1.18.6;
Fix y2038 bug: time_t != int.

branches: 1.17.10;
Actually accept "?" argument as documented in the manpage.
Don't attempt to ignore an untrappable signal.
Fix 6 possible buffer overflows.

Coverity CID 1786: Close file.

Pass lint(1).

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.

fix redundant decls and nested externs

Use setgid(), not setregid().

Use getopt instead of home grown command line parsing.
Patch from Joseph Myers <jsm28@cam.ac.uk> in PR/8062.

The patch below improves the security of the game atc(6), by having it
open the score file at the start and then drop all setgid privileges
while keeping a (close-on-exec) file descriptor open to it. In order
to allow this the static data files have to be made world readable.
In addition a potential buffer overrun with corrupted score files is
avoided by more careful use of scanf (note that SCORE_SCANF_FMT is
defined alongside the definition of the relevant structure).

Submitted in PR 8015 by Joseph Myers <jsm28@cam.ac.uk>

constify, per PR 6150 by Joseph Myers <jsm28@cam.ac.uk>

Replace special handling of \r by using ICRNL (PR#6078 by Joseph Myers <jsm28@cam.ac.uk>)

use warn instead of perror/fprintf/...

WARNSify

Use POSIX signal and tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

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

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Make the list of place types an enumeration. Use unsigned values for
place numbers.

use more suitable types

woops, more of previous.

Make some things unsigned that should be (why would we need e.g. a
negative number of airports?) and remove some related bogus casts.

branches: 1.6.42; 1.6.48;
Increase username length to match modern systems, and create #defines for a
couple of lengths.

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 improves the security of the game atc(6), by having it
open the score file at the start and then drop all setgid privileges
while keeping a (close-on-exec) file descriptor open to it. In order
to allow this the static data files have to be made world readable.
In addition a potential buffer overrun with corrupted score files is
avoided by more careful use of scanf (note that SCORE_SCANF_FMT is
defined alongside the definition of the relevant structure).

Submitted in PR 8015 by Joseph Myers <jsm28@cam.ac.uk>

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

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

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

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

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

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

Move game logic out of the file with the curses code.

Abolish the foul practice of including every header in existence via a
single central "include.h". Use only headers that are actually needed.

Make the list of place types an enumeration. Use unsigned values for
place numbers.

use more suitable types

Make some things unsigned that should be (why would we need e.g. a
negative number of airports?) and remove some related bogus casts.

branches: 1.22.4; 1.22.10;
Bug fix: in a game with 26 planes, the last one to be allocated wouldn't
be allocated if it was the only eligible one.
From Jonathan David Amery via Debian Bug report 214626.

branches: 1.21.2;
sprinkle static

sprintf -> snprintf

convert __attribute__s to applicable cdefs.h macros

branches: 1.18.8;
s/exceded/exceeded/, from Zafer.

Add missing parens.

Simplify (Masao Uebayashi)

Coverity CID 692: Another && that should be ||. Boy...

Pass lint(1).

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.

comment or delete text after CPP directives.

This patch marks unused parameters in atc(6) as such.

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

This patch makes atc(6) check for memory allocation failure.

Patch submitted by Joseph Myers <jsm28@cam.ac.uk> in PR 8056,
with small enhancement to make clearer when loser() is called
with NULL.

constify, per PR 6150 by Joseph Myers <jsm28@cam.ac.uk>

better define PI, plus improve prototype (PR#5867)

WARNSify

bfunctions -> memfunctions

Use POSIX signal and tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

remove unused files (PR#6083 by Joseph Myers <jsm28@cam.ac.uk>)

branches: 1.1.1;
Initial revision

adding 10 more games collected from ???? and some I wrote.

adding 10 more games collected from ???? and some I wrote.

branches: 1.1.1;
Initial revision

initial import of 386bsd-0.1 sources