gdk: Get rid of gdk_event_free()
authorBenjamin Otte <otte@redhat.com>
Tue, 29 May 2018 19:53:44 +0000 (21:53 +0200)
committerBenjamin Otte <otte@redhat.com>
Tue, 29 May 2018 19:53:44 +0000 (21:53 +0200)
Events are objects, so use g_object_unref().

17 files changed:
demos/gtk-demo/popover.c
docs/reference/gdk/gdk4-sections.txt
gdk/broadway/gdkeventsource.c
gdk/gdk-autocleanup.h
gdk/gdkdisplay.c
gdk/gdkevents.c
gdk/gdkevents.h
gdk/gdksurface.c
gdk/quartz/GdkQuartzNSWindow.c
gdk/quartz/gdkeventloop-quartz.c
gdk/quartz/gdkevents-quartz.c
gdk/win32/gdkdrag-win32.c
gdk/win32/gdkdrop-win32.c
gdk/win32/gdkevents-win32.c
gdk/x11/gdkeventtranslator.c
gtk/a11y/gtkentryaccessible.c
testsuite/gtk/gestures.c

index d39ed6103c860ceac014c6c3bb2521e6c809f79c..f49949adf0d6020b25e61c18bfca4a914ceb8396 100644 (file)
@@ -119,7 +119,7 @@ day_selected_cb (GtkCalendar *calendar,
 
   gtk_widget_show (popover);
 
-  gdk_event_free (event);
+  g_object_unref (event);
 }
 
 GtkWidget *
index 2948955fb849bac2b923bc191250191f9dc7ad7a..b99d2ea5592018608ef87c91fbb900b636b174c4 100644 (file)
@@ -627,7 +627,6 @@ GDK_BUTTON_SECONDARY
 <SUBSECTION>
 gdk_event_new
 gdk_event_copy
-gdk_event_free
 gdk_event_get_axes
 gdk_event_get_button
 gdk_event_get_click_count
index 327ac016a3552f4e9f988c49f3906b6ba4a4fecf..0eb603a960167f517689eece936c4890d2493116 100644 (file)
@@ -383,7 +383,7 @@ gdk_event_source_dispatch (GSource     *source,
     {
       _gdk_event_emit (event);
 
-      gdk_event_free (event);
+      g_object_unref (event);
     }
 
   return TRUE;
index b47e1c48c429904a2a2a1755fa450a4eef29c0d0..1a2e18eaf0c6999c4f71205c78e9a86def81a037 100644 (file)
@@ -39,7 +39,7 @@ G_DEFINE_AUTOPTR_CLEANUP_FUNC(GdkVulkanContext, g_object_unref)
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(GdkSurface, g_object_unref)
 
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(GdkContentFormats, gdk_content_formats_unref)
-G_DEFINE_AUTOPTR_CLEANUP_FUNC(GdkEvent, gdk_event_free)
+G_DEFINE_AUTOPTR_CLEANUP_FUNC(GdkEvent, g_object_unref)
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(GdkFrameTimings, gdk_frame_timings_unref)
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(GdkRGBA, gdk_rgba_free)
 
index 93449e99be15a04a9758dac3b24fcc9e0d3bffdd..e414adf52b0d938755e69f1ca3f9c1a5f0f794c8 100644 (file)
@@ -376,7 +376,7 @@ gdk_display_dispose (GObject *object)
 
   _gdk_display_manager_remove_display (gdk_display_manager_get (), display);
 
-  g_list_free_full (display->queued_events, (GDestroyNotify) gdk_event_free);
+  g_list_free_full (display->queued_events, (GDestroyNotify) g_object_unref);
   display->queued_events = NULL;
   display->queued_tail = NULL;
 
index 63437da78d095e4d831b55ed1ae9c3a7f3739609..f952df4985f52942575f617f52f9a85a87bb8a57 100644 (file)
@@ -419,7 +419,7 @@ _gdk_event_queue_handle_motion_compression (GdkDisplay *display)
             GDK_BUTTON4_MASK | GDK_BUTTON5_MASK)))
         gdk_event_push_history (last_motion, pending_motions->data);
 
-      gdk_event_free (pending_motions->data);
+      g_object_unref (pending_motions->data);
       display->queued_events = g_list_delete_link (display->queued_events,
                                                    pending_motions);
       pending_motions = next;
@@ -694,20 +694,6 @@ gdk_event_copy (const GdkEvent *event)
   return new_event;
 }
 
