[gtk/readonly-events-1] win32: Remove some leftover event struct access



commit 90a9aaf3a865cb4f5a886becb93c7d82ee71953d
Author: Matthias Clasen <mclasen redhat com>
Date:   Tue Feb 18 00:38:55 2020 -0500

    win32: Remove some leftover event struct access
    
    And leftover old event api calls.

 gdk/win32/gdkdevicemanager-win32.c |  7 ++--
 gdk/win32/gdkdrag-win32.c          | 83 +++++++++++++++++++-------------------
 gdk/win32/gdkevents-win32.c        | 19 +++++----
 gtk/gtkimcontextime.c              |  4 +-
 4 files changed, 55 insertions(+), 58 deletions(-)
---
diff --git a/gdk/win32/gdkdevicemanager-win32.c b/gdk/win32/gdkdevicemanager-win32.c
index 8ebd1822ab..c2030b1f1a 100644
--- a/gdk/win32/gdkdevicemanager-win32.c
+++ b/gdk/win32/gdkdevicemanager-win32.c
@@ -886,7 +886,6 @@ gdk_input_other_event (GdkDisplay *display,
   GdkEvent *event;
 
   PACKET packet;
-  gint root_x, root_y;
   gint num_axes;
   double x, y;
   guint translated_buttons, button_diff, button_mask;
@@ -1028,7 +1027,7 @@ gdk_input_other_event (GdkDisplay *display,
           event_button = 1;
 
           for (button_mask = 1; button_mask != 0x80000000;
-               button_mask <<= 1, event->button.button++)
+               button_mask <<= 1, event_button++)
             {
               if (button_diff & button_mask)
                 {
@@ -1073,7 +1072,7 @@ gdk_input_other_event (GdkDisplay *display,
           event = gdk_event_button_new (event_type,
                                         window,
                                         device_manager->core_pointer,
-                                        source_device,
+                                        GDK_DEVICE (source_device),
                                         NULL,
                                         _gdk_win32_get_next_tick (msg->time),
                                         event_state,
@@ -1106,7 +1105,7 @@ gdk_input_other_event (GdkDisplay *display,
 
           event = gdk_event_motion_new (window,
                                         device_manager->core_pointer,
-                                        source_device,
+                                        GDK_DEVICE (source_device),
                                         NULL,
                                         _gdk_win32_get_next_tick (msg->time),
                                         event_state,
diff --git a/gdk/win32/gdkdrag-win32.c b/gdk/win32/gdkdrag-win32.c
index 9eadebe3f8..24df191d05 100644
--- a/gdk/win32/gdkdrag-win32.c
+++ b/gdk/win32/gdkdrag-win32.c
@@ -2331,28 +2331,32 @@ gdk_local_drag_update (GdkDrag *drag,
 }
 
 static gboolean
-gdk_dnd_handle_motion_event (GdkDrag              *drag,
-                             GdkEventMotion *event)
+gdk_dnd_handle_motion_event (GdkDrag  *drag,
+                             GdkEvent *event)
 {
   GdkModifierType state;
   GdkWin32Drag *drag_win32 = GDK_WIN32_DRAG (drag);
   DWORD key_state;
+  double x, y;
+  double x_root, y_root;
 
-  if (!gdk_event_get_state ((GdkEvent *) event, &state))
-    return FALSE;
+  GDK_NOTE (DND, g_print ("gdk_dnd_handle_motion_event: 0x%p\n", drag));
 
-  GDK_NOTE (DND, g_print ("gdk_dnd_handle_motion_event: 0x%p\n",
-                          drag));
+  state = gdk_event_get_modifier_state (event);
+  gdk_event_get_position (event, &x, &y);
+
+  x_root = x + _gdk_offset_x;
+  y_root = y + _gdk_offset_y;
 
   if (drag_win32->drag_surface)
-    move_drag_surface (drag, event->x_root, event->y_root);
+    move_drag_surface (drag, x_root, y_root);
 
   key_state = manufacture_keystate_from_GMT (state);
 
   if (drag_win32->protocol == GDK_DRAG_PROTO_LOCAL)
     {
-      gdk_local_drag_update (drag, event->x_root, event->y_root, key_state,
-                             gdk_event_get_time ((GdkEvent *) event));
+      gdk_local_drag_update (drag, x_root, y_root, key_state,
+                             gdk_event_get_time (event));
     }
   else if (drag_win32->protocol == GDK_DRAG_PROTO_OLE2)
     {
@@ -2360,40 +2364,38 @@ gdk_dnd_handle_motion_event (GdkDrag              *drag,
 
       GDK_NOTE (DND, g_print ("Post WM_MOUSEMOVE keystate=%lu\n", key_state));
 
-      drag_win32->util_data.last_x = event->x_root;
-      drag_win32->util_data.last_y = event->y_root;
+      drag_win32->util_data.last_x = x_root;
+      drag_win32->util_data.last_y = y_root;
 
       API_CALL (PostThreadMessage, (clipdrop->dnd_thread_id,
                                     WM_MOUSEMOVE,
                                     key_state,
-                                    MAKELPARAM ((event->x_root - _gdk_offset_x) * drag_win32->scale,
-                                                (event->y_root - _gdk_offset_y) * drag_win32->scale)));
+                                    MAKELPARAM (x * drag_win32->scale,
+                                                y * drag_win32->scale)));
     }
 
   return TRUE;
 }
 
 static gboolean
-gdk_dnd_handle_key_event (GdkDrag           *drag,
-                          GdkEventKey *event)
+gdk_dnd_handle_key_event (GdkDrag  *drag,
+                          GdkEvent *event)
 {
   GdkWin32Drag *drag_win32 = GDK_WIN32_DRAG (drag);
   GdkModifierType state;
   GdkDevice *pointer;
   gint dx, dy;
 
-  if (!gdk_event_get_state ((GdkEvent *) event, &state))
-    return FALSE;
+  GDK_NOTE (DND, g_print ("gdk_dnd_handle_key_event: 0x%p\n", drag));
 
-  GDK_NOTE (DND, g_print ("gdk_dnd_handle_key_event: 0x%p\n",
-                          drag));
+  state = gdk_event_get_modifier_state (event);
 
   dx = dy = 0;
-  pointer = gdk_device_get_associated_device (gdk_event_get_device ((GdkEvent *) event));
+  pointer = gdk_device_get_associated_device (gdk_event_get_device (event));
 
-  if (event->any.type == GDK_KEY_PRESS)
+  if (gdk_event_get_event_type (event) == GDK_KEY_PRESS)
     {
-      switch (event->keyval)
+      switch (gdk_key_event_get_keyval (event))
         {
         case GDK_KEY_Escape:
           gdk_drag_cancel (drag, GDK_DRAG_CANCEL_USER_CANCELLED);
@@ -2453,31 +2455,29 @@ gdk_dnd_handle_key_event (GdkDrag           *drag,
   if (drag_win32->protocol == GDK_DRAG_PROTO_LOCAL)
     gdk_local_drag_update (drag, drag_win32->util_data.last_x, drag_win32->util_data.last_y,
                            manufacture_keystate_from_GMT (state),
-                           gdk_event_get_time ((GdkEvent *) event));
+                           gdk_event_get_time (event));
 
   return TRUE;
 }
 
 static gboolean
-gdk_dnd_handle_grab_broken_event (GdkDrag                  *drag,
-                                  GdkEventGrabBroken *event)
+gdk_dnd_handle_grab_broken_event (GdkDrag  *drag,
+                                  GdkEvent *event)
 {
   GdkWin32Drag *drag_win32 = GDK_WIN32_DRAG (drag);
 
-  GDK_NOTE (DND, g_print ("gdk_dnd_handle_grab_broken_event: 0x%p\n",
-                          drag));
+  GDK_NOTE (DND, g_print ("gdk_dnd_handle_grab_broken_event: 0x%p\n", drag));
 
   /* Don't cancel if we break the implicit grab from the initial button_press.
    * Also, don't cancel if we re-grab on the widget or on our grab window, for
    * example, when changing the drag cursor.
    */
-  if (event->implicit ||
-      event->grab_surface == drag_win32->drag_surface ||
-      event->grab_surface == drag_win32->grab_surface)
+  if (/* FIXME: event->implicit || */
+      gdk_grab_broken_event_get_grab_surface (event) == drag_win32->drag_surface ||
+      gdk_grab_broken_event_get_grab_surface (event) == drag_win32->grab_surface)
     return FALSE;
 
-  if (gdk_event_get_device ((GdkEvent *) event) !=
-      gdk_drag_get_device (drag))
+  if (gdk_event_get_device (event) != gdk_drag_get_device (drag))
     return FALSE;
 
   gdk_drag_cancel (drag, GDK_DRAG_CANCEL_ERROR);
@@ -2485,11 +2485,10 @@ gdk_dnd_handle_grab_broken_event (GdkDrag                  *drag,
 }
 
 static gboolean
-gdk_dnd_handle_button_event (GdkDrag              *drag,
-                             GdkEventButton *event)
+gdk_dnd_handle_button_event (GdkDrag  *drag,
+                             GdkEvent *event)
 {
-  GDK_NOTE (DND, g_print ("gdk_dnd_handle_button_event: 0x%p\n",
-                          drag));
+  GDK_NOTE (DND, g_print ("gdk_dnd_handle_button_event: 0x%p\n", drag));
 
 #if 0
   /* FIXME: Check the button matches */
@@ -2509,7 +2508,7 @@ gdk_dnd_handle_button_event (GdkDrag              *drag,
 }
 
 gboolean
-gdk_win32_drag_handle_event (GdkDrag        *drag,
+gdk_win32_drag_handle_event (GdkDrag  *drag,
                              GdkEvent *event)
 {
   GdkWin32Drag *drag_win32 = GDK_WIN32_DRAG (drag);
@@ -2523,17 +2522,17 @@ gdk_win32_drag_handle_event (GdkDrag        *drag,
       return FALSE;
     }
 
-  switch (event->any.type)
+  switch (gdk_event_get_event_type (event))
     {
     case GDK_MOTION_NOTIFY:
-      return gdk_dnd_handle_motion_event (drag, &event->motion);
+      return gdk_dnd_handle_motion_event (drag, event);
     case GDK_BUTTON_RELEASE:
-      return gdk_dnd_handle_button_event (drag, &event->button);
+      return gdk_dnd_handle_button_event (drag, event);
     case GDK_KEY_PRESS:
     case GDK_KEY_RELEASE:
-      return gdk_dnd_handle_key_event (drag, &event->key);
+      return gdk_dnd_handle_key_event (drag, event);
     case GDK_GRAB_BROKEN:
-      return gdk_dnd_handle_grab_broken_event (drag, &event->grab_broken);
+      return gdk_dnd_handle_grab_broken_event (drag, event);
     default:
       break;
     }
diff --git a/gdk/win32/gdkevents-win32.c b/gdk/win32/gdkevents-win32.c
index 8980d0de26..50dcc2fde1 100644
--- a/gdk/win32/gdkevents-win32.c
+++ b/gdk/win32/gdkevents-win32.c
@@ -892,7 +892,6 @@ fixup_event (GdkEvent *event)
        (event->any.type == GDK_LEAVE_NOTIFY)) &&
       (event->crossing.child_surface != NULL))
     g_object_ref (event->crossing.child_surface);
-  event->any.send_event = InSendMessage ();
 }
 
 void
@@ -1086,7 +1085,7 @@ send_crossing_event (GdkDisplay                 *display,
                                   window,
                                   device_manager->core_pointer,
                                   device_manager->system_pointer,
-                                  _gdk_win32_next_tick (time_),
+                                  _gdk_win32_get_next_tick (time_),
                                   mask,
                                   pt.x / impl->surface_scale,
                                   pt.y / impl->surface_scale,
@@ -2794,14 +2793,14 @@ gdk_event_translate (MSG  *msg,
                       ? GDK_SCROLL_RIGHT
                       : GDK_SCROLL_LEFT;
 
-      event = gdk_event_scroll_new (window,
-                                    device_manager_win32->core_pointer,
-                                    device_manager_win32->system_pointer,
-                                    NULL,
-                                    _gdk_win32_get_next_tick (msg->time),
-                                    build_pointer_event_state (msg),
-                                    direction,
-                                    TRUE);
+      event = gdk_event_discrete_scroll_new (window,
+                                             device_manager_win32->core_pointer,
+                                             device_manager_win32->system_pointer,
+                                             NULL,
+                                             _gdk_win32_get_next_tick (msg->time),
+                                             build_pointer_event_state (msg),
+                                             direction,
+                                             TRUE);
 
       _gdk_win32_append_event (event);
 
diff --git a/gtk/gtkimcontextime.c b/gtk/gtkimcontextime.c
index a44ac73304..ced066a66b 100644
--- a/gtk/gtkimcontextime.c
+++ b/gtk/gtkimcontextime.c
@@ -359,7 +359,7 @@ gtk_im_context_ime_filter_keypress (GtkIMContext *context,
   if (gdk_event_get_event_type ((GdkEvent *) event) == GDK_KEY_RELEASE)
     return FALSE;
 
-  gdk_event_get_state ((GdkEvent *) event, &state);
+  state = gdk_event_get_modifier_state ((GdkEvent *) event);
 
   if (state & GDK_CONTROL_MASK)
     return FALSE;
@@ -372,7 +372,7 @@ gtk_im_context_ime_filter_keypress (GtkIMContext *context,
   if (!GDK_IS_SURFACE (context_ime->client_surface))
     return FALSE;
 
-  gdk_event_get_keyval ((GdkEvent *) event, &keyval);
+  keyval = gdk_key_event_get_keyval ((GdkEvent *) event);
 
   if (keyval == GDK_KEY_space &&
       context_ime->priv->dead_key_keyval != 0)


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