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.

1 files changed, 1 insertions, 1 deletions

diff --git a/atspi/atspi-event-listener.c b/atspi/atspi-event-listener.c
index 5cdc8064..b4a07af4 100644
--- a/atspi/atspi-event-listener.c
+++ b/atspi/atspi-event-listener.c
@@ -965,7 +965,7 @@ _atspi_send_event (AtspiEvent *e)
 }
 
 DBusHandlerResult
-_atspi_dbus_handle_event (DBusConnection *bus, DBusMessage *message, void *data)
+_atspi_dbus_handle_event (DBusConnection *bus, DBusMessage *message)
 {
   char *detail = NULL;
   const char *category = dbus_message_get_interface (message);