[gtk+] x11: Constify XEvent usage
- From: Benjamin Otte <otte src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+] x11: Constify XEvent usage
- Date: Wed, 13 Dec 2017 00:14:05 +0000 (UTC)
commit f34297cfbade4f127f52d80ecbb119e1029378f3
Author: Benjamin Otte <otte redhat com>
Date: Wed Dec 13 00:39:32 2017 +0100
x11: Constify XEvent usage
gdk/x11/gdkdevicemanager-core-x11.c | 8 ++++----
gdk/x11/gdkdevicemanager-xi2.c | 12 ++++++------
gdk/x11/gdkdisplay-x11.c | 8 ++++----
gdk/x11/gdkdnd-x11.c | 24 +++++++++++-------------
gdk/x11/gdkeventsource.c | 10 +++++-----
gdk/x11/gdkeventtranslator.c | 4 ++--
gdk/x11/gdkeventtranslator.h | 10 +++++-----
gdk/x11/gdkkeys-x11.c | 6 +++---
gdk/x11/gdkprivate-x11.h | 18 +++++++++---------
gdk/x11/gdkscreen-x11.c | 4 ++--
gdk/x11/gdkscreen-x11.h | 2 +-
gdk/x11/gdkselection-x11.c | 2 +-
gdk/x11/gdkwindow-x11.c | 8 ++++----
gdk/x11/xsettings-client.c | 14 ++++++--------
gdk/x11/xsettings-client.h | 12 ++++++------
15 files changed, 69 insertions(+), 73 deletions(-)
---
diff --git a/gdk/x11/gdkdevicemanager-core-x11.c b/gdk/x11/gdkdevicemanager-core-x11.c
index 22e55a6..c1b5427 100644
--- a/gdk/x11/gdkdevicemanager-core-x11.c
+++ b/gdk/x11/gdkdevicemanager-core-x11.c
@@ -41,7 +41,7 @@ static void gdk_x11_device_manager_event_translator_init (GdkEventTranslator
static gboolean gdk_x11_device_manager_core_translate_event (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent);
+ const XEvent *xevent);
G_DEFINE_TYPE_WITH_CODE (GdkX11DeviceManagerCore, gdk_x11_device_manager_core, G_TYPE_OBJECT,
@@ -191,7 +191,7 @@ static void
translate_key_event (GdkDisplay *display,
GdkX11DeviceManagerCore *device_manager,
GdkEvent *event,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkKeymap *keymap = gdk_keymap_get_for_display (display);
GdkModifierType consumed, state;
@@ -334,7 +334,7 @@ is_parent_of (GdkWindow *parent,
static GdkWindow *
get_event_window (GdkEventTranslator *translator,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkDisplay *display;
GdkWindow *window;
@@ -368,7 +368,7 @@ static gboolean
gdk_x11_device_manager_core_translate_event (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkWindowImplX11 *impl;
GdkX11DeviceManagerCore *device_manager;
diff --git a/gdk/x11/gdkdevicemanager-xi2.c b/gdk/x11/gdkdevicemanager-xi2.c
index a7ec3a9..07c2113 100644
--- a/gdk/x11/gdkdevicemanager-xi2.c
+++ b/gdk/x11/gdkdevicemanager-xi2.c
@@ -79,13 +79,13 @@ static GdkDevice * gdk_x11_device_manager_xi2_get_client_pointer (GdkX11DeviceMa
static gboolean gdk_x11_device_manager_xi2_translate_event (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent);
+ const XEvent *xevent);
static GdkEventMask gdk_x11_device_manager_xi2_get_handled_events (GdkEventTranslator *translator);
static void gdk_x11_device_manager_xi2_select_window_events (GdkEventTranslator *translator,
Window window,
GdkEventMask event_mask);
static GdkWindow * gdk_x11_device_manager_xi2_get_window (GdkEventTranslator *translator,
- XEvent *xevent);
+ const XEvent *xevent);
enum {
PROP_0,
@@ -1225,7 +1225,7 @@ static gboolean
gdk_x11_device_manager_xi2_translate_core_event (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkEventTranslatorIface *parent_iface;
gboolean keyboard = FALSE;
@@ -1341,10 +1341,10 @@ static gboolean
gdk_x11_device_manager_xi2_translate_event (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkX11DeviceManagerXI2 *device_manager;
- XGenericEventCookie *cookie;
+ const XGenericEventCookie *cookie;
GdkDevice *device, *source_device;
gboolean return_val = TRUE;
GdkWindow *window;
@@ -1943,7 +1943,7 @@ gdk_x11_device_manager_xi2_select_window_events (GdkEventTranslator *translator,
static GdkWindow *
gdk_x11_device_manager_xi2_get_window (GdkEventTranslator *translator,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkX11DeviceManagerXI2 *device_manager;
XIEvent *ev;
diff --git a/gdk/x11/gdkdisplay-x11.c b/gdk/x11/gdkdisplay-x11.c
index 579510a..a474443 100644
--- a/gdk/x11/gdkdisplay-x11.c
+++ b/gdk/x11/gdkdisplay-x11.c
@@ -102,7 +102,7 @@ static void gdk_x11_display_event_translator_init (GdkEventTranslatorIface *
static gboolean gdk_x11_display_translate_event (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent);
+ const XEvent *xevent);
static void gdk_internal_connection_watch (Display *display,
XPointer arg,
@@ -575,7 +575,7 @@ get_cm_atom (GdkDisplay *display)
}
static Window
-get_event_xwindow (XEvent *xevent)
+get_event_xwindow (const XEvent *xevent)
{
Window xwindow;
@@ -613,7 +613,7 @@ static gboolean
gdk_x11_display_translate_event (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent)
+ const XEvent *xevent)
{
Window xwindow;
GdkWindow *window;
@@ -1124,7 +1124,7 @@ gdk_x11_display_translate_event (GdkEventTranslator *translator,
/* Let XLib know that there is a new keyboard mapping.
*/
- XRefreshKeyboardMapping (&xevent->xmapping);
+ XRefreshKeyboardMapping ((XMappingEvent *) xevent);
_gdk_x11_keymap_keys_changed (display);
return_val = FALSE;
break;
diff --git a/gdk/x11/gdkdnd-x11.c b/gdk/x11/gdkdnd-x11.c
index a6b76ea..3443fb2 100644
--- a/gdk/x11/gdkdnd-x11.c
+++ b/gdk/x11/gdkdnd-x11.c
@@ -526,11 +526,10 @@ gdk_window_cache_add (GdkWindowCache *cache,
}
GdkFilterReturn
-gdk_window_cache_shape_filter (GdkXEvent *xev,
- GdkEvent *event,
- gpointer data)
+gdk_window_cache_shape_filter (const XEvent *xevent,
+ GdkEvent *event,
+ gpointer data)
{
- XEvent *xevent = (XEvent *)xev;
GdkWindowCache *cache = data;
GdkX11Display *display = GDK_X11_DISPLAY (cache->display);
@@ -561,11 +560,10 @@ gdk_window_cache_shape_filter (GdkXEvent *xev,
}
GdkFilterReturn
-gdk_window_cache_filter (GdkXEvent *xev,
- GdkEvent *event,
- gpointer data)
+gdk_window_cache_filter (const XEvent *xevent,
+ GdkEvent *event,
+ gpointer data)
{
- XEvent *xevent = (XEvent *)xev;
GdkWindowCache *cache = data;
switch (xevent->type)
@@ -574,7 +572,7 @@ gdk_window_cache_filter (GdkXEvent *xev,
break;
case ConfigureNotify:
{
- XConfigureEvent *xce = &xevent->xconfigure;
+ const XConfigureEvent *xce = &xevent->xconfigure;
GList *node;
node = g_hash_table_lookup (cache->child_hash,
@@ -616,7 +614,7 @@ gdk_window_cache_filter (GdkXEvent *xev,
}
case CreateNotify:
{
- XCreateWindowEvent *xcwe = &xevent->xcreatewindow;
+ const XCreateWindowEvent *xcwe = &xevent->xcreatewindow;
if (!g_hash_table_lookup (cache->child_hash,
GUINT_TO_POINTER (xcwe->window)))
@@ -627,7 +625,7 @@ gdk_window_cache_filter (GdkXEvent *xev,
}
case DestroyNotify:
{
- XDestroyWindowEvent *xdwe = &xevent->xdestroywindow;
+ const XDestroyWindowEvent *xdwe = &xevent->xdestroywindow;
GList *node;
node = g_hash_table_lookup (cache->child_hash,
@@ -647,7 +645,7 @@ gdk_window_cache_filter (GdkXEvent *xev,
}
case MapNotify:
{
- XMapEvent *xme = &xevent->xmap;
+ const XMapEvent *xme = &xevent->xmap;
GList *node;
node = g_hash_table_lookup (cache->child_hash,
@@ -663,7 +661,7 @@ gdk_window_cache_filter (GdkXEvent *xev,
break;
case UnmapNotify:
{
- XMapEvent *xume = &xevent->xmap;
+ const XMapEvent *xume = &xevent->xmap;
GList *node;
node = g_hash_table_lookup (cache->child_hash,
diff --git a/gdk/x11/gdkeventsource.c b/gdk/x11/gdkeventsource.c
index cbd5c17..5de5b7b 100644
--- a/gdk/x11/gdkeventsource.c
+++ b/gdk/x11/gdkeventsource.c
@@ -55,9 +55,9 @@ static GSourceFuncs event_funcs = {
};
static gint
-gdk_event_apply_filters (XEvent *xevent,
- GdkEvent *event,
- GdkWindow *window)
+gdk_event_apply_filters (const XEvent *xevent,
+ GdkEvent *event,
+ GdkWindow *window)
{
GList *tmp_list;
GdkFilterReturn result;
@@ -79,7 +79,7 @@ gdk_event_apply_filters (XEvent *xevent,
}
filter->ref_count++;
- result = filter->function (xevent, event, filter->data);
+ result = filter->function ((GdkXEvent *) xevent, event, filter->data);
/* Protect against unreffing the filter mutating the list */
node = tmp_list->next;
@@ -97,7 +97,7 @@ gdk_event_apply_filters (XEvent *xevent,
static GdkWindow *
gdk_event_source_get_filter_window (GdkEventSource *event_source,
- XEvent *xevent,
+ const XEvent *xevent,
GdkEventTranslator **event_translator)
{
GList *list = event_source->translators;
diff --git a/gdk/x11/gdkeventtranslator.c b/gdk/x11/gdkeventtranslator.c
index 588fb06..7bb4c66 100644
--- a/gdk/x11/gdkeventtranslator.c
+++ b/gdk/x11/gdkeventtranslator.c
@@ -33,7 +33,7 @@ _gdk_x11_event_translator_default_init (GdkEventTranslatorInterface *iface)
GdkEvent *
_gdk_x11_event_translator_translate (GdkEventTranslator *translator,
GdkDisplay *display,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkEventTranslatorIface *iface;
GdkEvent *event;
@@ -89,7 +89,7 @@ _gdk_x11_event_translator_select_window_events (GdkEventTranslator *translator,
GdkWindow *
_gdk_x11_event_translator_get_window (GdkEventTranslator *translator,
GdkDisplay *display,
- XEvent *xevent)
+ const XEvent *xevent)
{
GdkEventTranslatorIface *iface;
diff --git a/gdk/x11/gdkeventtranslator.h b/gdk/x11/gdkeventtranslator.h
index 0e46e35..1bd9d78 100644
--- a/gdk/x11/gdkeventtranslator.h
+++ b/gdk/x11/gdkeventtranslator.h
@@ -41,28 +41,28 @@ struct _GdkEventTranslatorIface
gboolean (* translate_event) (GdkEventTranslator *translator,
GdkDisplay *display,
GdkEvent *event,
- XEvent *xevent);
+ const XEvent *xevent);
GdkEventMask (* get_handled_events) (GdkEventTranslator *translator);
void (* select_window_events) (GdkEventTranslator *translator,
Window window,
GdkEventMask event_mask);
GdkWindow * (* get_window) (GdkEventTranslator *translator,
- XEvent *xevent);
+ const XEvent *xevent);
};
GType _gdk_x11_event_translator_get_type (void) G_GNUC_CONST;
GdkEvent * _gdk_x11_event_translator_translate (GdkEventTranslator *translator,
- GdkDisplay *display,
- XEvent *xevent);
+ GdkDisplay *display,
+ const XEvent *xevent);
GdkEventMask _gdk_x11_event_translator_get_handled_events (GdkEventTranslator *translator);
void _gdk_x11_event_translator_select_window_events (GdkEventTranslator *translator,
Window window,
GdkEventMask event_mask);
GdkWindow * _gdk_x11_event_translator_get_window (GdkEventTranslator *translator,
GdkDisplay *display,
- XEvent *xevent);
+ const XEvent *xevent);
G_END_DECLS
diff --git a/gdk/x11/gdkkeys-x11.c b/gdk/x11/gdkkeys-x11.c
index 1814a4c..93a32a1 100644
--- a/gdk/x11/gdkkeys-x11.c
+++ b/gdk/x11/gdkkeys-x11.c
@@ -642,11 +642,11 @@ update_lock_state (GdkX11Keymap *keymap_x11,
* call in gdk_display_open()
*/
void
-_gdk_x11_keymap_state_changed (GdkDisplay *display,
- XEvent *xevent)
+_gdk_x11_keymap_state_changed (GdkDisplay *display,
+ const XEvent *xevent)
{
GdkX11Display *display_x11 = GDK_X11_DISPLAY (display);
- XkbEvent *xkb_event = (XkbEvent *)xevent;
+ const XkbEvent *xkb_event = (const XkbEvent *) xevent;
if (display_x11->keymap)
{
diff --git a/gdk/x11/gdkprivate-x11.h b/gdk/x11/gdkprivate-x11.h
index 990e203..8d09197 100644
--- a/gdk/x11/gdkprivate-x11.h
+++ b/gdk/x11/gdkprivate-x11.h
@@ -103,7 +103,7 @@ void _gdk_x11_window_translate (GdkWindow *window,
void _gdk_x11_display_free_translate_queue (GdkDisplay *display);
void _gdk_x11_selection_window_destroyed (GdkWindow *window);
-gboolean _gdk_x11_selection_filter_clear_event (XSelectionClearEvent *event);
+gboolean _gdk_x11_selection_filter_clear_event (const XSelectionClearEvent *event);
cairo_region_t* _gdk_x11_xwindow_get_shape (Display *xdisplay,
Window window,
@@ -117,12 +117,12 @@ void _gdk_x11_region_get_xrectangles (const cairo_region_t *region,
XRectangle **rects,
gint *n_rects);
-gboolean _gdk_x11_moveresize_handle_event (XEvent *event);
+gboolean _gdk_x11_moveresize_handle_event (const XEvent *event);
gboolean _gdk_x11_moveresize_configure_done (GdkDisplay *display,
GdkWindow *window);
void _gdk_x11_keymap_state_changed (GdkDisplay *display,
- XEvent *event);
+ const XEvent *event);
void _gdk_x11_keymap_keys_changed (GdkDisplay *display);
void _gdk_x11_keymap_add_virt_mods (GdkKeymap *keymap,
GdkModifierType *modifiers);
@@ -270,13 +270,13 @@ GdkWindowCache *
gdk_window_cache_get (GdkDisplay *display);
GdkFilterReturn
-gdk_window_cache_filter (GdkXEvent *xev,
- GdkEvent *event,
- gpointer data);
+gdk_window_cache_filter (const XEvent *xevent,
+ GdkEvent *event,
+ gpointer data);
GdkFilterReturn
-gdk_window_cache_shape_filter (GdkXEvent *xev,
- GdkEvent *event,
- gpointer data);
+gdk_window_cache_shape_filter (const XEvent *xevent,
+ GdkEvent *event,
+ gpointer data);
void _gdk_x11_cursor_display_finalize (GdkDisplay *display);
diff --git a/gdk/x11/gdkscreen-x11.c b/gdk/x11/gdkscreen-x11.c
index c38ed9c..6af8a3d 100644
--- a/gdk/x11/gdkscreen-x11.c
+++ b/gdk/x11/gdkscreen-x11.c
@@ -851,7 +851,7 @@ process_monitors_change (GdkX11Screen *screen)
void
_gdk_x11_screen_size_changed (GdkX11Screen *screen,
- XEvent *event)
+ const XEvent *event)
{
#ifdef HAVE_RANDR
GdkX11Display *display_x11;
@@ -861,7 +861,7 @@ _gdk_x11_screen_size_changed (GdkX11Screen *screen,
if (display_x11->have_randr13 && event->type == ConfigureNotify)
return;
- XRRUpdateConfiguration (event);
+ XRRUpdateConfiguration ((XEvent *) event);
#else
if (event->type != ConfigureNotify)
return;
diff --git a/gdk/x11/gdkscreen-x11.h b/gdk/x11/gdkscreen-x11.h
index 0b14d31..9553b4b 100644
--- a/gdk/x11/gdkscreen-x11.h
+++ b/gdk/x11/gdkscreen-x11.h
@@ -102,7 +102,7 @@ GdkX11Screen *_gdk_x11_screen_new (GdkDisplay *display,
void _gdk_x11_screen_update_visuals_for_gl (GdkX11Screen *screen);
void _gdk_x11_screen_window_manager_changed (GdkX11Screen *screen);
void _gdk_x11_screen_size_changed (GdkX11Screen *screen,
- XEvent *event);
+ const XEvent *event);
void _gdk_x11_screen_get_edge_monitors (GdkX11Screen *screen,
gint *top,
gint *bottom,
diff --git a/gdk/x11/gdkselection-x11.c b/gdk/x11/gdkselection-x11.c
index 7772881..04b6018 100644
--- a/gdk/x11/gdkselection-x11.c
+++ b/gdk/x11/gdkselection-x11.c
@@ -71,7 +71,7 @@ _gdk_x11_selection_window_destroyed (GdkWindow *window)
* reflect changes to the selection owner that we didn’t make ourself.
*/
gboolean
-_gdk_x11_selection_filter_clear_event (XSelectionClearEvent *event)
+_gdk_x11_selection_filter_clear_event (const XSelectionClearEvent *event)
{
GSList *tmp_list = owner_list;
GdkDisplay *display = gdk_x11_lookup_xdisplay (event->display);
diff --git a/gdk/x11/gdkwindow-x11.c b/gdk/x11/gdkwindow-x11.c
index 3c9106a..3675ae6 100644
--- a/gdk/x11/gdkwindow-x11.c
+++ b/gdk/x11/gdkwindow-x11.c
@@ -3933,8 +3933,8 @@ wmspec_send_message (GdkDisplay *display,
}
static void
-handle_wmspec_button_release (GdkDisplay *display,
- XEvent *xevent)
+handle_wmspec_button_release (GdkDisplay *display,
+ const XEvent *xevent)
{
GdkX11Display *display_x11 = GDK_X11_DISPLAY (display);
GdkWindow *window;
@@ -4261,7 +4261,7 @@ lookahead_motion_predicate (Display *xdisplay,
static gboolean
moveresize_lookahead (MoveResizeData *mv_resize,
- XEvent *event)
+ const XEvent *event)
{
XEvent tmp_event;
gboolean seen_release = FALSE;
@@ -4284,7 +4284,7 @@ moveresize_lookahead (MoveResizeData *mv_resize,
}
gboolean
-_gdk_x11_moveresize_handle_event (XEvent *event)
+_gdk_x11_moveresize_handle_event (const XEvent *event)
{
guint button_mask = 0;
GdkDisplay *display = gdk_x11_lookup_xdisplay (event->xany.display);
diff --git a/gdk/x11/xsettings-client.c b/gdk/x11/xsettings-client.c
index 5275b19..888cb50 100644
--- a/gdk/x11/xsettings-client.c
+++ b/gdk/x11/xsettings-client.c
@@ -502,13 +502,12 @@ check_manager_window (GdkX11Screen *x11_screen,
}
GdkFilterReturn
-gdk_xsettings_root_window_filter (GdkXEvent *xevent,
- GdkEvent *event,
- gpointer data)
+gdk_xsettings_root_window_filter (const XEvent *xev,
+ GdkEvent *event,
+ gpointer data)
{
GdkX11Screen *x11_screen = data;
GdkDisplay *display = x11_screen->display;
- XEvent *xev = xevent;
/* The checks here will not unlikely cause us to reread
* the properties from the manager window a number of
@@ -527,12 +526,11 @@ gdk_xsettings_root_window_filter (GdkXEvent *xevent,
}
GdkFilterReturn
-gdk_xsettings_manager_window_filter (GdkXEvent *xevent,
- GdkEvent *event,
- gpointer data)
+gdk_xsettings_manager_window_filter (const XEvent *xev,
+ GdkEvent *event,
+ gpointer data)
{
GdkX11Screen *x11_screen = data;
- XEvent *xev = xevent;
if (xev->xany.type == DestroyNotify)
{
diff --git a/gdk/x11/xsettings-client.h b/gdk/x11/xsettings-client.h
index aacac57..0dbeb9f 100644
--- a/gdk/x11/xsettings-client.h
+++ b/gdk/x11/xsettings-client.h
@@ -29,12 +29,12 @@ void _gdk_x11_xsettings_init (GdkX11Screen *x11_screen);
void _gdk_x11_xsettings_finish (GdkX11Screen *x11_screen);
void _gdk_x11_settings_force_reread (GdkX11Screen *x11_screen);
-GdkFilterReturn gdk_xsettings_root_window_filter (GdkXEvent *xevent,
- GdkEvent *event,
- gpointer data);
-GdkFilterReturn gdk_xsettings_manager_window_filter (GdkXEvent *xevent,
- GdkEvent *event,
- gpointer data);
+GdkFilterReturn gdk_xsettings_root_window_filter (const XEvent *xevent,
+ GdkEvent *event,
+ gpointer data);
+GdkFilterReturn gdk_xsettings_manager_window_filter (const XEvent *xevent,
+ GdkEvent *event,
+ gpointer data);
#endif /* XSETTINGS_CLIENT_H */
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]