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

this already passes WARNS=5

Add RCS Id.

Add a curses gizmo for solving Sunday-paper-type cryptograms based on
substitution ciphers. It gets the cleartext from fortune. I wrote this
some years ago for my own amusement; a couple people have suggested
that I should import it.

Approved only by groo, so I'm going to wait a couple days to hook it
to the build in case anyone demands it be removed again...

Fix typo.

add an Oscar Wilde quote

Use Mt for email addresses.

Fix sendmail section (8 -> 1) in xrefs.

Don't expect fairings.

Fix same typo in the other file in which it occurs

Fix minor typo

rename target paper.${PRINTER} to paper.ps -- PRINTER isnt defined anywhere!

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.

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.

Add some quotes by Henry Ford, found at:

http://www.goodreads.com/author/quotes/203714.Henry_Ford

Fix misspelling of acceptible (it's only acceptable with an 'a', not an 'i').

Fix two quotes from "The Tao of Programming".  From FreeBSD.

PR/47178 from Bug Hunting

src/games/wtf/wtf:
- Correct `usage' (to match new `SYNOPSIS' from man page);
- augment comment on quering pkgsrc's help facility;
- correct interpretation of pkgsrc' help facility query result.

src/games/wtf/wtf.6:
- enhance `SYNOPSIS', document description (`.Dd'), and overal
  program description;
- augment description about quering pkgsrc's help facility;
- improve wording;
- augment `HISTORY' section;
- bump date.

Attribute "The STAR WARS Song" to "Weird Al" Yankovic under the proper title,
"Yoda".  Correct said lyrics.

Correct lyrics to "Weird Al" Yankovic's "One Of Those Days".

Pass -Wstrict-overflow

Pass -Wstrict-overflow; while here, use curses TRUE and FALSE only with
curses.

Pass -Wstrict-overflow... mostly.

Pass -Wstrict-overflow.

Pass -Wstrict-overflow, and while here, don't read from index -1 of
an array.

Factor out some common code; pass -Wstrict-overflow.

Pass -Wstrict-overflow.

Move scratch variables of main to main, instead of having them be
file-static.

Add strategic assertion, now passes -Wstrict-overflow.

Remove ifdefs for V7 (as opposed to older) unix.

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

Crack down on global variables with single-letter names.

fix previous, thanks asau@

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

Pass -Wstrict-overflow.

if 0 is used for the dimensions, compute the maximum size.

Bump date for previous.

- improve wording and overall program description (notably for `-f');
- improve macro usage;
- bump date.

From Bug Hunting.

- correct `usage' message;
- improve wording.

From Bug Hunting.

Add a remarkably percipient fortune cookie.

Fix formatting of Churchill quote on declarations of war.

- improve wording
- sort option list
- add argument names in option list and descriptions and adjust table width
- improve punctuation
- bump date

From Bug Hunting.

src/games/fortune/fortune/fortune.6:
 * reference strfile(8) in `SEE ALSO' section.

src/games/fortune/strfile/strfile.8:
 * remove redundant argument to (successive) `.Nm' macros.

src/games/fortune/datfiles/fortunes2:
 * remove (the least complete variant of a) double fortune.

Patch submitted by Bug Hunting.

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

WARNS=5

WARNS=5

Typo. Reported by I-forget-who-but-they'd-probably-rather-not-be-credited,
during the -6 freeze, and then forgotten until now.

Remove unused fields