In clean_flist(), if a duplicate is found for a dir and a non-dir,
always dump the non-dir (because the dir might have contents in
the list).

The dir vs non-dir test now assumes the dir wins.

Mention the new sorting method for protocol 29.

Added a test for the merging of a dir and a non-dir with the same name.

Added some missing commas in some option summary lines.

- Made receive_file_entry() return the file_struct pointer instead
  of storing it into the files[] array.
- Made flist_find() return "no match" if the found item differs in
  its directory-ness from the search item.
- Changed f_name_cmp() to sort sub-directories after non-directories
  for each directory's contents.  This makes things like the upcoming
  --fuzzy patch easier to get right.
- One complicating factor is that clean_flist() needed some extra
  code to ensure that a directory doesn't duplicate a non-directory
  of the same name.
- Make sure that the "strip_root" code in clean_flist() (for relative
  paths) strips off all leading slashes.

Improved the call to f_name_cmp().

Changed the prefix on the fnc_state enums.

Improved the summary for the -F option.

Improved the help text for -F.

Improved the description of the --compress (-z) option.

Improved the documentation of the --compress (-z) option.

Forgot to make sure that the dir-times on the chkdir got synced
in the newest test case.

When --dry-run is set, note when a directory is missing and avoid
trying to stat() any items inside that dir's hierarchy.  This fixes
a bug where a symlink to a dir getting replaced by a dir with
identical contents to the dir at the other end of the symlink would
not report the updated files in the new hierarchy. (See bug #1673)

Some more improvements to exercise rule-restricted merge files
(i.e. exclude patterns only) and side-restricted filter rules.

Improved checkit() and rsync_ls_lR() to work with filenames that
might have spaces.

Mention the protocol change for --delete-excluded.

Document the new filter rule flags for sender-/receiver-specific
rules.

- Added modifiers for the include/exclude rules that makes them
  apply to the indicated (sender/receiver) side.
- Added the hide/show and protect/risk filter rules as an alternate
  way to specify sender-/receiver-specific include/exclude rules.
- send_rules() now allows f_out to be -1 to indicate that the list
  should be scanned but not sent.
- send_rules() now filters the list to remove any items that don't
  apply to the current side (after sending the item to the other
  side when f_out != -1).
- {send,recv}_filter_list() now transfer the list, even when the
  receiver is the server and --delete-excluded was specified (the
  exchanged list is appropriately filtered, of course).
- recv_filter_list() uses send_rules() to trim non-applicable rules
  when we're a local-server (because we got our filter list without
  send/recv calls when fork() duplicated it).

- Define MATCHFLG_SENDER_SIDE and MATCHFLG_RECEIVER_SIDE.
- Updated MATCHFLGS_FROM_CONTAINER.

We don't need to avoid the local filter list in send_file_name()
when --delete-excluded is set because our list has been trimmed to
only include rules that apply in the current mode.

- Since send_file_list() is no longer called with f == -1, got rid
  of all the conditional code to support that.
- Improved the comment before send_directory() to indicate that it
  gets called with f == -1 from delete_in_dir().

Don't use underscores in the example filter rules.

Made "i" a size_t in receive_sums().

A minor tweak for the dry_run logic.

Improved the "refuse options" section, including an update
to the section that talked about --del being a popt alias
(which is no longer true).

- Several improvements in describing how options imply other options.
- Improved --partial-dir.

Changed rule_match() to rule_strcmp().  Likewise for the
define RULE_MATCH() being changed into RULE_STRCMP().

- Added some new refused_* variables that note if certain options
  have been refused on the server daemon.  This allows us to reject
  implied options (e.g. if --partial is refused and -P specified).
- Changed the handling of the --delete refusals from the old idiom
  of upgrading "delete" to "delete*" into the new idiom of checking
  if refused_delete is set when we determine --delete was implied.
- Changed the --del option from a popt alias into a normal option.
- Mark all the daemon options as refused when a daemon is parsing
  the over-the-socket options.
- Created a new function, create_refuse_error(), which is now called
  from all the spots that check for refused options.
- Don't call clean_fname() on an empty string -- either reject it
  or handle it without erroneously expanding the string.
- If --delay-updates was specified without a --partial-dir option,
  don't send the default "--partial-dir=.~tmp~" option if the server
  is the receiver -- just let it default.

Have partial_dir_fname() also check if the dir is excluded
by the server, not just the partial file.

If a --delay-update can't use the partial-dir (e.g. if a server
daemon excludes the dir or file), update the file without delay.

Improved the --modify-window description.

Improved several items and added a new section describing what is
different for protocol version 29.

