Various cleanups I made while going through Erik Hovland's patch submissions,
some of which are even from him. :)

- convert old-style parameter declarations (K&R) to modern style.

Whitespace.  (Vim's auto-indent is kind of annoying at times, like "daytime"
and "nighttime"...)

Fix a possible race condition if two processes try to claim the same loop
device at the same time.  We should only CLR_FD if the set status fails,
not if the SET_FD fails.

A few patches from Erik Hovland, turning strncpy() into safe_strncpy() and
removing some unnecessary code.

Add a test from Glenn.

Document mount's new error return conventions.

The logic to make cp -d or -P treat things like regular files should only
trigger for symlinks, not for device nodes.  This should fix "cp -a /dev ."
to work as expected (when run by root, anyway).

While I was there, cleanup headers and make an #ifdef go away...

Slowly pulling more common headers into libbb.h.  Andre pointed out that
older uClibc didn't include sys/socket.h from netinet/in.h, so add an
explicit #include for that too...

Clean up.

Patch from Shaun Jackman to replace CFLAGS_EXTRA with .config.mak

CONFIG_DHCP -> CONFIG_APP_DHCP.

Jean Wolter writes: modprobe checks, whether a module is already loaded. The function used for this currently always returns 0.

- add note about strict ordering

Jean Wolter writes: ash uses bsearch to search in the list of built-in commands, therefore the list of built-in commands has to be ordered.  Revision 15077 introduced the source built-in and destroyed this order.  And now ash doesn't find the shift command anymore.

Patch from Paul Fox to improve mount's error handling behavior, which I beat
on a bit.

Andre (armcc2200@yahoo) patched a bug where successful mounts could sometimes
produce an error, due to a missing rc assignment.

Aurelien Jacobs pointed out inline functions that could be removed.  Doing so
shrank the code a bit.

Remove pointless "inline" wrapper around get_terminal_width_height().

Zubicaray reported a bug in vi that causes it to eat 100% cpu when you close
an xterm it's running in.  The vi signal behavior would catch and restarts lots
of signals, like SIGHUP, that should just kill the thing.  (Leftover behavior
from when it would segfault all the time.)  Filtered out the more obviously
bad ones.  If it segfaults, we should find and fix the problem.

only enable count_bit functions when actually needed to fix warning reported by Bernhard Fischer

Since rangecoder is just a bunch of C functions, move it into the one user
(decompress_unlzma.c).  Also a slight #include cleanup, and I've been meaning
to put #include <unistd.h> into libbb.h since it's so darn common...

Cleanup patch from Shaun Jackman converting %m to perror.

Patch by Robert P. Day to remove obsolete alias code that the 2.4 and 2.6
Linux kernels don't do anymore.

assign default debug level to the MSHDEBUG define

I broke allbareconfig with half-finished cleanups I'd forgotten weren't done.
Got both to a decent stopping point.  (There's a lot more cleanup to do on ls.c
but now is not the time...)

The type changed too.

Use the new names for tty_baud_to_value() and tty_value_to_baud() again.

Revert the last two patches to go back to a state before this file was
controversial.

Patch by Robert P. Day to make structs use c99 initializers.

Patch from Robert P. J. Day cleaning up some unnecessary preprocessor
directives.

Amrein-Marie Christophe pointed out that Mandrake changed its name.

Make some 64 bit warnings go away on x86-64.

Undo all of the ugliness and some of the bloat from 15412.

skip_whitespace() shouldn't claim its return value is const, it doesn't know
that and callers wind up typecasting it back.

Make this slightly more readable, and expose the hypocrasy of a comment that
says "our implementation makes it impossible to use blocksizes smaller than
22 octets" right above a check for blocksize < 8.

danf writes: stty doesn't support baud rates > 460800 bps, the attached patch adds support for all baud rates supported by Linux 2.6.16 on x86.

Make it so Erik has to forward me less email.

- add cross-reference generation target.
  Experimental.

- do away with the fwd-decls of functions. Should not contain any code changes
  AFAICT.

Revert svn 15333.  If you vfork, you must either exec or exit.  (The parent is
blocked until you do so.)

This directory was fallout from the great feature freeze of 2003, which led
up to the 1.00 release.  I just moved what was left of it to
http://busybox.net/~landley/pending because it does _not_ belong in the
tree anymore.

- hide CONFIG_GETOPT_LONG for the 1.2.0 release. Turned on per default.
  I will revisit this and make sure that run-parts and ifupdown are fixed; hdparm needs treatment too..

- add patch to provide a target 'html' to build a cross-reference of the source

- add needed includes and include busybox.h needed for the ATTRIBUTE define.

Remove warnings caused by #include <sys/mount.h> in platform.h.  Apparently
we redefine a lot of stuff from standard header files...

Patch from Rich Felker:
- duplicated format bit 14 forced LIST_EXEC always-on
- -p option was behaving just like -F (SUSv3 says -p only shows / for
  dirs, not other special symbols)
- tests for LIST_EXEC in color functions were nonsense (constant
  tests). i assume they were supposed to be (all_fmt & LIST_EXEC)
  however having coloring of executable files depend on -F seems
  undesirable.

- use shorter boilerplate, more accurate header guard
- don't use multi-line string literals

- drats.

- let the scripts be prerequisites of the files the generate
- make sure that bbconfig isn't too stupid

Fix bug found by Joey Hess and forwarded by Bastian Bunk.  Add Joey Hess's
test case to the testsuite, plus another one that would have been broken by
the first attempt at a fix.

Don't do workarounds for broken shells.  Busybox is about to grow a shell that
we can run our own testsuite under, and I want it to handle the full range of
strange and peverted behavior you can find in real world shell scripts.
In the meantime, I'll just say #!/bin/bash in places that may care.

