[gtk] gdk: Get rid of gdk_event_free()
- From: Benjamin Otte <otte src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk] gdk: Get rid of gdk_event_free()
- Date: Tue, 29 May 2018 20:08:38 +0000 (UTC)
commit 7e574fa98c04dc771a683d248b69bdfa9671da93
Author: Benjamin Otte <otte redhat com>
Date: Tue May 29 21:53:44 2018 +0200
gdk: Get rid of gdk_event_free()
Events are objects, so use g_object_unref().
demos/gtk-demo/popover.c | 2 +-
docs/reference/gdk/gdk4-sections.txt | 1 -
gdk/broadway/gdkeventsource.c | 2 +-
gdk/gdk-autocleanup.h | 2 +-
gdk/gdkdisplay.c | 2 +-
gdk/gdkevents.c | 16 +---------------
gdk/gdkevents.h | 2 --
gdk/gdksurface.c | 4 ++--
gdk/quartz/GdkQuartzNSWindow.c | 10 +++++-----
gdk/quartz/gdkeventloop-quartz.c | 2 +-
gdk/quartz/gdkevents-quartz.c | 2 +-
gdk/win32/gdkdrag-win32.c | 6 +++---
gdk/win32/gdkdrop-win32.c | 8 ++++----
gdk/win32/gdkevents-win32.c | 4 ++--
gdk/x11/gdkeventtranslator.c | 2 +-
gtk/a11y/gtkentryaccessible.c | 2 +-
testsuite/gtk/gestures.c | 6 +++---
17 files changed, 28 insertions(+), 45 deletions(-)
---
diff --git a/demos/gtk-demo/popover.c b/demos/gtk-demo/popover.c
index d39ed6103c..f49949adf0 100644
--- a/demos/gtk-demo/popover.c
+++ b/demos/gtk-demo/popover.c
@@ -119,7 +119,7 @@ day_selected_cb (GtkCalendar *calendar,
gtk_widget_show (popover);
- gdk_event_free (event);
+ g_object_unref (event);
}
GtkWidget *
diff --git a/docs/reference/gdk/gdk4-sections.txt b/docs/reference/gdk/gdk4-sections.txt
index 2948955fb8..b99d2ea559 100644
--- a/docs/reference/gdk/gdk4-sections.txt
+++ b/docs/reference/gdk/gdk4-sections.txt
@@ -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
diff --git a/gdk/broadway/gdkeventsource.c b/gdk/broadway/gdkeventsource.c
index 327ac016a3..0eb603a960 100644
--- a/gdk/broadway/gdkeventsource.c
+++ b/gdk/broadway/gdkeventsource.c
@@ -383,7 +383,7 @@ gdk_event_source_dispatch (GSource *source,
{
_gdk_event_emit (event);
- gdk_event_free (event);
+ g_object_unref (event);
}
return TRUE;
diff --git a/gdk/gdk-autocleanup.h b/gdk/gdk-autocleanup.h
index b47e1c48c4..1a2e18eaf0 100644
--- a/gdk/gdk-autocleanup.h
+++ b/gdk/gdk-autocleanup.h
@@ -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)
diff --git a/gdk/gdkdisplay.c b/gdk/gdkdisplay.c
index 93449e99be..e414adf52b 100644
--- a/gdk/gdkdisplay.c
+++ b/gdk/gdkdisplay.c
@@ -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;
diff --git a/gdk/gdkevents.c b/gdk/gdkevents.c
index 63437da78d..f952df4985 100644
--- a/gdk/gdkevents.c
+++ b/gdk/gdkevents.c
@@ -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)
{
diff --git a/gdk/gdkevents.h b/gdk/gdkevents.h
index a014def3dd..4f21e3b0f4 100644
--- a/gdk/gdkevents.h
+++ b/gdk/gdkevents.h
@@ -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);
diff --git a/gdk/gdksurface.c b/gdk/gdksurface.c
index be41a73674..aac5af5ea4 100644
--- a/gdk/gdksurface.c
+++ b/gdk/gdksurface.c
@@ -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
diff --git a/gdk/quartz/GdkQuartzNSWindow.c b/gdk/quartz/GdkQuartzNSWindow.c
index 74dddeeeaa..bac6225d4e 100644
--- a/gdk/quartz/GdkQuartzNSWindow.c
+++ b/gdk/quartz/GdkQuartzNSWindow.c
@@ -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;
diff --git a/gdk/quartz/gdkeventloop-quartz.c b/gdk/quartz/gdkeventloop-quartz.c
index bd015ff30a..a9875ed44f 100644
--- a/gdk/quartz/gdkeventloop-quartz.c
+++ b/gdk/quartz/gdkeventloop-quartz.c
@@ -685,7 +685,7 @@ gdk_event_dispatch (GSource *source,
{
_gdk_event_emit (event);
- gdk_event_free (event);
+ g_object_unref (event);
}
return TRUE;
diff --git a/gdk/quartz/gdkevents-quartz.c b/gdk/quartz/gdkevents-quartz.c
index f9178c3172..151e9ff0be 100644
--- a/gdk/quartz/gdkevents-quartz.c
+++ b/gdk/quartz/gdkevents-quartz.c
@@ -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];
}
diff --git a/gdk/win32/gdkdrag-win32.c b/gdk/win32/gdkdrag-win32.c
index e490ee07fc..f6ff6cf351 100644
--- a/gdk/win32/gdkdrag-win32.c
+++ b/gdk/win32/gdkdrag-win32.c
@@ -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);
}
}
diff --git a/gdk/win32/gdkdrop-win32.c b/gdk/win32/gdkdrop-win32.c
index 48a78e79de..5c3018985c 100644
--- a/gdk/win32/gdkdrop-win32.c
+++ b/gdk/win32/gdkdrop-win32.c
@@ -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
diff --git a/gdk/win32/gdkevents-win32.c b/gdk/win32/gdkevents-win32.c
index 102b75155e..ac8186c2b0 100644
--- a/gdk/win32/gdkevents-win32.c
+++ b/gdk/win32/gdkevents-win32.c
@@ -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;
diff --git a/gdk/x11/gdkeventtranslator.c b/gdk/x11/gdkeventtranslator.c
index 7344b882bc..d475ce0148 100644
--- a/gdk/x11/gdkeventtranslator.c
+++ b/gdk/x11/gdkeventtranslator.c
@@ -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;
}
diff --git a/gtk/a11y/gtkentryaccessible.c b/gtk/a11y/gtkentryaccessible.c
index 30bfaabb8c..3b23c7a64d 100644
--- a/gtk/a11y/gtkentryaccessible.c
+++ b/gtk/a11y/gtkentryaccessible.c
@@ -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;
}
diff --git a/testsuite/gtk/gestures.c b/testsuite/gtk/gestures.c
index 3f0f806636..42aac2d65f 100644
--- a/testsuite/gtk/gestures.c
+++ b/testsuite/gtk/gestures.c
@@ -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 *
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]