This is an ugly hack, but for now I am spliting ParseModifiers() out
from VarParseLong().  I am passing many the variables, but most of them
will go away once I fix the lengthPtr calculations.

VarParseLong():
Create new else block for if (haveModifier) {}
Move VarExpand() down into both branches of if () {}
Move cleanup section up into both branches.
Eliminate dead code.

VarParseShort(): Change else blocks to fall through.

VarParseLong():
Move setting of dynamic back before the if statement.
Moved lengthPtr down.

Changed code to avoid modifying the input string, its not really
nessisary to null terminate substrings, when you have the length
of the substring you are interested in.

- Add battd to rc.conf(5)

Submitted by: Devon H. O'Dell <dodell@sitetronics.com>

- Add battd rcng script

Submitted by: Our good friend Devon H. O'Dell <dodell@sitetronics.com>

VarParseLong() - Removed space at end of line.

GCC supports two pseudo variables to get the function name, __FUNCTION__
and __func__. The latter is C99, prefer that.

Use -pthread instead of adding -lc_r for linking the thread library.

Rename PACKAGES to REL_PACKAGES. PACKAGES is used by ports already.
Change the default for PACKAGES_LOC to ${PACKAGES}/All if it is defined.

- Better handling of a given(-p) invalid port number.

From: OpenBSD

The old USB ethernet code utilized a netisr to hand packets over
the ether_input and cleanup after txeof. This was broken some
time ago and is a workaround for the different IPL of USB and
the network stack.

Reduce the amount of hacks by removing usb_tx_done, it can be
done mostly in-place with a slight race-condition in the enqueue
code. usb_ether_input now directly calls ether_input, the caller
is responsible for rearming the receiver. That's doesn't open
a race since the receiver doesn't have to handle the interface
queues, for the driver state is the driver responsible.

The IPL protection of the USB ethernet drivers has to be reviewed,
it's mostly not existing.

Add better comments.

Add __pure as attribute. A __pure function can only depend on the
arguments, not on global memory. A __pure2 function in contrast
just has to return the same value for the same arguments. It can
use a hash table or similiar means.

For GCC before 3.0 __pure is mapped to __pure2.

Inspired-by: FreeBSD

- isupper(x) can be true for x in [128..255], but since tolower(x) is
  required to return x if it cannot be translated, we don't need the
  isupper() test. - NetBSD 1.16
- Kill registers
- Constify
- Style(9)

Import of FreeBSD rev. 1.2 by mlaier:
  Don't rely on properly setup linker.hints to figure out that pflog is now
  part of the pf module.
  While here fix a comment that was c'n'ped from rc.d/pf

  PR:             bin/71096 (partly)
  Submitted by:   Ville-Pertti Keinonen
  MFC after:      2 days

Test for my existance

Make pgrep(1) print a newline at EOL, as expected, instead of at the
delimiter specified with the -d switch.

Submitted-by: Robert Nagy <robert_at_bsd_dot_hu>
Same-as-in: OpenBSD: src/usr.bin/pkill.c:1.8

ALTQ support.

Add "exit" command as a synonym for "quit".

Submitted-by: Robert Nagy <robert_at_bsd_dot_hu>

ALTQ support.

ALTQ support.

Forgotten in last commit.

Raise WARNS to 6.

ALTQ support.

ALTQ support.

Fix lvalue cast.

Obtained-from: FreeBSD

Replace list of checks with loop. Fix lvalue casts.

Fix lvalue cast.

Change second argument of dc_crc_le to c_caddr_t to fix warning.

ALTQ support.

Make the second argument to ncp_conn_asssert a const char *.
Use __func__ while being here insteed of __FUNCTION__.

Replace lvalue cast.

Replace lvalue cast with explicit cast from u_short via int to void *.

Avoid casts as lvalues.

Obtained-from: FreeBSD

Don't cast lvalues.

ALTQ support.

Be more careful when doing el_parse() - only do it when el is
properly initialized, that happens when lpc is called from a tty.
Without this change, it's possible to get SIGSEGV simply doing:
        echo "..:" | lpc

Reported by:    Wojciech A. Koszek <dunstan at freebsd czest pl>
PR:             77462 (patch rewritten by myself)

lpc.c, 1.29
Obtained-from: FreeBSD
Submitted-by: Xin Li (delphij@freebsd)

- Use strcasecmp() for upper and lower cases.
- Style(9)

Create two new functions VarParseShort() and VarParseLong() from the
contents of the if-else-if-else blocks.   VarParseShort() handles
variables that are only one letter, while VarParseLong() handles
variables that have multiple characters, and might possibly have
modifiers.

