When dumping the contents of a struct sockaddr with an unknown address
family, dump individual bytes (u_chars) instead of u_shorts.  This
avoids endianness issues while also simplifying the code.

In-discussion-with: joerg

WARNS=6.

Submitted-by: Samuel J. Greear <sjg@evilcode.net>

Raise WARNS to 6:
- correctly initialize globals completely
- don't shadow globals
- change some locals to match the type in the ng headers

Submitted-by: Samuel J. Greear <sjg@evilcode.net>

When rendering the routename() or netname() of a struct sockaddr
with an unknown address family, use a more straightforward loop
for dumping the contents of sa_data as hex:

- start at sa_data, end at sa + sa_len (NOT sa_data + sa_len);
- don't make assumptions about the size of u_short.

Inspired-by-patch-by: Sepherosa Ziehau <sepherosa_at_softhome_dot_net>

Pass const ** as second argument to the command functions,
explicitly cast this away in calls to getopt. Change some
local variables to const char * as well, to match this.

Raise WARNS to 6:

- Rename local variables to avoid conflicts with globals;
- Make local functions static;
- Add missing function prototypes;
- Remove stale function prototypes and struct declarations;
- Un-nest nested extern declarations;
- Constify some char *'s where appropriate;
- Resolve comparisons between signed and unsigned;
- Copy some strings into work buffers instead of returning
  a const char * from a function defined to return a char *.

Submitted-by: Sepherosa Ziehau <sepherosa_at_softhome_dot_net>
Some-tweaks-by: cpressey

patch-7.135
 Don't pass endc since we can calucluate it in the function that
 uses it.

patch-7.134
 Cleanup calculation of consumed.

patch-7.133
 ParseModifier()
        - substitute in delim
        - Use shorter variable name patt instead of pattern.
        - Move calc of consumed into subroutine.

patch-7.132
 VarParseLong()
        - update value in vp->ptr instead of using a temp variable.
 ParseRestEnd()
        - Reduce number of explict parameter passed bu passing them
          via struct VarParser.

patch-7.131
 ParseRestEnd()
        - Removed consumed as a input param to ParseRestEnd(), since
          it can be calculated.

patch-7.130
 ParseRestEnd(), ParseRestModifier(), ParseModifier()
        - Reduce number of explict parameter passed bu passing them
          via struct VarParser.

patch-7.129
 VarParseLong()
        - Reduce number of explict parameter passed bu passing them
          via struct VarParser.
        - Change loop termination from endc to '\0'

patch-7.128
 Var_Parse()
        Add temp variable so calculations can be performed before
        we return.
 VarParse()
        Caclucate consumed via (vp->ptr - vp->input)
 VaraParseShort()
        Reduce number of explict parameter passed bu passing them
        via struct VarParser.

Var_Parse(), VarParse()
- Reduce the number of explict parameter passed to VarParse()
  by passing them via struct VarParser.

FreeBSD-Date: 2005/03/15 14:28:39
Author: harti

Get rid of a number of Lst_ForEach calls in favour of LST_FOREACH
and simplify the printing functions.

FreeBSD-Date: 2005/03/15 14:25:24
Author: harti

Yet another version of passing the last line to ParseFinishLine().
It turns out that some ports use the obscure feature of spreading
a dependency block across multiple include files. While this seems
bad style, allow it for now and call said function only at end of
all input to process the really last line of everything.

Get rid of another Lst_ForEach in favour of LST_FOREACH. Get rid
of the now unused struct LstSrc and collapes two functions into one.

FreeBSD-Date: 2005/03/14 17:04:58
Author: harti

Move the creation of a Src structure into its own function.

FreeBSD-Date: 2005/03/14 12:04:20
Author: harti

src/games WARNS 6 cleanups.

Submitted-by: Craig Dooley <xlnxminusx@gmail.com>

Bring in some CAM bug fixes from FreeBSD.

Submitted-by: Xin LI <delphij@frontfree.net>
Obtained-from: FreeBSD

Remove the euc(4) and utf2(4) man pages. They are now in section 5.

Fix incorrect comment.

Add following prototypes:
pthread_mutex_timedlock
pthread_rwlock_timedrdlock
pthread_rwlock_timedwrlock

Add prototypes for pthread_condattr_getclock and pthread_condattr_setclock.

unconstify a modifiable parameter.

Fix comments.

Add clockid to _thr_umtx_wait, so that clockid attribute of condition
variable can be supported.

flesh out a tiny bit so people at least know what the help key is.

Add 'mined' to the bin build and add a minimal manual page for it.

Support XTERM function key escape sequences.

Make ^L redraw the screen, like people expect.

Make ^Z tty-stop the editor, like people expect (but don't try to
deal with window resizes).

Bring in PicoBSD changes from the FreeBSD Attic.

Prepend the Minix LICENSE in all source files.

Add a few other random defines... slightly different from the way FreeBSD
did it.

Bring in the minix editor 'mined', in its original form except for necessary
DragonFly CVS IDs at the top of each file.  Bring the LICENSE file in from
http://www.cs.vu.nl/pub/minix/LICENSE.

Stop building old GDB.

GC

