wireplumber: improve property handling

Author: kotontrion

lib/wireplumber/src/device.c

@@ -517,13 +517,19 @@ static void astal_wp_device_params_changed(AstalWpDevice *self, const gchar *pro
     g_object_thaw_notify(G_OBJECT(self));
 }
 
+static void astal_wp_device_pw_properties_changed(AstalWpDevice *self) {
+    astal_wp_device_params_changed(self, "Props");
+}
+
 void astal_wp_device_constructed(GObject *object) {
     AstalWpDevice *self = ASTAL_WP_DEVICE(object);
     AstalWpDevicePrivate *priv = astal_wp_device_get_instance_private(self);
 
     g_signal_connect_swapped(priv->device, "params-changed",
                              G_CALLBACK(astal_wp_device_params_changed), self);
 
+    g_signal_connect_swapped(priv->device, "notify::properties",
+                             G_CALLBACK(astal_wp_device_pw_properties_changed), self);
     astal_wp_device_params_changed(self, "Props");
     astal_wp_device_params_changed(self, "EnumProfile");
     astal_wp_device_params_changed(self, "Profile");

lib/wireplumber/src/node.c

@@ -20,6 +20,7 @@ typedef struct {
 
     gulong mixer_signal_handler_id;
     gulong params_signal_handler_id;
+    gulong properties_signal_handler_id;
     gulong state_change_handler_id;
 
     guint id;
@@ -70,6 +71,8 @@ static GParamSpec *astal_wp_node_properties[ASTAL_WP_NODE_N_PROPERTIES] = {
     NULL,
 };
 
+static void astal_wp_node_pw_properties_changed(AstalWpNode *self);
+
 void astal_wp_node_update_volume(AstalWpNode *self) {
     AstalWpNodePrivate *priv = astal_wp_node_get_instance_private(self);
 
@@ -436,12 +439,16 @@ void astal_wp_node_set_node(AstalWpNode *self, WpNode *node) {
     if (node != NULL && WP_IS_NODE(node)) {
         if (priv->node != NULL) {
             g_signal_handler_disconnect(priv->node, priv->params_signal_handler_id);
+            g_signal_handler_disconnect(priv->node, priv->properties_signal_handler_id);
             g_signal_handler_disconnect(priv->node, priv->state_change_handler_id);
         }
         g_clear_object(&priv->node);
         priv->node = g_object_ref(node);
         priv->params_signal_handler_id = g_signal_connect_swapped(
             priv->node, "params-changed", G_CALLBACK(astal_wp_node_params_changed), self);
+        priv->properties_signal_handler_id =
+            g_signal_connect_swapped(priv->node, "notify::properties",
+                                     G_CALLBACK(astal_wp_node_pw_properties_changed), self);
         priv->state_change_handler_id = g_signal_connect_swapped(
             priv->node, "state-changed", G_CALLBACK(astal_wp_node_state_changed), self);
     }
@@ -670,6 +677,10 @@ void astal_wp_node_params_changed(AstalWpNode *self, const gchar *id) {
     (*klass->params_changed)(self, id);
 }
 
+static void astal_wp_node_pw_properties_changed(AstalWpNode *self) {
+    astal_wp_node_params_changed(self, "Props");
+}
+
 static void astal_wp_node_init(AstalWpNode *self) {
     AstalWpNodePrivate *priv = astal_wp_node_get_instance_private(self);
     priv->node = NULL;