- Remove more dead code, since VarExpand() is never going to be the same
  as v->var.

- Push more variables into local scope of the if else blocks.  This
  is in preperation for making those blocks into functions.

- Killing some dead code, since the conditions can never be true.

- Copy destroy code at end of Var_Parse() up into else-if blocks.
  This temporarily increases the code size.  Much of that goes
  away, because the certain branches become unreachable, and thus
  can be deleted.

- Cleanup Destroy code at end of Var_Parse()

- Clean up return statements at end of Var_Parse().

- Moved haveModifier handing up into else-if block.  This make that
  block huge, but we are going to turn that into a function.

- Factor out code into a new function VarExpand().

Use ifq_is_empty to allow USB drivers to support ALTQ.

Introduce vnodepv_entry_t as type for the vnodeopv_entry functions.
This is slightly better than casting all the functions to void *,
which is a data pointer.

ALTQ support.

Hook ru.koi8-r.win.kbd up to Makefile and INDEX.keymaps.

Import (slightly modified) ru.koi8-r.win.kbd:1.1 from FreeBSD (fjoe):

"Add ru.koi8-r.win.kbd based on ru.koi8-r.shift.kbd which closely mimics
 ru(winkeys) XFree86 layout.

 Differences with ru.koi8-r.shift.kbd are:
 - ctrl+shift = alock
 - left/right win = meta, menu = saver
 - alock'ed shift+[0-9] = )!@#$%^&*("

Submitted-by: Vyacheslav Bocharov <list_at_smz_dot_com_dot_ua>
Reviewed-by: Alexander 'alxl' Lobachov <alxl_at_alxl_dot_info>

Further cleanup:
- Remove `register' keyword.
- Declare local functions as `static'.
- Remove unneeded unions, and their associated #define'd aliases,
  around the struct fs and struct cg that are being used.

Merge with FreeBSD, revision 1.30 (markm):

"Easy warns fixes; use const, mark unused function parameters. Remove
 unused includes."

This allows us to raise WARNS to 6.

Merge with FreeBSD, revision 1.29 (imp):

"o unifdef -D__STDC__
 o use ansi function definitions
 o remove OLDSUN"

Make sure the $FreeBSD$ tags are correct.

Merge with FreeBSD, revision 1.27 (fenner):

"Use SIGUSR1 to propogate SIGURG to the child; security measures prevent
  the parent from sending SIGURG itself to the child.  This fixes the
  problem of occasionally failing to pass the window size."

Raise WARNS to 6:
- Constify strings as needed;
- Use correct number of initializer elements;
- Use ssize_t instead of size_t where needed;
- Correct usage of signal().

Partial merge with FreeBSD: revisions 1.23 through 1.34.

Highlights:

- Support C99 format specifiers %z, %t, %j, %hh.
- Support locale-sensitive "thousands" seperators.
- Some internal cleanup.

Make VGA palette registers available.

Change type of find_compare() so that it doesn't need to be casted
when being passed to fts_open().

Requested-by: joerg

Fix up a bug that crept in during the strcpy->strlcpy conversion;
prefix the device name with _PATH_DEV each time through the loop,
instead of just once before the loop.

Apply less dodgy range checks.

Requested-by: joerg

- Remove unnecessary headers (string.h, sys/uio.h, utmp.h)
- State any unlinkable file
- Style(9)
- Err magic
- Constify

ALTQ support. Remove bogus check of ifq max len.

ALTQ support.

Set ALTQ ready.

ALTQ support.

Rename IFM_homePNA to IFM_HPNA_1 and IFM_1000_TX to IFM_1000_T.

Obtained-from: FreeBSD
Submitted-by: Andreas Hauser

Greatly reduce the size of ISOFS's inode hash table.  CDs and DVDs are small
and slow compared to hard disks, an ultra-efficient inode hash table is not
necessary.

Suggested-by: Joerg Sonnenberger <joerg@britannica.bec.de>

Add support for ICH6 and some nForce AC97 chips.

Obtained-from: FreeBSD
Submitted-by: Andreas Hauser

Increase size of limit column by one character to better fit
on screen on machines with >= 1 GB RAM.

Expand *n't contractions.

Author: ru
Taken-from: FreeBSD

- Changed type of vlen from int to size_t.
- Replaced usage of function scope variable 'rw_str' with vname
  from a local scope.  This makes it clear that the value in rw_str
  is dead.

- Remove adding of '\0' to a Buffer object, since that is already
  done by the Buffer object itself.
- Renamed vname to name, I want to use that variable name in a
  different scope.

