From: Wayne Davison <wayned@samba.org>
Date: Mon, 13 Mar 2006 21:50:55 +0000 (+0000)
Subject: The tag char doesn't need to be unsigned.
X-Git-Url: https://mattmccutchen.net/rsync/rsync-patches.git/commitdiff_plain/dfadc68a0c54e98a996becf6d94579fcac0ba732

The tag char doesn't need to be unsigned.
---

diff --git a/acls.diff b/acls.diff
index 1c6fa10..8373667 100644
--- a/acls.diff
+++ b/acls.diff
@@ -365,7 +365,7 @@ the file that does not need any other attribute updates.
 + * lowercase in this instance means there's no ACL following, so the
 + * ACL is a repeat, so the receiver should reuse the last of the same
 + * type ACL. */
-+static void send_ida_list(int f, const ida_list *idal, uchar tag_char)
++static void send_ida_list(int f, const ida_list *idal, char tag_char)
 +{
 +	id_access *ida;
 +	size_t count = idal->count;
@@ -742,7 +742,7 @@ the file that does not need any other attribute updates.
 +		return;
 +
 +	while (count--) {
-+		uchar tag = read_byte(f);
++		char tag = read_byte(f);
 +		uchar access = read_byte(f);
 +		if (access & ~ (4 | 2 | 1)) {
 +			rprintf(FERROR, "receive_rsync_acl: bogus permset %o\n",
@@ -832,7 +832,7 @@ the file that does not need any other attribute updates.
 +	do {
 +		file_acl_index_list *fileaclidx_list =
 +			file_acl_index_lists(type);
-+		uchar tag;
++		char tag;
 +		expand_file_acl_index_list(fileaclidx_list);
 +
 +		tag = read_byte(f);