-/**
- * gdk_event_free:
- * @event:  a #GdkEvent.
- *
- * Frees a #GdkEvent, freeing or decrementing any resources associated with it.
- *
- * This is equivalent to g_object_unref().
- */
-void
-gdk_event_free (GdkEvent *event)
-{
-  g_object_unref (event);
-}
-
 void
 gdk_event_finalize (GObject *object)
 {
index a014def3dd7897fd20942d184b07d0844185e46b..4f21e3b0f47dd932255c5e036cb5d808e0751fec 100644 (file)
@@ -377,8 +377,6 @@ GDK_AVAILABLE_IN_ALL
 GdkEvent* gdk_event_new                 (GdkEventType    type);
 GDK_AVAILABLE_IN_ALL
 GdkEvent* gdk_event_copy               (const GdkEvent *event);
-GDK_AVAILABLE_IN_ALL
-void     gdk_event_free                (GdkEvent       *event);
 
 GDK_AVAILABLE_IN_ALL
 GdkSurface *gdk_event_get_surface       (const GdkEvent *event);
index be41a73674a50bb60d5c1901182cb29c380fdfe7..aac5af5ea42e6c1e71d2bd5f3872220f658b5416 100644 (file)
@@ -1706,7 +1706,7 @@ gdk_surface_process_updates_recurse (GdkSurface *surface,
   event->expose.region = cairo_region_reference (expose_region);
 
   _gdk_event_emit (event);
-  gdk_event_free (event);
+  g_object_unref (event);
 }
 
 /* Process and remove any invalid area on the native surface by creating
@@ -4343,7 +4343,7 @@ _gdk_windowing_got_event (GdkDisplay *display,
     {
       _gdk_event_queue_remove_link (display, event_link);
       g_list_free_1 (event_link);
-      gdk_event_free (event);
+      g_object_unref (event);
     }
 
   /* This does two things - first it sees if there are motions at the
index 74dddeeeaa34ae95963f1774841f359e65bef18a..bac6225d4ee1e8b5d8e347a14564475edba9e1b7 100644 (file)
@@ -602,7 +602,7 @@ update_context_from_dragging_info (id <NSDraggingInfo> sender)
 
   _gdk_event_emit (event);
 
-  gdk_event_free (event);
+  g_object_unref (event);
 
   return NSDragOperationNone;
 }
@@ -633,7 +633,7 @@ update_context_from_dragging_info (id <NSDraggingInfo> sender)
 
   _gdk_event_emit (event);
 
-  gdk_event_free (event);
+  g_object_unref (event);
   
   g_object_unref (current_context);
   current_context = NULL;
@@ -662,7 +662,7 @@ update_context_from_dragging_info (id <NSDraggingInfo> sender)
 
   _gdk_event_emit (event);
 
-  gdk_event_free (event);
+  g_object_unref (event);
 
   return drag_action_to_drag_operation (current_context->action);
 }
@@ -690,7 +690,7 @@ update_context_from_dragging_info (id <NSDraggingInfo> sender)
 
   _gdk_event_emit (event);
 
-  gdk_event_free (event);
+  g_object_unref (event);
 
   g_object_unref (current_context);
   current_context = NULL;
@@ -749,7 +749,7 @@ update_context_from_dragging_info (id <NSDraggingInfo> sender)
 
   _gdk_event_emit (event);
 
-  gdk_event_free (event);
+  g_object_unref (event);
 
   g_object_unref (_gdk_quartz_drag_source_context);
   _gdk_quartz_drag_source_context = NULL;
index bd015ff30a7d9e82a00687b7a823ee49d84fb70a..a9875ed44fee4fbe8f7898546f6bca3b831d1c8a 100644 (file)
@@ -685,7 +685,7 @@ gdk_event_dispatch (GSource     *source,
     {
       _gdk_event_emit (event);
 
-      gdk_event_free (event);
+      g_object_unref (event);
     }
 
   return TRUE;
index f9178c3172ff4c432bb3d59c6fc8f78410a56ef7..151e9ff0bea66523cdf38256e38819f65f5a4c7b 100644 (file)
@@ -1726,7 +1726,7 @@ _gdk_quartz_display_queue_events (GdkDisplay *display)
         {
          _gdk_event_queue_remove_link (display, node);
          g_list_free_1 (node);
-         gdk_event_free (event);
+         g_object_unref (event);
 
           [NSApp sendEvent:nsevent];
         }
index e490ee07fc415d9b44b96a6281c3f8c0bcdea7d6..f6ff6cf351c7197d85891983c4b711170be3d073 100644 (file)
@@ -1781,7 +1781,7 @@ local_send_leave (GdkDragContext *context,
 
       GDK_NOTE (EVENTS, _gdk_win32_print_event (tmp_event));
       _gdk_display_put_event (gdk_device_get_display (gdk_drag_context_get_device (context)), tmp_event);
-      gdk_event_free (tmp_event);
+      g_object_unref (tmp_event);
     }
 }
 
@@ -1825,7 +1825,7 @@ local_send_motion (GdkDragContext *context,
 
       GDK_NOTE (EVENTS, _gdk_win32_print_event (tmp_event));
       _gdk_display_put_event (gdk_device_get_display (gdk_drag_context_get_device (context)), tmp_event);
-      gdk_event_free (tmp_event);
+      g_object_unref (tmp_event);
     }
 }
 
@@ -1861,7 +1861,7 @@ local_send_drop (GdkDragContext *context,
 
       GDK_NOTE (EVENTS, _gdk_win32_print_event (tmp_event));
       _gdk_display_put_event (gdk_device_get_display (gdk_drag_context_get_device (context)), tmp_event);
-      gdk_event_free (tmp_event);
+      g_object_unref (tmp_event);
     }
 
 }
index 48a78e79de7250a50f11e845d6759d90d6f02206..5c3018985cb99be64a3b5522bead9caf1885dcec 100644 (file)
@@ -343,7 +343,7 @@ dnd_event_emit (GdkEventType    type,
 
   GDK_NOTE (EVENTS, _gdk_win32_print_event (e));
   _gdk_event_emit (e);
-  gdk_event_free (e);
+  g_object_unref (e);
 }
 
 static GdkContentFormats *
@@ -833,12 +833,12 @@ gdk_dropfiles_filter (GdkWin32Display *display,
 
       GDK_NOTE (EVENTS, _gdk_win32_print_event (event));
       _gdk_event_emit (event);
-      gdk_event_free (event);
+      g_object_unref (event);
 
       DragFinish (hdrop);
 
   gdk_display_put_event (display, event);
-  gdk_event_free (event);
+  g_object_unref (event);
 
   *ret_valp = 0;
 
@@ -1244,7 +1244,7 @@ _gdk_win32_local_send_enter (GdkDragContext *context,
 
   GDK_NOTE (EVENTS, _gdk_win32_print_event (tmp_event));
   _gdk_display_put_event (gdk_device_get_display (gdk_drag_context_get_device (context)), tmp_event);
-  gdk_event_free (tmp_event);
+  g_object_unref (tmp_event);
 }
 
 void
index 102b75155ef4d08b3125204d1730f72ce5806a69..ac8186c2b09a057e3f132a6ebafe859f2b84eeec 100644 (file)
@@ -3728,7 +3728,7 @@ gdk_event_translate (MSG  *msg,
       if (gdk_input_other_event (display, event, msg, window))
        _gdk_win32_append_event (event);
       else
-       gdk_event_free (event);
+       g_object_unref (event);
 
       break;
     }
@@ -3809,7 +3809,7 @@ gdk_event_dispatch (GSource     *source,
     {
       _gdk_event_emit (event);
 
-      gdk_event_free (event);
+      g_object_unref (event);
     }
 
   return TRUE;
index 7344b882bc35aebe986ca544fd79567ef75bd35d..d475ce0148cd2dba8bb4286df7b952fe6b0c11f2 100644 (file)
@@ -51,7 +51,7 @@ _gdk_x11_event_translator_translate (GdkEventTranslator *translator,
   if ((iface->translate_event) (translator, display, event, xevent))
     return event;
 
-  gdk_event_free (event);
+  g_object_unref (event);
 
   return NULL;
 }
index 30bfaabb8c96397e012a5eba164fe7683cf03cc7..3b23c7a64db341f4d88f04e3947edab5ba059060 100644 (file)
@@ -251,7 +251,7 @@ gtk_entry_icon_accessible_do_action (AtkAction *action,
   event->button.y = icon_area.y;
 
   g_signal_emit_by_name (widget, "icon-press", 0, icon->pos, event);
-  gdk_event_free (event);
+  g_object_unref (event);
   return TRUE;
 }
 
index 3f0f806636ab3cf261e4044a23899ce138597631..42aac2d65f09645ace7f88d2583718d6054f46bd 100644 (file)
@@ -61,7 +61,7 @@ point_press (PointState *point,
 
   gtk_main_do_event (ev);
 
-  gdk_event_free (ev);
+  g_object_unref (ev);
 
   point->widget = widget;
 }
@@ -114,7 +114,7 @@ point_update (PointState *point,
 
   gtk_main_do_event (ev);
 
-  gdk_event_free (ev);
+  g_object_unref (ev);
 }
 
 static void
@@ -168,7 +168,7 @@ point_release (PointState *point,
 
   gtk_main_do_event (ev);
 
-  gdk_event_free (ev);
+  g_object_unref (ev);
 }
 
 static const gchar *