properly terminate after using strncpy().

avoid leaving off the trailing nul in a literal string.

cgram: don't beep if the window is resized

cgram: place the 'solved' marker nearer to the text

This way, it cannot be overlooked as easily as before.  It also doesn't
change the position of the keyboard help text anymore.  Only if there is
not enough space, the 'solved' marker is put into the bottom line.

cgram: make the 'solved' stand out more

Suggested by Weitian LI via [1].

[1] https://github.com/NetBSD/src/commit/efec6410b2b5a736

cgram: fix assertion when substituting at the end of the longest line

Reported by Weitian LI via GitHub.

Contrary to the patch suggested in [1], still allow the cursor to be
placed to the very right of the text in a line since that is the usual
behavior of text editors.

Split the function substitute() into two parts:  one that handles the
curses-specific part of checking whether a substitution is possible at
the current cursor position, and one that performs the actual
substitution.  Only the latter is kept in the code section for the
string manipulation functions, the other is moved to the section for
curses code.  Having all the curses code in one place reduces the places
that call beep().  Previously, as well as now, there is a single beep
per invalid key before, but that was not obvious from the previous code.

[1]: https://github.com/DragonFlyBSD/DragonFlyBSD/commit/18d09f18cf4c

cgram: add advanced cursor movement with tab, shift+tab, return

cgram: properly handle input errors

