[PATCH] Comment an obscure bit of code that sets the --suffix-dels default.

Matt McCutchen matt at mattmccutchen.net
Tue Mar 11 04:52:58 GMT 2008


---
More work on backup-dir-dels...

 options.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/options.c b/options.c
index 459ad58..ed21704 100644
--- a/options.c
+++ b/options.c
@@ -1479,6 +1479,11 @@ int parse_arguments(int *argc_p, const char ***argv_p, int frommain)
 			backup_suffix);
 		return 0;
 	}
+	/* --suffix-dels defaults to --suffix, or empty for a client given an
+	 * explicit --backup-dir-dels (just as --suffix defaults to empty when
+	 * a --backup-dir is given).  The second case does not apply to the
+	 * server for consistency with server_options, which sends --suffix-dels
+	 * to the server iff it differs from --suffix. */
 	if (!backup_suffix_dels)
 		backup_suffix_dels = backup_dir_dels && !am_server ? "" : backup_suffix;
 	backup_suffix_dels_len = strlen(backup_suffix_dels);
-- 
1.5.4.3.193.g6dd0e




More information about the rsync mailing list