[gimp/gtk3-port: 106/228] app: port the device sort function to GdkDeviceManager



commit 7fa3bdb43a262f78a73d87fbab5d9dfc8e7caec5
Author: Michael Natterer <mitch gimp org>
Date:   Thu Dec 9 22:34:27 2010 +0100

    app: port the device sort function to GdkDeviceManager
    
    Also make sure we don't pass around uninitialized keyvals and
    modifiers of device keys.

 app/widgets/gimpdeviceinfo.c |   15 +++++++++++----
 1 files changed, 11 insertions(+), 4 deletions(-)
---
diff --git a/app/widgets/gimpdeviceinfo.c b/app/widgets/gimpdeviceinfo.c
index 0c7d110..e254661 100644
--- a/app/widgets/gimpdeviceinfo.c
+++ b/app/widgets/gimpdeviceinfo.c
@@ -625,8 +625,8 @@ gimp_device_info_set_device (GimpDeviceInfo *info,
 
       for (i = 0; i < MIN (info->n_keys, gdk_device_get_n_keys (device)); i++)
         {
-          guint           keyval;
-          GdkModifierType modifiers;
+          guint           keyval    = 0;
+          GdkModifierType modifiers = 0;
 
           gdk_device_get_key (device, i, &keyval, &modifiers);
           gimp_device_info_set_key (info, i, keyval, modifiers);
@@ -748,6 +748,9 @@ gimp_device_info_get_key (GimpDeviceInfo  *info,
 
   if (info->device)
     {
+      *keyval    = 0;
+      *modifiers = 0;
+
       gdk_device_get_key (info->device, key,
                           keyval,
                           modifiers);
@@ -857,13 +860,17 @@ gint
 gimp_device_info_compare (GimpDeviceInfo *a,
                           GimpDeviceInfo *b)
 {
+  GdkDeviceManager *manager;
+
   if (a->device && a->display &&
-      a->device == gdk_display_get_core_pointer (a->display))
+      (manager = gdk_display_get_device_manager (a->display)) &&
+      a->device == gdk_device_manager_get_client_pointer (manager))
     {
       return -1;
     }
   else if (b->device && b->display &&
-           b->device == gdk_display_get_core_pointer (b->display))
+           (manager = gdk_display_get_device_manager (b->display)) &&
+           b->device == gdk_device_manager_get_client_pointer (manager))
     {
       return 1;
     }



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]