Reorg docs, part 1:
Move all the reference manuals to subdirs of /usr/share/doc/reference.
We have subdirs ref1-ref9, corresponding to man page sections 1-9.

Everything that's the reference manual for a program (sections 1, 6,
8), C interface (sections 2, 3), driver or file system (section 4),
format or configuration (section 5), or kernel internal interface
(section 9) belongs in here.

Section 7 is a little less clear: some things that might go in section
7 if they were a man page aren't really reference manuals. So I'm only
putting things in reference section 7 that are (to me) clearly
reference material, rather than e.g. tutorials, guides, FAQs, etc.
This obviously leaves some room for debate, especially without first
editing the docs with this distinction in mind, but if people hate
what I've done things can always be moved again.

Note also that while roff macro man pages traditionally go in section
7, I have put all the roff documentation (macros, tools, etc.) in one
place in reference/ref1/roff. This will make it easier to find and
also easier to edit it into some kind of coherent form.

Rework /usr/share/doc.

Update the <bsd.doc.mk> infrastructure, and update the docs to match
the new infrastructure.

- Build and install text, ps, pdf, and/or html, not roff sources.

- Don't wire the chapter numbers into the build system, or use them in
the installed pathnames. This didn't matter much when the docs were a
museum, but now that we're theoretically going to start maintaining
them again, we're going to add and remove documents periodically and
having the chapter numbers baked in creates a lot of thrashing for no
purpose.

- Specify the document name explicitly, rather than implicitly in a
path. Use this name (instead of other random strings) as the name
of the installed files.

- Specify the document section, which is the subdirectory of
/usr/share/doc to install into.

- Allow multiple subdocuments. (That is, multiple documents in one
output directory.)

- Enumerate the .png files groff emits along with html so they can be
installed.

- Remove assorted hand-rolled rules for running roff and roff widgetry
and add enough variable settings to make these unnecessary. This
includes support for
   - explicit use of soelim
   - refer
   - tbl
   - pic
   - eqn

- Forcibly apply at least minimal amounts of sanity to certain
autogenerated roff files.

- Don't exclude USD.doc, SMM.doc, and PSD.doc directories from the
build, as they now actually do stuff.

Note: currently we can't generate pdf. This turns out to be a
nontrivial problem with no immediate solution forthcoming. So for now,
as a workaround, install compressed .ps as the printable form.

remove remaining makefile support for GCC < 45 that i found.

Add a little color. In order to minimize logic differences, keep 0 as the
empty board value, and since 7 is white, turn white into black (0) when
painting.

Remove unused.

Use bool (not int) for a boolean return type.

Use the curses beep function instead of writing a beep to stdout.

curses has printw, use it.

don't use the preprocessor to rename curses functions, just call them

fix off-by-one

Instead of #define STDIN 0, use STDIN_FILENO.

Remove SHORTLEN and LONGLEN defines; use sizeof() properly instead.
Fix two semi-compensating size bugs in wire transmission affecting
64-bit machines.

Naming reform for socket pieces, akin to what I did in huntd earlier.
(but not as comprehensive)

I wish the socket API didn't require casts. Easy to mess them up.

fix obvious mistake (seems to have been there since 4.4)

Tidy up list of broadcast addresses.

Clean up the handling of the list of hunt daemons.

use fewer globals

more cleanup

unexpose some globals that server.c previously required

name reform, fewer file statics

goto considered harmful

split off a bunch of the nastier network code into its own file

When indexing an array with a char, cast it to unsigned char, not int,
to not only silence the compiler warning but also get the correct
behavior.

Remove all the conditional compilation for INTERNET. Now you can run
this on either a local or internet socket (including via inetd on
either) and it will, or is supposed to, DTRT.

Does not really support ipv6 yet, but in a number of places will no
longer vomit or exhibit UB if it encounters an ipv6 address.

Clean up the names of the various pieces of socket apparatus.
Systematic naming is easier to follow...

fix non-INTERNET build

Er oops, fix previous.

Reduce ifdefs by making a common function to talk to either syslogd or
stderr.

group external functions by where they come from

be consistent about whether functions are static

handle compiled-in pathnames the customary way

make some globals static

Additional minor cleanup and remove a straggler data declaration
from hunt_common.h.

Group function declarations by which source file they come from, and
delete several that don't actually exist.

be consistent about whether functions are static

