diff options
author | Federico Mena Quintero <federico@gnome.org> | 2022-06-01 13:16:17 -0500 |
---|---|---|
committer | Federico Mena Quintero <federico@gnome.org> | 2022-06-02 16:26:07 -0500 |
commit | fbba144f52c52c1f331b93dd4225d115ebf04630 (patch) | |
tree | b2e7c1be997b590d4ff41a52631d856f27b07c00 /atspi/atspi-device-listener-private.h | |
parent | 6bec005915c4562a17608b0ea3726c9f6cbffc7c (diff) | |
download | at-spi2-core-fbba144f52c52c1f331b93dd4225d115ebf04630.tar.gz |
Remove unused field BusDataClosure.data
The comment "TODO this is still memory leak on c->data" from
destroy_deferred_message_item() caught my eye, so I went looking for
what uses it.
defer_message() is the only place that creates BusDataClosure structs,
and gets passed the data field. However, the only caller of
defer_message() is dbus_connection_add_filter(), and *that* call is
done with both NULL user_data and free_data_function. So,
defer_message() will always get data=NULL.
This commit also removes all the "void *user_data" arguments from the
functions called by process_deferred_message(); they just got passed
the closure->data but none of them actually use it.
Diffstat (limited to 'atspi/atspi-device-listener-private.h')
-rw-r--r-- | atspi/atspi-device-listener-private.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/atspi/atspi-device-listener-private.h b/atspi/atspi-device-listener-private.h index d8b533c5..bd54232d 100644 --- a/atspi/atspi-device-listener-private.h +++ b/atspi/atspi-device-listener-private.h @@ -31,7 +31,7 @@ G_BEGIN_DECLS -DBusHandlerResult _atspi_dbus_handle_DeviceEvent (DBusConnection *bus, DBusMessage *message, void *data); +DBusHandlerResult _atspi_dbus_handle_DeviceEvent (DBusConnection *bus, DBusMessage *message); gchar *_atspi_device_listener_get_path (AtspiDeviceListener *listener); |