CVS log for src/games/hunt/hunt/hunt.c

Up to [NetBSD] / src / games / hunt / hunt

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

sprinkle static and prune some dead code

Remove config options for 4.2BSD and 4.3BSD, and associated code. Highly
obsolete, and I doubt most of it still worked anyway.

Remove the non-curses screen handling, since it relies on knowing
about various internals of SVR4 curses, old BSD curses, and/or other
unclean things. (Yes, the non-curses handling still used bits of
curses. Fie.)

Remove config for the return type of signal handlers, which is also a
long-dead issue.

Whitespace.

ANSIfy function declarations.

Remove two more bogus casts. These *do* create object diffs on amd64,
for no clear reason, but it looks like the diffs are harmless.

Remove bogus casts. Two object file diffs inspected and found to be harmless.

sprintf -> snprintf

Sync with wrstuden-revivesa-base-2.

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

sync with HEAD

clean for WARNS=4

sync with HEAD

convert __attribute__s to applicable cdefs.h macros

change (mostly) int to socklen_t.  GCC 4 doesn't like that int and
socklen_t are different signness.

Add (unsigned char) cast to ctype functions

Mark unused parameters, nonreturning functions and format functions in
the few cases not already marked.

Remove uses of __P.

Make build.

Merge with v20030416.

Import of hunt as of 2003-04-16 from ftp://ftp.cgl.ucsf.edu/pub/hunt.shar.Z,
with files moved to proper subdirs.
Changes since 19930717: BSD licensed, and minimal fixes.

don't print error messages using warn, because we clear the screen immediately
afterwards and they vanish.

Rewrite broken broadcast interface detection code using getifaddrs(3).
"hunt -q" works now.

Avoid strict alias warning.

Remove an unused declaration.

If we're searching for a huntd, probe local_address along with broadcast
addresses.

select() -> poll()

eliminate nested externs

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

Include <sys/param.h> for MAXHOSTNAMELEN.

Pull up revision 1.9 (requested by mycroft):
  Always do the errno indirection hack, so that non-threaded
  libraries get the right errno value when linked with -lpthread.
  This means "always include <errno.h> and never declare errno
  yourself".

Nuke `extern int errno;' in code we compile with -Wstrict-prototypes.  We get
the correct definition from errno.h.

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

Add braces to make the new egcs happy.

- use an array MAXHOSTNAMELEN+1 size to hold hostnames
- ensure hostname from gethostname() is nul-terminated in all cases
- minor KNF
- use MAXHOSTNAMELEN over various other values/defines
- be safe will buffers that hold hostnames

change static to static int

FD_ZERO select mask before FD_SET-ing it

use err/warn instead of perror

Yet Another Monster Commit:
- WARNSify
- getopt returns -1 not EOF
- select() uses an fd_set, not int/long; modify code to use FD_* et al
  instead of direct bitwise operations
- in otto.c::look (renamed to ottolook() to prevent name clash), the
  case WEST section had a 'goto cont_east', instead of 'goto cont_west'.
  (picked up by WARNS=1, because cont_west was an unused label because
  of this typo). probably meant that otto got lost in the maze :-/
- deprecate register, convert bcmp() -> memcmp()

hunt version 1993-07-17

Initial revision