sprinkle static

Flush unwanted file.

Move definitions to the file they're used in.

move a comment that should have gone with its symbol

Remove tautological #if DEBUG.

Split hunt and huntd's includes, instead of incestuously sharing a pile
of external data and function declarations between the two programs.

Common constants and defines now go in hunt_common.h. Stuff that belongs
only to hunt is in hunt/hunt_private.h. Stuff that belongs only to huntd
is in huntd/hunt.h.

Copy some declarations that are used in both programs under the same
names (but are not actually the same objects) from huntd/hunt.h to
hunt/hunt_private.h. Move others that are only used in hunt. Remove
some entirely unused material, and tidy up standard includes.

Remove unsafe and unnecessary pointer casts.

catch up to huntd changes.
XXX: this needs to stop promiscuously sharing .h files.

tsort contents of file

const in unused code

remove stubbed-out function

Rip out the code that broadcasts talk requests to the local
hunt-players mailing list.

In this day and age it doesn't even begin to work: even if you were to
have such a mailing list locally, sendmail wouldn't expand it for you,
the addresses you got back wouldn't be talk-requestable addresses,
talk requests don't cross NAT or firewalls safely, talk doesn't
support ipv6, and nobody runs talkd anyway.

It doesn't make sense to try to replace it with some other kind of
notify scheme either; there might still be a multiuser machine
somewhere with enough simultaneous users that broadcasting something
akin to talk requests locally might make sense, and where a nontrivial
number of the users actually play hunt, but probably not, and if so
those users are likely to be better off using wall(1) anyway. Nowadays
games will be set up by looking for people in chat or by emailing or
texting friends.

Theoretically someone could set up an internet hunt metaserver for
finding hunt games, but that would be something entirely different
anyway and I doubt there's demand.

Remove empty header file.

Use setproctitle() instead of handrolled version.

Make the code for issuing talk requests to find players actually build.

don't declare own boolean type

use random(), not a handrolled 16-bit generator

Use getaddrinfo() instead of gethostbyname().

typo in error message

const

simplify silly logic

temp is a pointer, and tempmax, not sizeof(temp), is the length of
the buffer that it points to.  Adjust a strlcat() call to suit.

If you are going to use memcpy instead of struct assignment, then at
least use sizeof(the correct variable).  The use of memcmp to test
two pointers for equality, one line earlier, is also weird but may be
correct.

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;
                      ^

fix signedness issue

gratuitous (but not harmful) changes to pass gcc48's -Wconversion

Use ssize_t for read() results. Don't use int for lseek() results.
Call srandom() with time(), not getpid().

Use ssize_t for read() results. Don't use int for lseek() results.

Use uid_t to hold getuid() results, and don't check getuid for failure
as it isn't allowed to fail.

use size_t instead of int for a length

Use random(), not rand().

avoid unnecessary floating point ops

avoid signed left-shift

Use CHAR_BIT, not 8.

don't declare ospeed; it's in <termcap.h>

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

use a more suitable type

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.

Don't cast the return value of malloc.

Minor tidyup.

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

Revert part of previous -r1.21 and fix the errno-handling bug instead.

move extern decls to .h files

Merge riastradh-drm2 to HEAD.

fix typo, from Henning Petersen in PR misc/48613

remove duplicate

another minor fix

Reinstate typo since kre claims it's by purpose.
Discussion in PR 48442.

Error and typo fixes from OpenBSD and FreeBSD via Henning Petersen
in PR 48443.

While here, remove a duplication, fix another typo, and
don't claim that Mr. Sheperd was the first man in space.

Error and typo fixes from OpenBSD and FreeBSD via Henning Petersen
in PR 48442.

WARNS=5

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.

fix unused variable warnings

Untabify the header definition to avoid misaligned comment on
str_delim in PostScript output.

Use .Fa instead of .Ar to refer to header fields.

Now that we use "nbtool_config.h" we can use __dead again and drop
local attempts to define its equivalent.

Also drop the comment that warns this file is a tool; use of
"nbtool_config.h" makes it evident.

Don't define TRUE/FALSE, which may conflict with host headers.
TRUE is unused anyway.  Replace all instances of FALSE with 0.

#include "nbtool_config.h" to pull in getprogname() declaration.

GC descr and helpm.

hook cgram to the build

WARNS=5

typo in comment