- Decided that flist_find() should be public, not static.
- Modified f_name_cmp() so that, beginning with protocol 29,
  it will guarantee that a directory name will sort one slot
  before its contents (prior versions could sort other items
  in between in rare instances).

Also test the new long-named filter rules.

Mention the new long-name filter rules.

Allow a ',' to prefix the MODIFIERS for a single-letter filter rule.

- Added long-name filter rules.
- Only parse the "!" token when MATCHFLG_NO_PREFIXES is set if
  MATCHFLG_CVS_IGNORE is also set.

Fixed a typo.

- Added more calls to safe_fname().
- Improved safe_fname() so that it changes all non-printable chars
  into '?'s, and accomodates more simultaneous (and longer) names.

Added missing calls to safe_fname() and full_fname().

Made NS define make use of safe_fname().

Added missing call to safe_fname().

Adding more calls to safe_fname().

Needed to call safe_fname() when listing the remote names.

One more tweak.

A few tweaks to the FILTER RULES section, including a mention of
using the +/- modifiers on merge-file rules.

Fixed an off-by-one error in the prefix-length checking in
get_rule_prefix().

Improved the name of a variable.

Mention a couple more changes.

Added the "C" modifier to the itemized list of +/- modifiers.

Mention the new --filter=-C syntax that lets you position the default
CVS rules within your other filter rules.

- Moved get_cvs_excludes() up above its only use and made it static.
- Changed get_cvs_excludes() to make an mflags arg.

If we're a "local server" (i.e. we just forked a server for a
local transfer), don't duplicate all the filter rules that we
already know about in the forked process.

Fixed the way get_rule_prefix() turns the MATCHFLG_WORD_SPLIT flag
back into a modifier character.

- Added MATCHFLG_CVS_IGNORE.
- Made the match_flags var a uint32.

- Moved the cvs_exclude checks into send_filter_list() and
  recv_filter_list().  Because of this, we never skip these
  calls anymore, though read-batch processing indicates that
  the list should not really be sent by setting f_out to -1.
- Moved the definintion of the "local_server" variable here
  from options.c.

- Changed filter_rule() to add_rule().
- Improved the debug info coming from add_rule().
- Changed add_filter() to parse_rule() and add_filter_file()
  to parse_filter_file().
- Changed get_filter_tok() to parse_rule_tok().
- Made the mflags a uint32.
- Added an extra arg to get_rule_prefix() to indicate if we're
  building the options for transmission or other purposes.
- The 'C' modifier can now be applied to a '-' rule, so it now
  sets a new flag: MATCHFLG_CVS_IGNORE.
- The send_filter_list() and recv_filter_list() functions now handle
  all the logic needed to do the right thing for -C (this used to be
  in main.c).

- Changed add_filter() to parse_rule() and add_filter_file()
  to parse_filter_file().
- Moved the definintion of the "local_server" variable into main.c.

Changed add_filter() to parse_rule() and add_filter_file()
to parse_filter_file().

Call get_rule_prefix() with its new arg.

Fixed the MODIFIERS_MERGE_FILE define.

Changed delete_missing() to not limit the use of DEL_FORCE_RECURSE
to just --delete-during now that the other delete routines are using
delete_in_dir().

Added an mflags arg to get_filter_tok(), add_filter_file(), and
add_filter().  This made for less flag conversion between various
MATCHFLG_* and XFLG_* values.  It also made it easy to fix a bug
in the handling of no-prefix per-directory include/exclude files.
We also use the new XFLG_OLD_PREFIXES and MATCHFLGS_FROM_CONTAINER
defines.

Call add_filter() and add_filter_file() with their new flag args.

- Got rid of a bunch of XFLG_* options.
- Added XFLG_OLD_PREFIXES and MATCHFLGS_FROM_CONTAINER defines.

Use the new negated exclude to filter out all non-dirs in a few
of the rsync commands.

Document the new "!" modifier for include/exclude filter rules (to
negate the match).

Implemented a "!" modifier for filter rules that lets a rule trigger
on a non-matching pattern.

Added MATCHFLG_NEGATE.

Mention the latest delete optimization.

Changed delete_files() to use the delete_in_dir() function, which
saves a ton of memory for a large set of files.

- We now separate the user-specified top-dir flag (via the restored
  FLAG_TOP_DIR) from the del-in-this-directory flag (FLAG_DEL_HERE).
  This was needed to properly handle -x in delete_in_dir().
- The delete_in_dir() function takes a slightly changed set of args.
- Always set the FLAG_DEL_HERE flags, not just for --delete-during.

