My adaptation of Matt's cleanup of the delete_item() code.
The new code splits out part of the directory deletion into
its own function, delete_dir_contents(), and it outputs more
warnings when something could not be deleted.

Docment that --max-delete=0 now works.

Die if we sent --max-delete=0 to a receiving side that is too old.

Allow --max-delete=0 (zero used to be ignored).

Increased the PROTOCOL_VERSION to 30.

Mention another internal change.

Test --del too.

Fixed the hard-linking of symlinks test (we need to use a symlink
to a non-existent file for the test to work right).

Changed how we deal with the alternate-dest directory because
some systems change the time on a directory that gets moved.

Changed two more references to "st" to "real_st" in the dir-handling
code.

- Fixed a problem where the real stat struct for a newly-created
  root-of-transfer directory could get overwritten by the check
  for basis dirs.
- Fixed a potential problem where the stat struct for a new dir
  could get passed uninitialized to delete_in_dir().

If the OS can't hard-link a symlink, tweak the expected output
when using --link-dest.

Mention latest bug fixes.

Changed to reflect new (more correct) output of what updating is
happening.

Improved to test --link-dest (when hard-linking of devices works).

Allow all itemize types to change all-dots (no changes) into spaces.

- Improved try_dests_non() so that it works with all non-regular
  files (including dirs).
- Changed the directory-creating code to call try_dests_non() when a
  --*-dest option is used.
- Revamped the device/special-files code to have the same logic as
  the symlink code, making it both simpler and more correct.
- All the above makes us output changes for all items consistently
  when a --*-dest option is used.
- Fixed an output bug in try_dests_reg() and try_dests_non() when
  -ii is enabled (we use to fail to output unchanged files).

Added flag ITEM_MATCHED.

Changed strcat() calls to strlcat().

Disable POPT_WCHAR_HACK for now.

Got rid of type-casting into isFOO() and toFOO() functions by
using static inline functions that take a signed char pointer.

Improved the vmefail() code.

Fixed a problem with the xstrdup() change.

Get rid of the last strcpy() call by using an static inline function.

Use a new isSpace() inline function to call isspace() safely
when using a signed character pointer.

Need to include lib/snprintf.o when building wildtest.

Changed _ABS to be defined as MY_ABS.

Got rid of code to twiddle _ABS.

Moved definition of DBL_EPSILON.

Try a full prototype for alloca() for those systems that need it.

- Define snprintf() when needed.
- Improved the compatibility of the standard includes.

The Mac version of GCC can now handle __attribute__ .

Rolling over the NEWS file for the next version.

Change the version for new CVS work.

- Upgraded popt to version 1.10.2.
- Modified all sprintf() and strcpy() calls to use snprintf()
  and strlcpy().

Preparing for release of 2.6.9

Modified to recognize FreeBSD's setfacl command.

Updated to the version dated 2006-07-02.

Need to include netinet/tcp.h to get TCP_NODELAY defined.

Mention the addition of the fake-super.diff patch.

Don't put backslashes before the dashes in " -- ".

Mention the fix for bug #4079.

Mention WIFEXITED() change.

Mention Mac OS X xattr work.

Preparing for release of 2.6.9pre3

Run tweak_manpage_dashes on the man pages we generate.

Undo temporary changes used for build-farm testing of ACL patch.

Added a define for WIFEXITED() that is used if one is missing.

Document the --no-motd option.

Document the --no-motd option, with a caveat on how it can also
hide the module listing.

Obey the new --no-motd option.

Added the --no-motd option to silence just the text output at the
start of a daemon transfer.

Post-process manpages with the tweak_manpage_dashes perl script.

Run patch with -t so that it doesn't ask any questions.

A perl script that does appropriate backslash escaping of '-' chars
in manpages.

- Fixed an error-handling path in try_dests_reg().
- Added a comment.

Use "test" instead of "[ ... ]" for better portability.