Partial merge with recent revisions from FreeBSD:

- pass -1 to setfile() (cp.c:1.49)
- When -R is not specified, fail to copy the contents of dangling
  symlinks instead of making a copy of the link itself, as required
  by SUSv3 (cp.c:1.36)
- Fix the case of a trailing slash following the destination
  mistakenly creating a file instead of a directory (cp.c:1.34)
- Mechanically kill hard sentence breaks in manual page (cp.1:1.29)
- Clarify what -p option does and spell "set-{user,group}-ID" with
  hyphens (cp.1:1.27)

Submitted-by: Sarunas Vancevicius <svan_at_redbrick_dot_dcu_dot_ie>

Clean up:
- Remove casts to void when ignoring return values.
- Use explicit cast when calling fts_open().
- Raise WARNS to 6.

Submitted-by: Alexander 'alxl' Lobachov <alxl_at_alxl_dot_info>

Partial merge with recent revisions[1] from FreeBSD.

Highlights:
- Use a larger field for the size in blocks; the current width
  (4 digits) is only good for 5 MB.
- Introduce a new primary `-depth n', which tests whether
  the depth of the current file relative to the starting
  point of the traversal is n. The usual +/- modifiers
  to the argument apply.
- Support SysV-style -exec args {} + function, required by SUSv3.
- Remove partial support for building on NetBSD.
- Lots of style(9) work.
- Update manual page.

Features not merged:
- The -acl option for locating files with extended ACLs.

Submitted-by: Alexander 'alxl' Lobachov <alxl_at_alxl_dot_info>
[1] Makefile:1.17 extern.h:1.22 find.1:1.71 find.c:1.17 find.h:1.17
    function.c:1.52 getdate.y:1.3 ls.c:1.17 main.c:1.15 misc.c:1.7
    operator.c:1.14 option.c:1.23

Raise WARNS to 6:
- Double-cast values when testing their ranges for validity.

Raise WARNS to 6:
- Use ssize_t instead of size_t for storing size of inode buffer.

Clean up:
- ANSIfy function definitions.
- Constify where appropriate and remove `auto' declarations.
- Normalize whitespace and placement of braces.
- Parentheses around arguments to sizeof.
- Explicitly test against NUL character.
- Use curses functions instead of accessing internal members of the
  WINDOW structure.
- Eliminate a convoluted and unnecessary goto.
- Raise WARNS to 6.

Brought-to-my-attention-by: my CPSC 213 lab assignment

Copy input parameter 'char *str' pointer into two variables.  One which
is a 'const char *' pointer and the other a 'char *'.  This way we can
seperate out the code that actually modifies the input string.

Move statement into both branches of a if-statement so that
common grouping of the statements become more apperent.

Move statement into both branches of a if-statement so that
common grouping of the statements become more apperent.

Use strchr() instead of a switch statement to determine if a char
is part of a set.

Re-order conditions in if-statement a group of so common conditions
are visible.

Remove unneeded conditionals.

Scrap PC98 support.

* Remove Alpha support and redundant flags from the Makefile.
* Rename 2 local variables.
* Cast 2 sizeof's to int for comparison.
* Add WARNS?= 6.

Submitted by: Sepherosa Ziehau <sepherosa@softhome.net>
              (with minor modifications by me)

Fix a little redundancy from my last commit. The border cannot be outside
of the drawing area.

In VESA_800x600, never draw the mouse pointer in the border area
which means the border does no longer have to be redrawn in
remove_pxlmouse_planar().

I have added a per cpu buffer for ktr, so this note can be scraped.

Add per cpu buffer for storing KTR information.
Dynamically allocate entries to each cpu buffer on boot.

Allow numbers to be used after a letter in the modifier section after a
command.

Fix a cache_resolve() vs cache_inval() race which can result in a livelock.
The namecache invalidation code was being a bit overzealous when asked to
invalidate a subhierarchy.  It was retrying until the subhierarchy was
completely invalidated, re-invalidating new entries created after the initial
call to cache_inval().  This can occur if the filesystem is heavily loaded
and a high level directory is being recursively invalidated.

It is unnecessary to retry in this case... the purpose is to invalidate
as-of the call to cache_inval(), so it is acceptable to allow new entries
to be resolved within the subhierarchy undergoing the invalidation.

Certain higher level entities... rename, and vnode reclamation, require
complete invalidation.  The retry has been moved to a higher level for
these entities.  The basic cache_inval() code is now single-pass.

Reported-by: Richard Nyberg <rnyberg@it.su.se>

Mark it as ALTQ ready too.

ALTQ support.

ALTQ support.