Reduce warnings as part of WARNS=6 cleaning, stage 2/6 (or so):

- Mark many callback function arguments as __unused.

Submitted-by: Sepherosa Ziehau <sepherosa_at_softhome_dot_net>

Also, do not use M_NOWAIT for an allocation that must succeed.

Reduce warnings as part of WARNS=6 cleaning, stage 1/6:

- Fully specify all elements in static initializers.

Submitted-by: Sepherosa Ziehau <sepherosa_at_softhome_dot_net>

Reduce diffs between FreeBSD

Reduce diff between FreeBSD.

Add the pfi_backend variable, which can be used to specify which
installer backend program to start.  Also, correctly pass the name of
the rendezvous to the backend program when the CGI frontend is used.

Update cdrtools package requirement to version 2.01.

Use malloc(M_ZERO) in preference to separate bzero() after allocation.

My regression test had a bug in it.  Quoting a value in a Makefile
causes make(1) to treat it as one word.

APPLE = "AAA AAA AAA" # one word
ORANGE = AAA AAA AAA # three words

Remove bad line breaks.

parse.c:
    ParseLinkSrc()
Update the description comment of the function to reflect
the previous change.

parse.c:
Fix parsing of archive specifications on the target side
of dependency lines.  It seems that nobody is actually is
using the archive-feature of make.

arch.c
Fix parsing of archives: in System V archive format the
member names is terminated with a slash. Although we are
not System V, ar has been configured to put that slash in.
This format allows filenames with trailing spaces.

Style: remove unneeded parantheses in conditionals.

Author: harti
Date: 2005/03/03 10:07:28
Date: 2005/03/03 10:09:39
Date: 2005/03/03 10:46:23
Date: 2005/03/03 11:34:04

- Remove unnecessary header
- err() magic
- Constify
- Minor Style(9)

off_t is a signed value.  The last commit caused the kernel to fail with
a end < start panic.

Remove 'pristine' file tree.  It is no longer needed now that there is
a /etc.hdd directory on the install image.

Update installer to version 1.1.4.  Highlights include:

- NetBoot Installation Services support.
- Allow user to MFS-back any of their new filesystems.
- Handle cancel/error when setting root password more gracefully.
- Show "Press F1 for Help" only when there really is help to be shown.
- Display progress bar and bold chars correctly on monochrome terms.
- Coalesce multiple rc.conf writes and do them all at once at the end.
- Write keymaps, screenmaps and fonts w/o dirnames and extensions.
- Correctly install/remove prerequisite/dependent packages.
- Sources to copy during install can be specified via a conf file.
- Many and various generally-user-invisible bugfixes.
- NLS support (not currently enabled.)
- Buildable on multiple platforms.

Work-by: geekgod, cpressey, Denis Peplin, Robert Nagy, Brenden Grace,
         & many others who sent in bug reports & feature suggestions

Use the LST_FOREACH macro instead of the Lst_ForEach function. This
saves function calls and reduces void casting.

Date: 2005/03/02 08:30:49 and 2005/03/02 12:00:21
Author: harti
Taken-from: FreeBSD

- Back out part of previous commit. I thought free() was the prefered
  method instead of leaving it up to exit().

PHOLD is a throwback to FreeBSD that we don't actually need here.  This
also fixes an init lockup (due to a missing PRELE before process exit).

Reported-by: Rumko <rumcic@gmail.com>

Rename the flags for sf_buf_alloc(9) to be in line with FreeBSD:

SFBA_PCATCH -> SFB_CATCH
SFBA_QUICK -> SFB_CPUPRIVATE

Discussed-with: Matthew Dillon <dillon at apollo.backplane.com>

- Make sure we free() 'modep' after setmode()
- Add stdlib.h for both free() and exit()

Be consistent in the use of BOOLEAN.

Merge from vendor branch NCURSES:
Add ncurses 5.4 source code.

Add ncurses 5.4 source code.

Correct a mixup between NLSGRP and NLSOWN. Add LOCALE{DIR,GRP,OWN,MODE}.

Use ULLONG_MAX, off_t is unsigned.

Fixed bad result file.

Add another. This one check if the behavior of ths S modifier is the same.

Note: It appears that both ${MAKE_VAR:S/^FOO/BAR} and ${MAKE_VAR:S/FOO$/BAR}
      do not work.

Hack update code since we now have separate src and work directories.

Fix prototypes. This functions actually need a parameter.

Fix warning about pre-ANSI function types.

Add WARNS?=6 to Makefile.

Raise WARNS to 6:
- Un-const-ify a function argument;
- Use a signed integer in a loop bounded by a signed integer.