Added missing cleanup of empty ACL files.

Change the unpatch code for NetBSD.

Temporary changes to have the build-farm configure and build
the ACL version of rsync (patched in via patches/acls.diff).

Mention the fix that lets a push now override the partial-dir exclude.

The server should not be adding an assumed filter rule based on the
partial-dir setting (since the client is sending us the rules).

Added a risk filter for the partial-dir since we use --delete-after.

Stop disabling the RSYNC_PARTIAL_DIR environment variable now that
there is a newer rsync on samba.org.

Make sure the new link_dest_used value gets zeroed on init.

- Fixed a compile problem ("len" not defined) in inet_ntop4().
- Replaced sprintf() with snprintf() in inet_ntop6().

- Die if we can't edit the release date into NEWS or OLDNEWS.
- Don't remove any *pre*diff* files until the final release.

Improved a comment.

Improved a couple existing items.

Use the new case_N.h file to make sure that our cleanup_step case
statements never get out of sequence when the code changes.

A helper file for cleanup.c.

Now that samba.org has a modern rsync, we can go back to using
hard-links on the moved files and use rsync to effect the hard-
links on the remote system (which will make a future transfer
more efficient in the absense of --detect-renamed).

Preparing for release of 2.6.9pre2

Use the MAX_RENAMES_DIGITS define in the relevant snprintf().

Improved the description of the code-checker work..

Updated the news about the exit_cleanup() improvement.

Improved the _exit_cleanup() function to handle potential recursions
back to the function via the calls that it is making.  The new code
treats each recursion back into the function as an opportunity to
take up where we left off (skipping whatever step just failed).

Got rid of EXIT_OR_RETURN() macro again -- switching to a better
recursion-prevention heuristic in _exit_cleanup().

The cleanup code that handles moving the temp file to the partial
file needed to check that cleanup_fname was not NULL to prevent
any possibility of a recursion starting into that code again.

Mention the latest fixes and internal changes.

Got rid of the recent paranoid length check because it didn't help
to silence a code-checker warning.

Restore build order back to normal.

Changed sprintf() calls to snprintf().

Turned an sprintf() call into snprintf().

- Changed some sprintf() calls to snprintf() calls.
- Changed two last strcpy() calls to strlcpy() calls.

Changed sprintf() calls to snprintf() calls.

Changed strcpy() calls into memcpy() calls.

Changed strcpy() calls into strlcpy() calls, just to be extra safe.

Added a paranoid length check.

- Changed strcpy() calls into strlcpy() calls, just to be extra safe.
- Added a default: exit_cleanup() case to a switch.
- Made the check_name() function take a buffer size arg.

- Use precompiler conditionals in to_wire_mode() and from_write_mode()
  to get rid of code that is not needed on the current system.
- Silence a code-checker warning in send_file_entry().
- Silence a code-checker warning in make_file().
- Fixed a bug in f_name_cmp() that could compare an empty filename
  incorrectly (which might only happen if we transformed a dot-dir
  into an empty name).  (Thanks, code checker!)

More temporary name juggling to get all code analyzed.

Changed one more exit_cleanup() call into EXIT_OR_RETURN().

Temporarily reorder the OBJS line so that the IBM code checker
can analyze some of the other programs (since it runs out of
time analyzing them all).

Use new FNONE value when setting/comparing enum logcode vars.

- Use the new FNONE enum instead of a literal 0 value.

If in_exit_cleanup is set, some of our functions return instead of
calling exit_cleanup() (which is a safer way to avoid an infinite
loop when the exit code uses the log functions than allowing the
_exit_cleanup() function to return back into our code).

- When exiting, set the flag "in_exit_cleanup".
- Mark _exit_cleanup() with NORETURN.
- Don't ever return from _exit_cleanup().

Mark out_of_memory() and overflow_exit() with NORETURN.

- Added FNONE to the "enum logcode" values.
- Added a NORETURN define to mark a function that never returns.