- Changed {XMIT,FLAG}_DEL_START back to {XMIT,FLAG}_TOP_DIR.
- Define FLAG_DEL_HERE.

Call delete_in_dir() with its new args.

In hlink_compare(), call f_name_cmp() directly instead of using
file_compare().

- Improved some comments.
- Changed the index passed to recv_generator() from "i" to "ndx".
- Call delete_in_dir() with its new args.
- During the redo phase, clear a bunch of options that might interfere
  with the generator deciding it needs to redo the file (for instance,
  --update-only could have interfered if --partial was specified).

- Moved an optimization from file_compare() into f_name_cmp() (the one
  that avoids doing a string-compare on two identical char pointers),
  making it just a wrapper for f_name_cmp().  Also made file_compare()
  static because everyone now calls f_name_cmp() directly.
- Improved the flist summary that is output at high verbosity.
- Improved delete_in_dir() to better handle the push/pop semantics
  based on the depth value that is now passed in as an arg (so that
  we don't have to parse it out of the paths each time).

- If recv_files() is sent the index of a directory, complain about it
  and die.
- Moved a few code snippets in recv_files() into better locations.

If the generator sends us the index of a directory, generate an
error and die.

- Fixed a problem with the setting of the --recurse option from the
  batch file.
- Save the state of the --dirs option in the batch flags in a way
  that is compatible with older protocol versions.

In read_sum_head: output the who_am_i() info in our error messages.

Set am_sender to -1 until the end of the option parsing.  This
lets who_am_i() output better values for the command-line
--filter/--include/--exclude options.

Changed who_am_i() to output "server" or "client" when am_sender
is negative.  This will only occur during the startup phase
before we know if we're the sender or not (e.g. when parsing
client-side --filter/--exclude/--include options).

Make sure the %f expansion handles the new dir.root properly.

- Made flist_find() and receive_file_entry() static functions.
- Changed receive_file_entry() args.
- We now store the directory depth into file->dir.depth when
  receiving the file list.  This will be used to aid in the
  traversal of the file list on the receiving side for things
  such as the future --fuzzy option and better --delete-during
  processing.
- Fixed a bug when -R was used with "." as a source directory:
  subdirectories were not being promoted to FLAG_DEL_START when
  --delete-during was specified.
- Refer to the old basedir variable as dir.root.
- Got rid of the first-push code -- I decided that it wasn't
  the right thing to do for per-dir merge-file processing.
- Improved the flist_find() function to make it a little more
  optimal.  The new code no longer needs the inline function
  flist_up().
- In clean_flist() we now set two new values in the flist struct:
  "low" and "high".  These are used by the new flist_find().
- Tweaked the output_flist() function to output each entry's
  flags and to output the directory-depth on the receiving side
  in place of the directory-root (which is still output on the
  sending side).

- Put "basedir" into a union "dir" and named it "root".  The other
  member of the union is "depth".
- Added members "low" and "high" to the file list structure (which
  are set when the list is cleaned).
- Got rid of the flist_up() inline function.

Refer to the old basedir variable as dir.root.

Fixed a typo in a variable name.

Changed the "s" modifier to "w" for word-splitting tokens from a
merge filter file.  Also sprinkled some bf(...) macros into that
section.

Changed the "s" modifier to "w" for a word-splitting merge file.

- Mark references to command-line switches with the bf(...) macro.
- Changed some verb(...) macros to use other yodl idioms because
  yodl inserts way too much vertical whitespace surrounding the
  item.
- Some items weren't indenting enough in the manpage, though they
  were indenting fine in the HTML version, so used some creative
  idioms to get things to render well in both.
- Got rid of some empty lines in lists.
- Made several quote(...) items also tt(...).

Tweaked one or more warning messages.

Improved the exclude tests to make sure that the filter files
are interacting with --delete-during correctly.

Added "need_first_push" logic to delete_in_dir().

The --delete-during processing only happens during the first phase
(not the redo phase).  We also call delete_in_dir(NULL, NULL) to
make sure that all the local filter files get popped.

delete_in_dir() now mananges the push/pop levels of nested
directories properly.

- Now delete_in_dir() calls send_directory(), not send_file_name().
- Moved the local-list push/pop code out of send_directory() so that
  the callers (send_file_name() and delete_in_dir()) can call the
  push/pop functions themselves.  This is in preparation for
  delete_in_dir() getting fully correct push/pop semantics (for
  its --delete-during handling of per-dir filter files).

- Restored the io_error test to delete_files().
- Sanity check the dir_list value from send_file_list().

- Made delete_files() call send_file_list() and delete_missing()
  instead of delete_in_dir().
- Tweaked check in front of delete_files() call.