]> granicus.if.org Git - graphviz/commitdiff
Xlib plugin: squash some -Wsign-compare warnings
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 5 Mar 2022 18:25:24 +0000 (10:25 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 6 Mar 2022 05:28:32 +0000 (21:28 -0800)
plugin/xlib/gvdevice_xlib.c

index e2423a15105b9bdfde47db03abb1eb0fe4147d25..5758d907a78210a2dd144b75b1598377cd25f344 100644 (file)
@@ -10,6 +10,7 @@
 
 #include "config.h"
 
+#include <assert.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -64,10 +65,14 @@ static void handle_configure_notify(GVJ_t * job, XConfigureEvent * cev)
 /*     plugin/xlib/gvdevice_xlib.c */
 /*     lib/gvc/gvevent.c */
 
+    assert(cev->width >= 0 && "Xlib returned an event with negative width");
+    assert(cev->height >= 0 && "Xlib returned an event with negative height");
+
     job->zoom *= 1 + MIN(
        ((double) cev->width - (double) job->width) / (double) job->width,
        ((double) cev->height - (double) job->height) / (double) job->height);
-    if (cev->width > job->width || cev->height > job->height)
+    if ((unsigned)cev->width > job->width ||
+        (unsigned)cev->height > job->height)
         job->has_grown = true;
     job->width = cev->width;
     job->height = cev->height;