dovecot-2.0: Compiler warning fixes.

dovecot at dovecot.org dovecot at dovecot.org
Thu Mar 11 03:22:51 EET 2010


details:   http://hg.dovecot.org/dovecot-2.0/rev/588892d877dc
changeset: 10887:588892d877dc
user:      Timo Sirainen <tss at iki.fi>
date:      Thu Mar 11 03:22:36 2010 +0200
description:
Compiler warning fixes.

diffstat:

 src/lib-storage/index/maildir/maildir-uidlist.c |  4 ++--
 src/lib-storage/index/mbox/istream-raw-mbox.c   |  4 ++--
 src/lib-storage/index/mbox/mbox-sync-update.c   |  2 +-
 src/lib/test-istream-concat.c                   |  2 +-
 src/lib/test-istream-seekable.c                 |  2 +-
 src/master/service-anvil.c                      |  2 +-
 6 files changed, 8 insertions(+), 8 deletions(-)

diffs (90 lines):

diff -r f49bb060d83d -r 588892d877dc src/lib-storage/index/maildir/maildir-uidlist.c
--- a/src/lib-storage/index/maildir/maildir-uidlist.c	Wed Mar 10 11:56:49 2010 +0200
+++ b/src/lib-storage/index/maildir/maildir-uidlist.c	Thu Mar 11 03:22:36 2010 +0200
@@ -1080,7 +1080,7 @@
 	p = rec->extensions;
 	while (*p != '\0') {
 		/* <key><value>\0 */
-		if (*p == (char)key)
+		if (*p == (unsigned char)key)
 			return (const char *)p + 1;
 
 		p += strlen((const char *)p) + 1;
@@ -1176,7 +1176,7 @@
 		while (*p != '\0') {
 			/* <key><value>\0 */
 			len = strlen((const char *)p) + 1;
-			if (*p != (char)key)
+			if (*p != (unsigned char)key)
 				buffer_append(buf, p, len);
 			p += len;
 		}
diff -r f49bb060d83d -r 588892d877dc src/lib-storage/index/mbox/istream-raw-mbox.c
--- a/src/lib-storage/index/mbox/istream-raw-mbox.c	Wed Mar 10 11:56:49 2010 +0200
+++ b/src/lib-storage/index/mbox/istream-raw-mbox.c	Thu Mar 11 03:22:36 2010 +0200
@@ -285,7 +285,7 @@
 			rstream->body_offset = stream->istream.v_offset + i + 1;
 			eoh_char = -1;
 		}
-		if (buf[i] == *fromp) {
+		if ((char)buf[i] == *fromp) {
 			if (*++fromp == '\0') {
 				/* potential From-line, see if we have the
 				   rest of the line buffered. */
@@ -335,7 +335,7 @@
 			}
 		} else {
 			fromp = mbox_from;
-			if (buf[i] == *fromp)
+			if ((char)buf[i] == *fromp)
 				fromp++;
 		}
 	}
diff -r f49bb060d83d -r 588892d877dc src/lib-storage/index/mbox/mbox-sync-update.c
--- a/src/lib-storage/index/mbox/mbox-sync-update.c	Wed Mar 10 11:56:49 2010 +0200
+++ b/src/lib-storage/index/mbox/mbox-sync-update.c	Thu Mar 11 03:22:36 2010 +0200
@@ -96,7 +96,7 @@
 
 		/* see if this is unknown flag for us */
 		for (i = 0; flags_list[i].chr != 0; i++) {
-			if (flags_list[i].chr == data[pos])
+			if (flags_list[i].chr == (char)data[pos])
 				break;
 		}
 
diff -r f49bb060d83d -r 588892d877dc src/lib/test-istream-concat.c
--- a/src/lib/test-istream-concat.c	Wed Mar 10 11:56:49 2010 +0200
+++ b/src/lib/test-istream-concat.c	Thu Mar 11 03:22:36 2010 +0200
@@ -39,7 +39,7 @@
 			test_assert(size == buffer_size);
 		}
 		for (j = 0; j < size; j++) {
-			test_assert(data[j] == input_string[(input->v_offset + j) % STREAM_BYTES]);
+			test_assert((char)data[j] == input_string[(input->v_offset + j) % STREAM_BYTES]);
 		}
 	}
 	i_stream_unref(&input);
diff -r f49bb060d83d -r 588892d877dc src/lib/test-istream-seekable.c
--- a/src/lib/test-istream-seekable.c	Wed Mar 10 11:56:49 2010 +0200
+++ b/src/lib/test-istream-seekable.c	Thu Mar 11 03:22:36 2010 +0200
@@ -55,7 +55,7 @@
 			test_assert(size == buffer_size);
 		}
 		for (j = 0; j < size; j++) {
-			test_assert(data[j] == input_string[(input->v_offset + j) % STREAM_BYTES]);
+			test_assert((char)data[j] == input_string[(input->v_offset + j) % STREAM_BYTES]);
 		}
 	}
 	for (i = 0; i < STREAM_COUNT; i++)
diff -r f49bb060d83d -r 588892d877dc src/master/service-anvil.c
--- a/src/master/service-anvil.c	Wed Mar 10 11:56:49 2010 +0200
+++ b/src/master/service-anvil.c	Thu Mar 11 03:22:36 2010 +0200
@@ -72,7 +72,7 @@
 		return -1;
 	}
 	/* this is a pipe, it either wrote all of it or nothing */
-	i_assert(ret == strlen(ANVIL_HANDSHAKE));
+	i_assert((size_t)ret == strlen(ANVIL_HANDSHAKE));
 	return 0;
 }
 


More information about the dovecot-cvs mailing list