Implement CLOCK_MONOTONIC using getnanouptime(), which in DragonFly is
guarenteed to be monotonically increasing.

Instead of using the non-standard conforming %+ format string,
use nl_langinfo to query the default format string.

Use NXCC to build make_hash and make_keys.

Move SHLIB_MAJOR / SHLIB_MINOR assignment up into Makefile.inc.

Merge r1.2 from NetBSD:
anonymous union between chlen and _UTF7StatePrive has
compilation problem with gcc295.
this union attempt to make mbsinit(3) handle multibyte state correctly,
but it's useless as far as we use utf-7 only iconv interface.
so i eliminate ctype feature.

patch contributed by Joerg Sonnenberg(who porting Citrus to DragonFlyBSD).
and yamt-san gave me advice, thanks a lot.

Back out switch to getopt_long implementation of getopt, it breaks
too much. This will be done (more carefully) later.

Convert a couple of other uses of Lst_ForEach to LST_FOREACH and
simplify code.

FreeBSD-Date: 2005/03/14 08:29:58
Author: harti

Add '+' to argument list of find to prevent getopt from reordering
the arguments.

Correctly return -1 for "-" as argument as required by SUS.
Fix a typo where specifying a single invalid short option could
result in an infinite loop.

SUS says that getopt() should return -1 immediately if the argument is "-".
Correctly handle this case. We formerly depended on the (undocumented)
behaviour of getopt to return this as '-' option.

Fix nested comments. Vim added another */ while I copy/pasted some of the lines
there.

Submitted by: Chris Pressey <cpressey@catseye.mine.nu>

