Use "named" parameters for subscribe_to_signal

The signal_subscribe callback signature is hard to understand and easy
to get subtly wrong, with its four unnamed str arguments.

In subscribe_to_signal introduce a new DBusSignalRef struct and put all
received parameters into this struct, to provide callback users with
"named" arguments which are easier to understand and play well with e.g.
editor auto-completion.

Author: swsnr

gio/src/dbus_connection.rs

@@ -187,6 +187,30 @@ impl Drop for WeakSignalSubscription {
     }
 }
 
+// rustdoc-stripper-ignore-next
+/// An emitted D-Bus signal.
+#[derive(Debug, Copy, Clone)]
+pub struct DBusSignalRef<'a> {
+    // rustdoc-stripper-ignore-next
+    /// The connection the signal was emitted on.
+    pub connection: &'a DBusConnection,
+    // rustdoc-stripper-ignore-next
+    /// The bus name of the sender which emitted the signal.
+    pub sender_name: &'a str,
+    // rustdoc-stripper-ignore-next
+    /// The path of the object on `sender` the signal was emitted from.
+    pub object_path: &'a str,
+    // rustdoc-stripper-ignore-next
+    /// The interface the signal belongs to.
+    pub interface_name: &'a str,
+    // rustdoc-stripper-ignore-next
+    /// The name of the emitted signal.
+    pub signal_name: &'a str,
+    // rustdoc-stripper-ignore-next
+    /// Parameters the signal was emitted with.
+    pub parameters: &'a glib::Variant,
+}
+
 // rustdoc-stripper-ignore-next
 /// Build a registered DBus object, by handling different parts of DBus.
 #[must_use = "The builder must be built to be used"]
@@ -515,9 +539,7 @@ impl DBusConnection {
     /// To avoid reference cycles you may wish to downgrade the returned
     /// subscription to a weak one with [`SignalSubscription::downgrade`].
     #[must_use]
-    pub fn subscribe_to_signal<
-        P: Fn(&DBusConnection, &str, &str, &str, &str, &glib::Variant) + 'static,
-    >(
+    pub fn subscribe_to_signal<P: Fn(DBusSignalRef) + 'static>(
         &self,
         sender: Option<&str>,
         interface_name: Option<&str>,
@@ -535,7 +557,16 @@ impl DBusConnection {
             object_path,
             arg0,
             flags,
-            callback,
+            move |connection, sender_name, object_path, interface_name, signal_name, parameters| {
+                callback(DBusSignalRef {
+                    connection,
+                    sender_name,
+                    object_path,
+                    interface_name,
+                    signal_name,
+                    parameters,
+                });
+            },
         );
         SignalSubscription(self.clone(), Some(id))
     }

gio/src/lib.rs

@@ -32,8 +32,8 @@ mod dbus;
 pub use self::dbus::*;
 mod dbus_connection;
 pub use self::dbus_connection::{
-    ActionGroupExportId, FilterId, MenuModelExportId, RegistrationBuilder, RegistrationId,
-    SignalSubscription, SignalSubscriptionId, WatcherId, WeakSignalSubscription,
+    ActionGroupExportId, DBusSignalRef, FilterId, MenuModelExportId, RegistrationBuilder,
+    RegistrationId, SignalSubscription, SignalSubscriptionId, WatcherId, WeakSignalSubscription,
 };
 mod dbus_message;
 mod dbus_method_invocation;