[Xfce4-commits] <xfwm4:master> Don't fool with signedness ^_~

Olivier Fourdan fourdan at xfce.org
Fri Sep 11 09:00:02 CEST 2009


Updating branch refs/heads/master
         to 6326bfdd74094ff7f700aa069d02dd965316fcda (commit)
       from f7582c64227880e7bbfc623004f6f2413ebbdbbe (commit)

commit 6326bfdd74094ff7f700aa069d02dd965316fcda
Author: Olivier Fourdan <fourdan at xfce.org>
Date:   Thu Sep 10 14:32:07 2009 +0200

    Don't fool with signedness ^_~

 src/mypixmap.c |   11 +++++------
 1 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/mypixmap.c b/src/mypixmap.c
index 343f088..8415615 100644
--- a/src/mypixmap.c
+++ b/src/mypixmap.c
@@ -346,7 +346,7 @@ xpm_extract_color (const gchar *buffer, xfwmColorSymbol *color_sym)
     gint new_key;
     gint key;
     gint current_key;
-    guint space;
+    gint space;
 
     p = &buffer[0];
     space = 128;
@@ -367,7 +367,7 @@ xpm_extract_color (const gchar *buffer, xfwmColorSymbol *color_sym)
         for (r = word;
                  (*p != '\0') &&
                  (!g_ascii_isspace (*p)) &&
-                 (r - word < (gchar) sizeof (word) - 1);
+                 (r - word < (gint) sizeof (word) - 1);
              p++, r++)
         {
                 *r = *p;
@@ -428,7 +428,7 @@ xpm_extract_color (const gchar *buffer, xfwmColorSymbol *color_sym)
                 space -= MIN (space, 1);
             }
             strncat (color, word, space);
-            space -= MIN (space, strlen (word));
+            space -= MIN (space, (gint) strlen (word));
         }
         else if (key == 5)
         {
@@ -519,8 +519,7 @@ pixbuf_create_from_xpm (gpointer handle, xfwmColorSymbol *color_sym)
     const gchar *buffer;
     gchar *name_buf;
     gint w, h, n_col, cpp, items;
-    gint cnt, xcnt, ycnt;
-    guint wbytes, n;
+    gint cnt, xcnt, ycnt, wbytes, n;
     GHashTable *color_hash;
     XPMColor *colors, *color, *fallbackcolor;
     guchar *pixtmp;
@@ -633,7 +632,7 @@ pixbuf_create_from_xpm (gpointer handle, xfwmColorSymbol *color_sym)
         pixtmp = gdk_pixbuf_get_pixels (pixbuf) + ycnt * gdk_pixbuf_get_rowstride(pixbuf);
 
         buffer = file_buffer (op_body, handle);
-        if ((!buffer) || (strlen (buffer) < wbytes))
+        if ((!buffer) || (wbytes > (gint) strlen (buffer)))
         {
             continue;
         }



More information about the Xfce4-commits mailing list