[Telepathy-commits] [telepathy-salut/master] Fix comparisons between signed and unsigned in tests

Will Thompson will.thompson at collabora.co.uk
Sun Jan 11 09:20:49 PST 2009


---
 tests/check-gibber-r-multicast-causal-transport.c |    2 +-
 tests/check-gibber-r-multicast-packet.c           |    4 ++--
 tests/check-gibber-r-multicast-sender.c           |    2 +-
 tests/check-gibber-xmpp-error.c                   |    2 +-
 tests/test-r-multicast-transport-io.c             |    4 ++--
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/tests/check-gibber-r-multicast-causal-transport.c b/tests/check-gibber-r-multicast-causal-transport.c
index 9cd3bde..8dc9278 100644
--- a/tests/check-gibber-r-multicast-causal-transport.c
+++ b/tests/check-gibber-r-multicast-causal-transport.c
@@ -96,7 +96,7 @@ depends_send_hook (GibberTransport *transport,
                    gpointer user_data)
 {
   GibberRMulticastPacket *packet;
-  int i, n;
+  guint i, n;
 
   packet = gibber_r_multicast_packet_parse (data, length, NULL);
   fail_unless (packet != NULL);
diff --git a/tests/check-gibber-r-multicast-packet.c b/tests/check-gibber-r-multicast-packet.c
index 44e1a4c..52bfef4 100644
--- a/tests/check-gibber-r-multicast-packet.c
+++ b/tests/check-gibber-r-multicast-packet.c
@@ -64,7 +64,7 @@ START_TEST (test_data_packet)
   gsize len;
   guint8 *pdata;
   gsize plen;
-  int i,n;
+  guint i,n;
   sender_t senders[] =
     { { 0x300, 500, FALSE }, { 0x400, 600, FALSE }, { 0, 0, FALSE } };
   gchar *payload = "1234567890";
@@ -139,7 +139,7 @@ START_TEST (test_attempt_join_packet)
   guint32 packet_id = 1200;
   guint8 *data;
   gsize len;
-  int i, n;
+  guint i, n;
   sender_t senders[] =
     { { 0x300, 500, FALSE }, { 0x400, 600, FALSE }, { 0, 0, FALSE } };
   sender_t new_senders[] =
diff --git a/tests/check-gibber-r-multicast-sender.c b/tests/check-gibber-r-multicast-sender.c
index 2530273..b0a4c90 100644
--- a/tests/check-gibber-r-multicast-sender.c
+++ b/tests/check-gibber-r-multicast-sender.c
@@ -16,7 +16,7 @@
 #define NR_PACKETS ((guint32)40)
 
 guint32 serial_offset;
-int expected;
+guint32 expected;
 
 typedef struct {
   guint32 serial_offset;
diff --git a/tests/check-gibber-xmpp-error.c b/tests/check-gibber-xmpp-error.c
index e601fc8..3acc61c 100644
--- a/tests/check-gibber-xmpp-error.c
+++ b/tests/check-gibber-xmpp-error.c
@@ -119,7 +119,7 @@ START_TEST (test_message_get_xmpp_error)
       fail_if (error == NULL);
 
       fail_if (error->domain != GIBBER_XMPP_ERROR);
-      fail_if (error->code != xmpp_error);
+      fail_if (error->code != (gint) xmpp_error);
       fail_if (strcmp (error->message, gibber_xmpp_error_description (
               xmpp_error)) != 0);
 
diff --git a/tests/test-r-multicast-transport-io.c b/tests/test-r-multicast-transport-io.c
index c12a7c4..62a4e96 100644
--- a/tests/test-r-multicast-transport-io.c
+++ b/tests/test-r-multicast-transport-io.c
@@ -127,7 +127,7 @@ static void
 new_senders_cb (GibberRMulticastTransport *transport,
     GArray *names, gpointer user_data)
 {
-  int i;
+  guint i;
   GString *str = g_string_new ("NEWNODES:");
 
   for (i = 0; i < names->len; i++)
@@ -143,7 +143,7 @@ static void
 lost_senders_cb (GibberRMulticastTransport *transport,
     GArray *names, gpointer user_data)
 {
-  int i;
+  guint i;
   GString *str = g_string_new ("LOSTNODES:");
 
   for (i = 0; i < names->len; i++)
-- 
1.5.6.5




More information about the Telepathy-commits mailing list