From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Mon, 5 Sep 2022 17:25:59 +0000 (-0700)
Subject: GTK plugin: squash -Wsign-conversion warnings for 'GdkEventButton' values
X-Git-Tag: 6.0.1~2^2~3
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9474d4dfd8f721dba0a0855d9369e9e74db7781b;p=graphviz

GTK plugin: squash -Wsign-conversion warnings for 'GdkEventButton' values

The GDK API documents these events as coming from an enum, all of whose values
fit in an int.
---

diff --git a/plugin/gtk/callbacks.c b/plugin/gtk/callbacks.c
index d601e7977..fa3bc4c49 100644
--- a/plugin/gtk/callbacks.c
+++ b/plugin/gtk/callbacks.c
@@ -11,6 +11,7 @@
 #include "config.h"
 #include <assert.h>
 #include <gtk/gtk.h>
+#include <limits.h>
 #include <stdbool.h>
 
 #include <gvc/gvplugin_device.h>
@@ -381,7 +382,8 @@ on_drawingarea1_button_press_event     (GtkWidget       *widget,
     GVJ_t *job = g_object_get_data(G_OBJECT(widget),"job");
     pointer.x = event->x;
     pointer.y = event->y;
-    (job->callbacks->button_press)(job, event->button, pointer);
+    assert(event->button <= INT_MAX);
+    (job->callbacks->button_press)(job, (int)event->button, pointer);
     
     load_store_with_attrs(GTK_LIST_STORE(g_object_get_data(G_OBJECT(widget), "attr_store")));
     return FALSE;
@@ -400,7 +402,8 @@ on_drawingarea1_button_release_event   (GtkWidget       *widget,
     GVJ_t *job = g_object_get_data(G_OBJECT(widget),"job");
     pointer.x = event->x;
     pointer.y = event->y;
-    (job->callbacks->button_release)(job, event->button, pointer);
+    assert(event->button <= INT_MAX);
+    (job->callbacks->button_release)(job, (int)event->button, pointer);
 
     return FALSE;
 }