Jens Michaelsen pointed out that Linksys' firmware download side moved.

We came up with our own names for ATTRIBUTE_*, so why check if they're already
#defined?  Also remove a check for a gcc version older than RH9 (if non-linux
needs that #define they can put it in their section.)

Patch from Yann Morin to put BLKGETSIZE64 in platform.h had rather a lot of
fallout due to the #include <sys/mount.h>.  Removed that #include from various
applets and fixed up those that were unhappy when that #include was made
because they'd block copied stuff out of it.  (Sigh.)

Help text update for libbb, talking about make standalone.

- there is no "CONFIG_STANDALONE" in the repository. Also iff there was a
  CONFIG_STANDALONE, we'd need to toggle this on per default.

- rename CONFIG_FEATURE_TFTP_DEBUG to CONFIG_DEBUG_TFTP

- rename CONFIG_FEATURE_TFTP_DEBUG to CONFIG_DEBUG_TFTP so it's off in defconfig
- expand the cmd_get/cmd_put macros
- Jason Schoon writes: unlink only if non-stdio

- correct typo

- minor size tweak
   text    data     bss     dec     hex filename
    613       0       0     613     265 loginutils/su.o.orig
    605       0       0     605     25d loginutils/su.o

- minor shrinkage

Attempt at fixing bug 836, vaguely based on patch from somebody named
clausmuus, forwarded to me by Yann E. Morin.

Upgrade the standalone shell code to use CONFIG_BUSYBOX_EXEC_PATH.

Attempt at fixing bug 815 by upgrading bb_spawn() so that builtins are at
the start of the path.  (This should be under the same config option as
the standalone shell, but right now that's buried in the shell menu.)

Also add the ability to specify CONFIG_BUSYBOX_EXEC_PATH with /proc/self/exe
as an overrideable default.

Don't break allbareconfig.

Patch from Shaun Jackman to detect "name=value" variable assignments.  Random
cleanup at the end by me to make some unrelated #ifdefs go away...

Make lash smaller.  (Use llist_t for file closing, and different allocation
functions.)

Rewrite of dmesg, smaller and simpler.

Random cleanup of platform.h.

Work around a persistent uClibc bug, since 0.9.29 still hasn't shipped.
Poked to do this by Jason Schoon.

Patch from Denis Vlasenko:
* Rename a var: statbytes -> transferred
* cursize == transferred, always. Nuke cursize.
* Make progressmeter() a nop if !CONFIG_FEATURE_WGET_STATUSBAR
  (reduces #ifdef forest)
* double elapsed -> int elapsed
* Do not sprintf to buf first and then write(STDERR) it,
  just fprintf directly to stderr
* Progress bar printing code made smaller
* Style fixes

Whitespace cleanup from Denis Vlasenko.

Use -ffunction-sections -fdata-sections --gc-sections if the compiler supports
it.  If nothing else, this gives us better granularity in bloatcheck.

Use xstat() instead of if(stat()) die()

Patch from Denis Vlasenko, tweak error messages.

Minor size shrinkage.

Patch from Denis Vlasenko: unlzma was make files with mode 777.  Tweak
everything to do stat() and use xopen3().

Fix from Paul Fox to make compressed help text notice config changes.

- make VERBOSE= and V= work equally; use BUILD_VERBOSE internally.
- move the link flags to cmd_link* so they are printed when requesting verbose output

- fix stupid typo from last checkin

- remove superfluous CONFIG_BEGIN and CONFIG_END

- add fancy mode to start-stop-daemon to support --oknodo and --verbose

- invert logic for nocheck_targets to exclude the empty default target.

- don't check for toolchain-setting for make targets that don't need them
- we already depend on sed, so do away with tr. Avoids pulling in yet another
  dependency.

- revert incorrect select change

- fix two segfaults (reported by Horst Kronstorfer)
- remove dangling file if get fails (spotted and fixed by Jason Schoon)
- shrink it (Bernhard Fischer)
Thanks, all!
   text    data     bss     dec     hex filename
   2684       0       0    2684     a7c networking/tftp.o.orig
   2748       0       0    2748     abc networking/tftp.o.allfixed
   2666       0       0    2666     a6a networking/tftp.o.+shrink

- use bb_xbind
  Thanks Erik Hovland

- use bb_xchdir instead of ignoring eventual errors of chdir
  Thanks to Erik Hovland

- fix bug #887, in bb_get_chomped_line_from_file(), the last char was removed
  unconditionally, even if it was not a newline.
  This was apparently broken by r14254
- whitespace while at it.

add ordering warning on enum Location

- make sure that we see string_insert only if we need it.

remove leading "Enable" from the config option text.  not only is it
redundant, but it made the menuconfig shortcut mechanism mostly
useless, since so many of the entries started with 'E'.

made "test" an ash built-in.

moved the contents of libbb/bb_echo.c back into coreutils/echo.c,
which is a more reasonable place for them than libbb.  this
forces anyone who wants echo and test to be builtin to ash to
also have them available as applets.  their cost is very small,
and the number of people who wouldn't want them as applets is
also very small.

added warning about shell builtins vs. CONFIG_FEATURE_SH_STANDALONE_SHELL,
which conflicts with their use.

thanks to nathanael copa for debugging help.

some string size optimization in test.c may have been lost with
this commit, but this is a good new baseline.

Patch from Erik Hovland, via Tito.

Some day, maybe gcc will be able to optimize out static functions that are
never used so we don't have to #ifdef them.  Wouldn't that be nice?

need prototype for new vfork_daemon()

initial support for no-mmu systems

sometimes daemonizing and vfork()/exit() is ok

bb_xdaemon() isnt available on no-mmu systems