Change the behavior of -fstype so as not to terminate the process
when statfs() failed(meaning that it couldn't type a filesystem),
but report false and continue. This fixes locate.updatedb program
failing on a mounted-but-unavailable mountpoint unless such a
mountpoint is explicitly specified in PRUNEPATHS.

Reported-by: Rumko <rumcic (at) gmail.com>
Ok: Matt

Fix a couple of NFS client side caching snafus.  First, we have to update
np->n_mtime if the file has not being modified locally, otherwise n_mtime
becomes stale and both the dynamic cache timeout calculation breaks and
stat() calls checking for the last modified time break.  This was causing
indeterminant operation with make for obvious reasons.

The Second problem was that we were not checking for an attrstamp of 0
(indicating a timed out entry), resulting in another incorrect timeout
calculation.

Add a function that returns an approximate time_t for realtime for
routines which do not need the exactness of microtime().

- Remove sys/stat.h, sys/signal.h, varargs.h and ctype.h
- Issue the correct amount of space for hostname
- Use strtol to parse integers
- We want to use signal.h, not sys/signal.h
- Use STDIN_FILENO over '0'. More informative.
- Static functions
- Remove unnecessary cast
- Style(9)

Ok: Joerg

Another man page which was moved to lib/libc/gen.

Use libc's getopt_long, I've been building and using ports for a while with
this. Also move th USE_RC_SUBR check into the shared section, just in case
we forget this for an override port.

Use getopt_long's getopt implementation.

Move ctype man page from locale to gen, they are locale-sensitive, but
not locale-specific. Move description of euc, gb18030, gbk, utf2 and utf8
to share/man/man5, they are not device drivers.

Don't build groff as buildtool, it's not needed anymore.

Don't create some directories which will now be empty.

Don't build / install the various historic troff papers to
/usr/share/doc. Don't install the KAME implementation details,
the document is outdated too.

There is no need to set *entry on each entry traversed in the red-black tree
when looking up a record.

Remove DEC Alpha and PC98 support.

Clean up generated files

Switch to CVS 1.12.11

Remove PC98 support.

Fix build. Move common CFLAGS additions up.

Makefiles should never use CFLAGS=.

Update ncurses to version 5.4.
The framework has been written from scratch and I moved everything into
lib/libncurses to centralize the build process for it.

The man-pages is modified by the build in the ncurses paged so I have added the
modified man-pages here.

Merge from vendor branch NCURSES:
Remove Ada95 files and man files.
The man-pages will be added with the new build framework.

Remove Ada95 files and man files.
The man-pages will be added with the new build framework.

Remove Ada95 files and man files.
The man-pages will be added with the new build framework.

Forgot updating SYSACPICA_VERSION.
Reported-by: David Rhodus <sdrhodus [at] gmail.com>

Update ACPI build wrappers to use new ACPICA code.

Submitted-by: Craig Dooley <xlnxminusx [at] gmail.com>
Add a small patch for userland ACPI tools by me, to compile
osunixxf.c . The absence of working AeLocalGetRootPointer() only
affects AcpiOsGetRootPointer() in userland ACPI tools. For kernel
module it's implemented in /sys/${ARCH}/acpica5/OsdEnvironment.c .
The callers of AcpiOsGetRootPointer() properly check anyway to see
if the returned pointer is valid, so making it no-op doesn't make
a problem.

Import acpica-20050211 from Intel.
http://developer.intel.com/technology/iapc/acpi/downloads/acpica-unix-20050211.tar.gz

patch-7.126
    ParseModifier()
cleanup handling of colons.

patch-7.125
    ParseModifier()
Unfold parsing of single letter modifiers from multi letter modifiers.

patch-7.124
Remove <spc><tab> and indent rest of file to 8 space tabs.

 patch-7.123
    ParseModifier(), modifier_C()
Split out modifier_C() as its own function.

patch-7.122
    ParseModifier()
Reindent case 'C', it will become its own function.

patch-7.121
- Reduce number of parameters passed between functions by
  creating VarParser type to contain parser state info.

patch-7.120
    Var_Parse(), VarParse()
- Convert Var_Parse() into a wrapper function.

patch-7.117
    modifier_M()
        Use estrdup() instead of parsing the pattern string twice.

patch-7.116
    Start style changes that harti requested, and other general
    cleanup.
        - Moved variable out of local scope.
        - Renamed result variable to value, common code becomes
          more visible.
        - Add comment that explains why places consumed is adjusted.
        - Rename freePtr to freeResult to clearify what should be
          free'ed

patch-7.115
    VarParseLong()
        - Moved detection of modifier ':' into loop.
        - add comment about why consumed is updated.

patch-7.114
    ParseModifier()
        - Renamed rw_str to value now that we know what it really
          contains.
        - Reindent cleanup section.
    ParseRestModifier() and ParseRestEnd()
        - Move advancement of ptr into parts of if-else.  This
          removes the confusing (rlen - 1) calculation.
    VarParseLong()
        - cleanup calculation of consumed.

patch-7.113
    ParseRest(), ParseRestModifier() and ParseRestEnd()
        - split ParseRest() into two different functions.
          ParseRestModifier() will handle bracketed variables with
          modifiers.  ParseRestEnd() will handle ones that do not
          have modifiers.

VarParseLong()
    - split long line
    - update comments to match code
    - renamed variable lengthPtr to consumed
VarParseShort()
    - renamed variable lengthPtr to consumed
    - update comments to clearify why we are returning an error.
VarParse()
    - update comments to match code
    - renamed variable lengthPtr to consumed

VarParseLong(), ParseRest()
Move nested scope into its own function.

Update the bktr driver to match the current state of FreeBSD-current,
apart from changes that are purely to suit FreeBSD changes. It adds support
for three new cards (including the MT2032 tuner) and has some long -> uint32_t
changes that (according to the commit log) make it work on AMD64.

Obtained-from: FreeBSD
Submitted-by: Steve O'Hara-Smith <steve at sohara.org>

Simplify the print routines by using LST_FOREACH instead of Lst_ForEach
and inlining the small printing utility functions.
Create a function that can be used to produce printable representations
of flag words.

FreeBSD-Date: 2005/03/11 13:24:08
Author: harti

Remove leading underscores from the pathname defines. All identifiers
with leading underscore followed by an uppercase letter are in the
implementation namespace.

FreeBSD-Date: 2005/03/11 13:02:38
Author: harti

Remove the leading underscore from structure tags. All identifiers
with a leading underscore are in the implementation namespace.

FreeBSD-Date: 2005/03/11 12:57:25
Author: harti

Clean up:
- Do not implicitly treat NUL characters, NULL pointers, or 0-valued
  integers as false-valued booleans; use explicit tests instead
- Remove unnecessary header
- err() magic
- Minor Style(9)

Style: Fix indentation.

FreeBSD-Date: 2005/03/11 12:40:55
Author: harti

Call ParseFinishLine() also for the last line in a file. This
patch differs from the previous one in that it calls the function
only when a real file hits EOF. The bodies of .for loops are also
handled as files, but for these we don't want to end a dependency block
on the 'EOF' as in:

foo:
        do-this
.for ...
        do-something
.endfor
        do-more

FreeBSD-Date: 2005/03/11 11:29:39
Author: harti

Constify Var_Dump and simplify it by inlining VarPrintVar.

FreeBSD-Date: 2005/03/10 15:38:01
Author: harti

Reorganize Suff_EndTransform to be called only for nodes for
which it is needed (transforms).

FreeBSD-Date: 2005/03/10 14:54:47
Author: harti

The previous commit should have had the following comment.

Split SuffExpandChildren into three functions: one that skips
variables and expands archive specifications, one that expands
wild cards and a driver that loops over the children list and
expands each child if necessary replacing it with it's expansions.

FreeBSD-Date: 2005/03/09 16:51:43
Author: harti

This commit has the following comment.

Style: fix function style before working on it (mainly intendation).

FreeBSD-Date: 2005/03/10 10:16:03
Author: harti

Implement a new macro LST_NEXT which is like Lst_Succ but doesn't check
for its argument to be non-NULL.

FreeBSD-Date: 2005/03/09 16:53:31
Author: harti

Clean up entries for undefined mbuf tags to catch up with
revision 1.21 of sys/sys/mbuf.h.

Unbreak the build of netstat and systat by hiding stale
references to MT_FTABLE.

Var_Subst() cannot return NULL so there is no reason to check
for it.

FreeBSD-Date: 2005/03/09 07:53:03
Author: harti

Style: fix indentation of SuffExpandChildren before working on it.

FreeBSD-Date: 2005/03/08 16:30:32
Author: harti