Improve data validation:
- Simplify the check for a well-formed -a argument.
- Move the check for overflowing the command table from addcmd()
  (which is just a driver) into addmac() (which does the real work,
  and which is called from places besides addcmd().)
- Add a check for overflowing the macro-pair table.
- In each case, use a clearer (while) loop for detecting overflow.
- In each case, error out by using errx().

WARNS=6 cleanup.

Copy collate.h from libc/locale instead of using -I, because the current
ctype.h includes runetype.h and that gets picked from the same directory.

Don't define isatty, it's already defined in unistd.h.

Sync with FreeBSD, random.c:1.17 and random.6:1.7-1.8 (ru):
- Fix SYNOPSIS and sync program's usage output with it;
- Expand stdin/stdout;
- Order SEE ALSO;
- Trim BUGS and HISTORY;
- Sort sections.

In kern_sigtimedwait, do not mask SIGKILL because a stopped process
is always able to be killed by SIGKILL.

Use isdigit instead of isnumber.

Replace digittoint(c) with (c - '0'), we already know that c is a digit,
so the hex numbers can't occur. digittoint is a BSD function and not
part of any standard.

Use the standard isxdigit instead of the non-standard ishexnumber.

Raise WARNS to 6.

Merge with FreeBSD, revision 1.16 (stefanf):

"Properly initialise 'filename' so that random -l doesn't try to open
NULL.

Const-qualify 'filename' to avoid a strdup() call due to
-Wwrite-strings silliness."

Raise WARNS to 6:
- Declare tables of commands as arrays of mutable string buffers
  instead of as arrays of string constants, and just copy new
  data into the table, instead of malloc()ing new space for it.

Use NXCC to compile a runnable executable. The libc we are building might
not be compatible.

Merge from vendor branch GCC:
Add C++ ctype based on NetBSD-style ctype.h.

Add C++ ctype based on NetBSD-style ctype.h.

Constify filename variable.

RIP Alpha libc.

Clean up the XIO API and structure.  XIO no longer tries to 'track' partial
copies into or out of an XIO.  It no longer adjusts xio_offset or xio_bytes
once they have been initialized.  Instead, a relative offset is now passed
to API calls to handle partial copies.  This makes the API a lot less confusing
and makes the XIO structure a lot more flexible, shareable, and more suitable
for use by higher level entities (buffer cache, pipe code, upcoming MSFBUF
work, etc).

patch-7.100

ParseModifier()
- constify 'M' and 'N' modifiers.

patch-7.99

VarParseLong()
- Move default code up into previous if-block since the
  else block always returns.

patch-7.98

VarParseLong()
- Merge two if-statement into a if-else-statement since the
  two tests are compliments of each other.

patch-7.97

VarParseLong()
- cleanup calculation of dynamic

patch-7.96

VarParseLong()
- Remove dead code, and reindent

patch-7.95

VarParseLong()
- Create new scope around default code at end of function.
  Then copy that function above in a conditional. if (v != NULL)

patch-7.94

VarParseLong()
- Create variable consumed, which contains the number of characters
  that we have already consumed.

patch-7.93

VarParseLong()
- renamed variable tstr to ptr in while loop.
- Create a new subscope to contain new tstr pointer which is
  a const pointer, this will prove that the code in the scope
  does not move the pointer.
- Move other variables from function scope into the new scope,
  this scope will prob end up as a new function.

patch-7.92

VarParseLong()
 - moved checking for end of string into while loop.
 - cleanup Buffer object if premature end of string.
 - Var_Prase() can never return NULL so remove the check for it.

patch-7.91
VarParseLong()
- Cleanup some indent problems.

This should have been part of the previous commit messsage to var.c

patch-7.90

VarParseShort():
       - Update lengthPtr value instead of setting it.
       - Copy the string constants and return them, to get rid of those
         pesky warnings about const strings.

Var_Parse():
Start migrating lengthPtr calculations down into sub-routines.

Add compatibility with old style extended options to CVS.
Before, you couldn't check out from a NetBSD/FreeBSD/OpenBSD repo and at the
same time get correct $Tags$.

This fix has been in my tree for several months now. Please note that using
OpenBSD repo will usually result into some warnings as not all old features are
natively supported by our cvs.

- Use variable 'len' instead of making an unnecessary call to strlen()
- Dont use fprintf(), exit() and getprogname(). Just use errx()

Cleanup comment and add option so the working dir can be set.

Don't have binsrch() alter a global variable if it fails.  Instead,
have it take an extra integer, passed by reference, and modify that.

WARNS=6 cleaness, some constification, some cleanups.

