[gtk+] Drop the last mentions of motion hint events



commit bfcb9780791c4ca00b054874c03d9a3722771ea8
Author: Emmanuele Bassi <ebassi gnome org>
Date:   Mon Feb 5 13:52:46 2018 +0000

    Drop the last mentions of motion hint events
    
    The GDK_POINTER_MOTION_HINT_MASK enumeration value is gone, but we're
    still keeping around the "is_hint" field in GdkEventMotion, even though
    every backend sets it to `false` — except for the core X11 device
    manager.

 gdk/gdkeventsprivate.h              |    3 ---
 gdk/quartz/gdkevents-quartz.c       |   12 ++++--------
 gdk/wayland/gdkdevice-wayland.c     |    2 --
 gdk/win32/gdkdevicemanager-win32.c  |    1 -
 gdk/win32/gdkevents-win32.c         |    6 ++----
 gdk/x11/gdkdevicemanager-core-x11.c |    1 -
 gdk/x11/gdkdevicemanager-xi2.c      |    3 ---
 7 files changed, 6 insertions(+), 22 deletions(-)
---
diff --git a/gdk/gdkeventsprivate.h b/gdk/gdkeventsprivate.h
index 41ffc52..1daffc2 100644
--- a/gdk/gdkeventsprivate.h
+++ b/gdk/gdkeventsprivate.h
@@ -99,8 +99,6 @@ struct _GdkEventExpose
  * @state: (type GdkModifierType): a bit-mask representing the state of
  *   the modifier keys (e.g. Control, Shift and Alt) and the pointer
  *   buttons. See #GdkModifierType.
- * @is_hint: set to 1 if this event is just a hint, see the
- *   %GDK_POINTER_MOTION_HINT_MASK value of #GdkEventMask.
  * @device: the master device that the event originated from. Use
  * gdk_event_get_source_device() to get the slave device.
  * @x_root: the x coordinate of the pointer relative to the root of the
@@ -118,7 +116,6 @@ struct _GdkEventMotion
   gdouble y;
   gdouble *axes;
   guint state;
-  gint16 is_hint;
   GdkDeviceTool *tool;
   gdouble x_root, y_root;
   GList *history;
diff --git a/gdk/quartz/gdkevents-quartz.c b/gdk/quartz/gdkevents-quartz.c
index 521c1a3..4544bc1 100644
--- a/gdk/quartz/gdkevents-quartz.c
+++ b/gdk/quartz/gdkevents-quartz.c
@@ -316,7 +316,7 @@ get_event_mask_from_ns_event (NSEvent *nsevent)
     case NSOtherMouseUp:
       return GDK_BUTTON_RELEASE_MASK;
     case NSMouseMoved:
-      return GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK;
+      return GDK_POINTER_MOTION_MASK;
     case NSScrollWheel:
       /* Since applications that want button press events can get
        * scroll events on X11 (since scroll wheel events are really
@@ -324,20 +324,18 @@ get_event_mask_from_ns_event (NSEvent *nsevent)
        */
       return GDK_SCROLL_MASK | GDK_BUTTON_PRESS_MASK;
     case NSLeftMouseDragged:
-      return (GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK |
+      return (GDK_POINTER_MOTION_MASK |
              GDK_BUTTON_MOTION_MASK | GDK_BUTTON1_MOTION_MASK | 
              GDK_BUTTON1_MASK);
     case NSRightMouseDragged:
-      return (GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK |
+      return (GDK_POINTER_MOTION_MASK |
              GDK_BUTTON_MOTION_MASK | GDK_BUTTON3_MOTION_MASK | 
              GDK_BUTTON3_MASK);
     case NSOtherMouseDragged:
       {
        GdkEventMask mask;
 
-       mask = (GDK_POINTER_MOTION_MASK |
-               GDK_POINTER_MOTION_HINT_MASK |
-               GDK_BUTTON_MOTION_MASK);
+       mask = (GDK_POINTER_MOTION_MASK | GDK_BUTTON_MOTION_MASK);
 
        if (get_mouse_button_from_ns_event (nsevent) == 2)
          mask |= (GDK_BUTTON2_MOTION_MASK | GDK_BUTTON2_MOTION_MASK | 
@@ -535,7 +533,6 @@ generate_motion_event (GdkWindow *window)
   /* FIXME event->axes */
   event->motion.state = _gdk_quartz_events_get_current_keyboard_modifiers () |
                         _gdk_quartz_events_get_current_mouse_modifiers ();
-  event->motion.is_hint = FALSE;
   device_manager = GDK_QUARTZ_DEVICE_MANAGER_CORE (_gdk_device_manager);
   event->motion.device = device_manager->core_pointer;
   gdk_event_set_seat (event, gdk_device_get_seat (device_manager->core_pointer));
@@ -1070,7 +1067,6 @@ fill_motion_event (GdkWindow *window,
   /* FIXME event->axes */
   event->motion.state = get_keyboard_modifiers_from_ns_event (nsevent) |
                         _gdk_quartz_events_get_current_mouse_modifiers ();
-  event->motion.is_hint = FALSE;
   device_manager = GDK_QUARTZ_DEVICE_MANAGER_CORE (_gdk_device_manager);
   event->motion.device = device_manager->core_pointer;
   gdk_event_set_seat (event, gdk_device_get_seat (device_manager->core_pointer));
diff --git a/gdk/wayland/gdkdevice-wayland.c b/gdk/wayland/gdkdevice-wayland.c
index 0946c32..7d8882f 100644
--- a/gdk/wayland/gdkdevice-wayland.c
+++ b/gdk/wayland/gdkdevice-wayland.c
@@ -1482,7 +1482,6 @@ pointer_handle_motion (void              *data,
   event->motion.time = time;
   event->motion.axes = NULL;
   event->motion.state = device_get_modifiers (seat->master_pointer);
-  event->motion.is_hint = 0;
   gdk_event_set_display (event, seat->display);
 
   get_coordinates (seat->master_pointer,
@@ -3525,7 +3524,6 @@ tablet_tool_handle_motion (void                      *data,
   gdk_event_set_device_tool (event, tool->tool);
   event->motion.time = tablet->pointer_info.time;
   event->motion.state = device_get_modifiers (tablet->master);
-  event->motion.is_hint = FALSE;
   gdk_event_set_display (event, seat->display);
 
   get_coordinates (tablet->master,
diff --git a/gdk/win32/gdkdevicemanager-win32.c b/gdk/win32/gdkdevicemanager-win32.c
index f6dbe60..28507fb 100644
--- a/gdk/win32/gdkdevicemanager-win32.c
+++ b/gdk/win32/gdkdevicemanager-win32.c
@@ -1123,7 +1123,6 @@ gdk_input_other_event (GdkDisplay *display,
       else
         {
           event->motion.time = _gdk_win32_get_next_tick (msg->time);
-          event->motion.is_hint = FALSE;
           gdk_event_set_device (event, device_manager->core_pointer);
           gdk_event_set_source_device (event, GDK_DEVICE (source_device));
           gdk_event_set_seat (event, gdk_device_get_seat (device_manager->core_pointer));
diff --git a/gdk/win32/gdkevents-win32.c b/gdk/win32/gdkevents-win32.c
index 59c9f5c..83102ad 100644
--- a/gdk/win32/gdkevents-win32.c
+++ b/gdk/win32/gdkevents-win32.c
@@ -815,10 +815,9 @@ _gdk_win32_print_event (const GdkEvent *event)
               event->expose.count);
       break;
     case GDK_MOTION_NOTIFY:
-      g_print ("(%.4g,%.4g) (%.4g,%.4g) %s",
+      g_print ("(%.4g,%.4g) (%.4g,%.4g)",
               event->motion.x, event->motion.y,
-              event->motion.x_root, event->motion.y_root,
-              event->motion.is_hint ? "HINT " : "");
+              event->motion.x_root, event->motion.y_root);
       print_event_state (event->motion.state);
       break;
     case GDK_BUTTON_PRESS:
@@ -2947,7 +2946,6 @@ gdk_event_translate (MSG  *msg,
          event->motion.y_root = current_root_y;
          event->motion.axes = NULL;
          event->motion.state = build_pointer_event_state (msg);
-         event->motion.is_hint = FALSE;
          gdk_event_set_device (event, device_manager_win32->core_pointer);
          gdk_event_set_source_device (event, device_manager_win32->system_pointer);
           gdk_event_set_seat (event, gdk_device_get_seat (device_manager_win32->core_pointer));
diff --git a/gdk/x11/gdkdevicemanager-core-x11.c b/gdk/x11/gdkdevicemanager-core-x11.c
index 3062902..0845f2e 100644
--- a/gdk/x11/gdkdevicemanager-core-x11.c
+++ b/gdk/x11/gdkdevicemanager-core-x11.c
@@ -595,7 +595,6 @@ gdk_x11_device_manager_core_translate_event (GdkEventTranslator *translator,
       event->motion.y_root = (gdouble) xevent->xmotion.y_root / scale;
       event->motion.axes = NULL;
       event->motion.state = (GdkModifierType) xevent->xmotion.state;
-      event->motion.is_hint = xevent->xmotion.is_hint;
       gdk_event_set_device (event, device_manager->core_pointer);
 
       gdk_event_set_display (event, display);
diff --git a/gdk/x11/gdkdevicemanager-xi2.c b/gdk/x11/gdkdevicemanager-xi2.c
index bd8fe7d..7f079b0 100644
--- a/gdk/x11/gdkdevicemanager-xi2.c
+++ b/gdk/x11/gdkdevicemanager-xi2.c
@@ -1643,9 +1643,6 @@ gdk_x11_device_manager_xi2_translate_event (GdkEventTranslator *translator,
 
         event->motion.state = _gdk_x11_device_xi2_translate_state (&xev->mods, &xev->buttons, &xev->group);
 
-        /* There doesn't seem to be motion hints in XI */
-        event->motion.is_hint = FALSE;
-
         event->motion.axes = translate_axes (device,
                                              event->motion.x,
                                              event->motion.y,


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