On both NetBSD and Cygwin, a missing /usr/bin/fortune would previously
continue since popen does not return an error (as /bin/sh is found and
can be executed), so the next chance to catch an error is pclose.  At
that point, the shell has already printed an informative error message
about what happened (or what didn't happen), so that cgram does not need
to print an error by itself.

cgram: allow navigation with KEY_PPAGE and KEY_NPAGE

cgram: rewrite completely, fixing bugs and style

Fixed bugs:

Do not consider the puzzle solved if all letters in the visible area are
substituted correctly.  To be properly solved, the whole puzzle must be
solved, even those parts that are currently off-screen.

Never place the cursor at the very right edge of the screen since that
does not work well with some terminals.  The maximum valid x coordinate
is COLS - 1.

Add horizontal scrolling.  Make all coordinate handling symmetric in
regard to the horizontal and vertical axes.  Previously, lines longer
than 80 characters could not be seen on the screen.

Improvements:

Remove the arbitrary limit of 128 characters per line.  Even if
fortune(6) may never generate such long lines, the code is easy enough
to adapt to other sources.

Properly clean up the allocated memory.  Previously, only the string
arrays were freed but not the strings themselves.

Stylistic:

Add RCS ID.

Fix ctype functions in lint's strict bool mode.

Avoid excessive calls to strlen whenever the cursor moves.  Given that
the whole screen is redrawn every time a key is pressed, this is an
unnecessary optimization, but the code smelled nevertheless.

cgram: WARNS=6, use int for all coordinates

Eliminate some frequently occurring subexpressions during substitution.

No functional change, assuming that fortune(6) always spits out less
than 2 gigacharacters of text.

cgram: consistently use char for characters

Having to convert back and forth between char, unsigned char and int is
confusing.  Just stay with char, until the support for wide characters
is added.

No functional change.

cgram: fix undefined behavior when pressing function keys

The key codes for function keys are outside of the "range representable
by an unsigned char".  This resulted in parts of the screen being
cleaned.

cgram: adjust style to survive lint's strict bool mode

The biggest change is the return type of substitute().  Before, 0 meant
success, now false means failure.

No functional change.

cgram: sort includes

cgram: indent

With manual corrections afterwards:

- indent removes empty lines between statement and declaration.  This
  had been a syntax error in C90, since C99 this is common style and
  should be accepted by indent.

- indent didn't format the first line of main for whatever reason, did
  that manually.

- indent removed the empty line above the '/////' separators.  It should
  have left these empty lines as-is.

cgram: fix Home key for cursor navigation

Reported by liweitianux via GitHub.

Fix circle dep

Fix darwin-amd64

Use PREFIX now

Get all the games compiling for iOS

- Don't allocate memory dynamically on the stack (for SSP)
- Don't return errno as exit code
- Fold long lines

set the cursor to invisible

trailing whitespace

Markup/typo fixes.

Add testpat(6) to games.

Testpat displays a test pattern in curses(3).

speed limit 80

speed limit 80

roundsleep(): Too many zeros; tv_nsec should be compared with
500 msec, not 5000. Raised by clang for ILP32 archs.

Avoid common symbols. Use __dead.

Comment out debugging build (it still randomly crashes though) pointed out
by wiz

use strchr, strrchr, random, more lint removal, savefile in /var/games/warp
not /usr/share/games/warp...

no need for -lcompat

- use termios
- enable setgid games
- enable savedir

Fix the old leftover license note

Larry Wall: "I outgrew commercial-free licensing about 30 years ago."

remove error(1) comments

Antivaxer support.

can't have config.h and config.H in case-preserving but case-folding file-systems.

don't segv if the terminal is too big.

pray-tell where are you trying to write?

Time warp forward 34 years so that it compiles (but not work)

Remove the smap.* files

They are generated during the build.

Integrate Warp Kit into the NetBSD build

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

Reset CVS keywords

Add Warp Kit, Version 7.0 by Larry Wall

Warp is a real-time space war game that doesn't get boring very quickly.
Read warp.doc and the manual page for more information.

games/warp originally distributed with 4.3BSD-Reno, is back to the BSD
world via NetBSD. Its remnants were still mentioned in games/Makefile.

Larry Wall, the original author and the copyright holder, generously
donated the game and copyright to The NetBSD Foundation, Inc.

Import the game sources as-is from 4.3BSD-Reno, with the cession
of the copyright and license to BSD-2-clause NetBSD-style.

Signed-off-by: Larry Wall <larry@wall.org>
Signed-off-by: Kamil Rytarowski <kamil@netbsd.org>

Restore the catman joke.

It needs the (8), or it fails to work for people who don't know or had
forgotten catman(8) was ever a thing.

as the de facto fortunes editor and also I think the person who added
that fortune in the first place, and given the discussion in
tech-userlevel, I think I get to do this.

Revert to the original phrasing, but remove the man 8 section reference.

catman is a historical Unix tool.

catman(8) is a past thing

Add one.

rain(6): Make the default delay useful for modern terminals

worms(6): Use a reasonable default delay for non-1980s terminals.

This is still fast, but not unreasonably fast.

- remove duplicate comment
- flush after printing the number
(from kre@)

more clarifications.

mention hex numbers

Remove is_hex_str() (trying to guess if a number was hex or not). It is not
documented and can lead to unexpected behavior.

From gson@
- don't assume -h is always on for large factors
- fix saved large factors printing when -h

- for readability when the exponent < 10 don't prefix it with 0x (from kre)
- fix usage
- merge printing code.

PR/55695: Andreas Gustafsson: factor(6) -h option doesn't always work
Handle -h for factors greater than the primes table.

revert previous and don't parse octal

factor: usage(): mark __dead

- Accept octal input.
- Don't play with the original string so we can print it.

Fix a typo (s/int/in/)

PR/55693: Andreas Gustafsson: factor(6) lists factors in wrong order
Sync with FreeBSD and change their -h (that printed hex) to -x because
we were already using -h.

avoid uninit memory and use memmove vs strncpy for a will be
truncated on purpose string.

apply ${GCC_NO_FORMAT_TRUNCATION} to scores.c

s/ be be / be /

robots: Use arc4random_uniform for better uniform distribution

random(6): Use arc4random_uniform to simplify code

Revert arc4random usage for now

this is a host tool and needs to be portable

future plans: add arc4random to libnbcompat

fortune: arc4random_uniform for better uniform values than random() % ...

tetris: Use arc4random_uniform instead of modulo for better randomness

Avoid messing up the display when too many letters are guessed at once.

The field to put them in was made 26 characters wide... but includes
the string "Guessed: ". So if you get to 17 it wraps to the next line
and clreol()'s it. Instead, when reaching this point step on the
"Guessed:" string instead.

Reported by phil@.

cgram(6): use standard cursor keys, use standard shuffle algorithm

The previous shuffle algorithm asked for 100 random numbers, on average.
The new algorithm asks exactly for 26 random numbers.

Curses predefines numeric constants for keys, and there is no apparent
reason not to use these standard keys for cursor movement.

A revision of "The purpose of computing is insight, not numbers" by
Richard Hamming.
From The Art of Doing Science and Engineering

Recommend using `pkg_admin {fetch-pkg-vulnerabilities,audit}' instead of
{download-vulnerability-list,audit-packages}.

Properly quote printf string

Noticed by Ottavio Caruso on #netbsd@Freenode, thanks!

Heads up on Bell patents
P21 on https://minnie.tuhs.org/Archive/Documentation/AUUGN/AUUGN-V01.1.pdf

Remove extra semicolon.

Simplify, little KNF

Simplify, errors to stderr.

unstr: Check that the input filename fits in the buffer.

strfile: Check that input/output filenames don't exceed the buffer size

buffnum is shared between subs.c and fancy.c

Disks can sense vibes, DTrace confirms it.
https://youtu.be/tDacjrSCeq4

Andrew Weatherall's tattoo and a track on A Pox On The Pioneers album.
https://pbs.twimg.com/media/ERAuddhU4AAFv3w?format=jpg&name=orig
Story:
https://www.dummymag.com/features/andrew-weatherall-interview-it-s-bollocks-it-s-discos-tell-me-tales-of-the/

something I apparently forgot to commit months or years ago

games/hack: Suppress -Werror=stringop-truncation error.

Add GCC_NO_STRINGOP_TRUNCATION to hack.end.c to prevent build failure.

Error was reported when build.sh was run with MKLIBCSANITIZER=yes flag.

Reviewed by: kamil@

games/hack: Revert the strlcpy(1) change since this changes expected behavior from strncpy(3).

Reviewed by: kamil@

games/hack:  Fix -Wstringop-truncation warning.

Replace strncpy(3) with strlcpy(3).

Error was reported when build.sh was run with MKLIBCSANITIZER=yes flag.

Reviewed by: kamil@

games/battlestar: Replace snprintf(3) with strlcpy(3) for better performance.

Reviewed by: kamil@

games/battlestar: Fix the -Werror=restrict warning.

Replace strcpy(1) with the safer snprintf(3) which guarantees NULL
termination of strings.

Error was reported when build.sh was run with MKLIBCSANITIZER=yes flag.

Reviewed by: kamil@

s/saftey/safety/

s/opration/operation/

Introspection. From "Surely You're Joking, Mr. Feynman!"

https://twitter.com/sei6r/status/1182433546331279360

Definition of austerity by Alexei Sayle on Radio 4
https://web.archive.org/web/20190916192807/https://www.bbc.co.uk/sounds/play/m0008bbc

Gcc-8 bug confusion with restrict on sh3

Chris' happiest moment
http://bbc.co.uk/programmes/b09b1zbb

adjust a fallthru comment and add a missing break.

introduce some common variables for use in GCC warning disables:

GCC_NO_FORMAT_TRUNCATION    -Wno-format-truncation (GCC 7/8)
GCC_NO_STRINGOP_TRUNCATION  -Wno-stringop-truncation (GCC 8)
GCC_NO_STRINGOP_OVERFLOW    -Wno-stringop-overflow (GCC 8)
GCC_NO_CAST_FUNCTION_TYPE   -Wno-cast-function-type (GCC 8)

use these to turn off warnings for most GCC-8 complaints.  many
of these are false positives, most of the real bugs are already
commited, or are yet to come.

we plan to introduce versions of (some?) of these that use the
"-Wno-error=" form, which still displays the warnings but does
not make it an error, and all of the above will be re-considered
as either being "fix me" (warning still displayed) or "warning
is wrong."