Some indentation fixes and the removal of some superfluous parens.

Changed a couple sizeof calls and improved some string handling in
the {send,recv}_exclude_list() calls.

Modified sizeof usage and eliminated trailing whitespace.

Added some temporary code to try to figure out what is going wrong
on one NetBSD box in our build farm.

If verbose > 2, mention it if we're setting the uid or the gid.

correct sizeof usage.

Made hard-link sending compatible with pre-28 protocols again.

Corrected sizeof usage:
sizeof obj
sizeof (type)

Fix for memset bug found by buildfarm.

Change to try to debug a failure on the build farm.

Don't warn about lack of -l support if we're the sender -- we just won't
send the receiver any symlinks.

- Call init_flist().
- Got rid of the symbolic-links-warning code (it was not right, and it is
  now handled in options.c).

- Use as little memory as possible for the file_struct (saves another 3
  bytes per file).
- Made the idev struct allocated by the group-malloc again (J.W. suggested
  the proper way to align it for all systems).
- Added init_flist().

Added comment about the position of file_list's "flags" var.

Don't free link_u.idev.

Use "%H:%M:%S" instead of "%T" in strftime() call since it looks to be
more portable.

Clarify the safe/unsafe symlink options in the usage text.

Clarify the various safe/unsafe symlink options.

Go back to using malloc() for the hard-link data structure (for now
at least).

Free the hlink data again.

Make sure that the idev struct's data is 4-byte aligned.

- Changed the file-list allocation to alloc all the memory we need
  with one call, not with several mallocs and strdups.
- Got rid of string_area() stuff (as suggested in the "@todo").
- Enhanced free_file() to optionally free the whole struct or to
  clear it (since we don't need to do both).
- Don't zero out the other flist data when we're about to free it.

Don't free the hlink data (it is not separately allocated anymore).

Changed args to make_file() and free_file().

Added CLEAR_STRUCT and FREE_STRUCT defines.

Added some missing #if SUPPORT_HARD_LINK and #if SUPPORT_LINK directives.

- Fixed the bug where the --daemon option would squelch all option errors
  for later parameters.
- Complain if someone passes us -l and we don't support symlinks.

Changed the rdev code to have both an "rdev" variable (which always
has the full device value) and an "rdev_high" variable (which zeros
the lower byte in the last rdev value).

One more improvement to the 0-flag-byte avoidance algorithm.

A small optimization to the symlink-send code, and an extra sanity
check to the symlink-receive code.

Fixed the test to ensure that we don't send a flag-byte of 0 to the
receiver when transferring the file list.

One more minor variable-name change.

Switched a buffer name in make_file() to make the code a little more
similar to what is in receive_file_entry().

Fixed the optional sanitization of the files_from arg.

OK, "find ... -print0 | xargs -0 ..." wasn't portable.  This is.

Got rid of indeterminacy in the overlapping-files.

A couple more unsigned char changes to silence compiler warnings
on signed-char systems (like Solaris).

New test to make sure that merging files from multiple directories
continues to work.

Use who_am_i() to qualify some debug messages.

Set the am_generator flag.  Use who_am_i().

New function: who_am_i() returns "sender", "receiver", or "generator".

Added "am_generator".

Don't free lastdir!  It is still needed by the flist basedir pointers.
Also, output the basedir pointer when dumping the flist for debugging.

Relocated the externs.

Silenced some compiler warnings.

Better pid handling.

Cast getpid() to a long for output.

Better pid handling.

Fixed a comment.

Tweaked the externs.

Percentage of file-count output was off by one.

EXCLUDE PATTERNS is not just about syntax.

Mention the latest -x improvement.  Also improved the description of
one of my previous fixes.

Optimized the -x option by removing the skip_filesystem() call and using the
new FLAG_MOUNT_POINT flag.

Added FLAG_MOUNT_POINT.

Tweaked a comment.

Improved the differentiation of the transmit flags from the live flags
by naming the transmit flags with an XMIT_ prefix.  Improved the names
of two other flags, got rid of the LIVE_FLAGS mask, and shortened the
flag storage in the flist struct (to save memory).

Use the new names for the transmit-flag defines.  We also make sure
that the FLAG_TOP_DIR flag doesn't get set in the live data for a
non-directory (when it's used as a kluge to prevent a 0x00 byte being
sent over the wire for the flag data).

Use the new names for the transmit-flag defines.

Fixed reference to -u that should have been -o.

Back out bad refactor (thanks Wayne)

Mention new hard-link optimization.

Added defines (FLAG_HLINK_EOL, HL_CHECK_MASTER, and HL_SKIP) and changed
the struct hlink to have an index into the hlink_list[] array instead of
a head pointer.

We now link the hard-linked flist entries into a circular list, mark
the last item with FLAG_HLINK_EOL, and save off an index for our
master entry in the hlink_list[] array.  Added the hard_link_check()
function.

Use the new hard_link_check() function to figure out which one of the
hard-linked files to send (when -H was specified, of course).  Moved
the call to do_hard_links() and the final directory-permissions scan
into generate_files() from recv_files().

Moved the call to do_hard_links() and the final directory-permissions
scan from recv_files() into generate_files().

Changed F_HEAD to F_HLINDEX.

Tweaked a comment.

Tightened up code in skip_filesystem()

Replaced an snprintf() call with pathjoin().

The static last_in value in match_gid() needed to default to -2 so
that we don't accidentally return a GID of 0 when we don't have the
permissions to set it.

Another improvement in the "use chroot" section.

Refer to I/O (not IO), ID (not id), and "an rsync" (not "a rsync").

Improved the "use chroot" discussion on ID mapping.

More changes to make the --owner and --group items clearer.

Use "an" in front of rsync, not "a".

Tweaked the --help text to refer to I/O (not IO) and use "an" in
front of rsync (not "a").

Improved the "use chroot" section to mention how to get user/group
preservation by name (getting rid of the erroneous bit about the
--numeric-ids option being implied).

Improved comments on --owner and --groups, and removed misinformation
on a chroot daemon (it does not imply --numeric-ids).

Properly punctuate "etc." and changed all occurrences of "IO" to "I/O".

Fixed two more misspelled words.

Some spelling corrections and other simple fixes.

When setting an flist entry's flags, mask it with LIVE_FLAGS.

A couple comment tweaks.

Made the getgroups() code a little more portable.  This will hopefully
make the chgrp test work on the NetBSD and OpenBSD systems where it is
failing.

Improved to add the default gid if it is missing from the list.

Removed an obsolete comment.

Tweaked an argv-type variable so that it's a little clearer what
the code is doing.  Also added a comment.

Use the new f_name_to() calling syntax.

- Ensure in make_file() that we can't make a name that overflows
  our buffers (note that receive_file_entry() already does this).
- Changed f_name_to() to not do any length checking since it is
  now redundant.

Added more missing news.

New news.

Size-check the --temp-dir string so we don't have to worry about it
overflowing elsewhere in the code.

Improved a couple comments.

Use new push_dir() syntax.

- Use the new push_dir() and pop_dir() calling syntax.
- The "olddir" handling changed to accommodate the above, and to
  avoid using malloc()/free().

- Changed push_dir() to not take a "save" arg and to return 1 or 0
  for success or failure instead of a string pointer.  The function
  also ensures that we don't overflow the curr_dir[] buffer.
- Changed pop_dir() to not free() anything and to return 1 or 0
  for success or failure.

Use new stringjoin() function.

Find last '@' in the user@site string, just in case the username has
a '@' in it.

Use new stringjoin() and pathjoin() functions.