Changed regression tests to write to /tmp instead of in the src dir.

Style(9):
- Normalize spacing, mostly around binary operators.
- Wrap some long lines and fix some indentation.
- Use standard usage message formatting.
- Braces around multiline blocks.
- Parentheses around argument to sizeof.

Add better comment for hack_cmp(), and reduced number of paramters passed
to function.

Raise WARNS to 6:
- Remove NOTSET from the STYLE enumeration.  Use a new flag
  variable in main(), style_set, for this purpose.
- The ARG() macro is rather large and unwieldy, so replace it with
  an equivalent function, getarg().
- Rename a function to avoid conflict with a local variable inside
  that function.  Rename another function for consistency.
- Rename locals to avoid conflict with library function names.
- Sprinkle `const'.
- Use ssize_t instead of u_int to record line length.
- Ensure several local variables are initialized.
- Some explicit checks against NULL and 0.
- Add $DragonFly$ to Makefile.

In the parent process, close the slave file descriptor at the earliest
opportunity (just after we fork.)  This is not strictly necessary, but
it is good programming practice; and, since this program is one of the
only users of openpty(3) in the tree, it behooves us to set a good
example here.

- Back out rev 1.6 noticed by Matt Dillon. However, still keep the use
  of errc()

Problem: http://leaf.dragonflybsd.org/mailarchive/commits/2005-03/msg00009.html

Add comment harti made about outofdate comment.

Style(9):
- Braces around multiline blocks.
- Explicitly test against NULL.
- Spell NULL as NULL, not 0.

Get rid off Alpha mentions.

Reduce foot-shooting potential by adding -dP for cvs update and -P for
checkout.

Remove the paragraph about /usr/dfports/distfiles, there's no need for
such a thing.

Style(9):
- Add `static' to function prototypes and definitions;
- Remove `register' keyword;
- Rename local variables to avoid conflict with globals;
- Remove unneeded function argument;
- Sprinkle `const'.

Submitted-by: Larry Lansing <lansil_at_fuzzynerd_dot_com>

Header include protection from userland.

Fix whitespace for function prototypes.

Remove a comment that does not make sense; it was just a remnant of
random thoughts.

Cleanup FAIL message

Fix a bug in the ioctl mapping (written by me), which basically failed
to provide the neccessary infrastructure for linux drm to work.

This changes DRM and sound handling in the linuxulator to make use of
the new code.

API compatibility is retained in wide parts; nevertheless modules using
ioctl maps (nvidia) need to be recompiled.

Discovered-and-fixed-by: Brock Johnson <wildefire@isentry.homelinux.org>
Reviewed-by: corecode, joerg

Sync with FreeBSD cp.

o Bump FreeBSD CVS IDs and manual page date
o Style(9)
o Constify
o Add SIGINFO handler (Author: mdodd, 1.40)
o Don't mmap(2) and munmap(2) zero-length files. (Author: alc, 1.42)
o Note how cp(1) handles directories ending in "/" (Author: trhodes, 1.29.2.1)
o Sync program's usage() with manpage's SYNOPSIS. (Author: ru, 1.45)
o Include <signal.h> instead of depending on namespace pollution in
  <sys/param.h>.  Include <sys/types.h> instead of of <sys/param.h>
  so that further such dependencies don't develop. (Author: bde, 1.47)

Submitted-by: Devon H. O'Dell <dodell@sitetronics.com> and
              Sarunas Vancevicius <svan@redbrick.dcu.ie>
Reviewed-by: joerg
Some-minor-changes: corecode

Rest of last commit. Define SA_LEN in one place.

Simplify patches by defining SA_LEN in one place.

client.c.patch and server.c are intentionally left as empty patches.

Define __DECONST here for the sake of FreeBSD.

Rearm receiver, it was lost in the conversion.

Move the MALLOC_DECLARE into sys/msfbuf.h header file.

- Static
- Remove limits.h

- Remove an unnecessary call to stat(). We can just use lstat before
  checking the file. This saves calling both stat() and lstat().

Journaling layer work.  Generate output for path names, creds, and vattr,
for all major modifying VFS operations.  Start working on a vnode
identification and uio abstraction.

Remove some unnecessary braces and fix a gcc 3.4 warning.

Submitted by: Andreas Hauser <andy@splashground.de>

Use u_char define from sys/types.h.

Suggested-by: Andreas Hauser.

Remove extra tokens after #else and #endif.

Submitted-by: Andreas Hauser

Regen.

Update Atheros entries.

Obtained-from: OpenBSD

Mark symname const

lvalue casts.

Add some consts.

lvalue casts.