[Spice-devel] [spice-gtk v2] gstaudio: check before setting mute/volume properties

[Frediano Ziglio]

> 
> From: Victor Toso <me at victortoso.com>
> 
> Otherwise we can get warnings such as:
> 
>     GLib-GObject-WARNING **: 16:10:00.857:
>     g_object_set_is_valid_property: object class 'GstAutoAudioSink' has no
>     property named 'volume'
> 
> Signed-off-by: Victor Toso <victortoso at redhat.com>

The patch remove the warning... but who control the volume?
Is it just constant at the maximum?
Maybe we should detect and add a "volume" element to the
pipeline?

Does GstAutoAudioSink forward the volume settings to the chosen
element (if the property is available) ?

> ---
>  src/spice-gstaudio.c | 32 +++++++++++++++++++++++---------
>  1 file changed, 23 insertions(+), 9 deletions(-)
> 
> diff --git a/src/spice-gstaudio.c b/src/spice-gstaudio.c
> index 98c1f4f..51ff028 100644
> --- a/src/spice-gstaudio.c
> +++ b/src/spice-gstaudio.c
> @@ -368,10 +368,13 @@ static void playback_volume_changed(GObject *object,
> GParamSpec *pspec, gpointer
>      if (!e)
>          e = g_object_ref(p->playback.sink);
>  
> -    if (GST_IS_STREAM_VOLUME(e))
> +    if (GST_IS_STREAM_VOLUME(e)) {
>          gst_stream_volume_set_volume(GST_STREAM_VOLUME(e),
>          GST_STREAM_VOLUME_FORMAT_CUBIC, vol);
> -    else
> +    } else if (g_object_class_find_property(G_OBJECT_GET_CLASS (e),
> "volume") != NULL) {
>          g_object_set(e, "volume", vol, NULL);
> +    } else {
> +        g_warning("playback: ignoring volume change on %s",
> gst_element_get_name(e));
> +    }
>  
>      g_object_unref(e);
>  }
> @@ -394,8 +397,13 @@ static void playback_mute_changed(GObject *object,
> GParamSpec *pspec, gpointer d
>      if (!e)
>          e = g_object_ref(p->playback.sink);
>  
> -    if (GST_IS_STREAM_VOLUME(e))
> +    if (GST_IS_STREAM_VOLUME(e)) {
>          gst_stream_volume_set_mute(GST_STREAM_VOLUME(e), mute);
> +    } else if (g_object_class_find_property(G_OBJECT_GET_CLASS (e), "mute")
> != NULL) {
> +        g_object_set(e, "mute", mute, NULL);
> +    } else {
> +        g_warning("playback: ignoring mute change on %s",
> gst_element_get_name(e));
> +    }
>  
>      g_object_unref(e);
>  }
> @@ -427,10 +435,13 @@ static void record_volume_changed(GObject *object,
> GParamSpec *pspec, gpointer d
>      if (!e)
>          e = g_object_ref(p->record.src);
>  
> -    if (GST_IS_STREAM_VOLUME(e))
> +    if (GST_IS_STREAM_VOLUME(e)) {
>          gst_stream_volume_set_volume(GST_STREAM_VOLUME(e),
>          GST_STREAM_VOLUME_FORMAT_CUBIC, vol);
> -    else
> -        g_warning("gst lacks volume capabilities on src");
> +    } else if (g_object_class_find_property(G_OBJECT_GET_CLASS (e),
> "volume") != NULL) {
> +        g_object_set(e, "volume", vol, NULL);
> +    } else {
> +        g_warning("record: ignoring volume change on %s",
> gst_element_get_name(e));
> +    }
>  
>      g_object_unref(e);
>  }
> @@ -453,10 +464,13 @@ static void record_mute_changed(GObject *object,
> GParamSpec *pspec, gpointer dat
>      if (!e)
>          e = g_object_ref(p->record.src);
>  
> -    if (GST_IS_STREAM_VOLUME (e))
> +    if (GST_IS_STREAM_VOLUME (e)) {
>          gst_stream_volume_set_mute(GST_STREAM_VOLUME(e), mute);
> -    else
> -        g_warning("gst lacks mute capabilities on src: %d", mute);
> +    } else if (g_object_class_find_property(G_OBJECT_GET_CLASS (e), "mute")
> != NULL) {
> +        g_object_set(e, "mute", mute, NULL);
> +    } else {
> +        g_warning("record: ignoring mute change on %s",
> gst_element_get_name(e));
> +    }
>  
>      g_object_unref(e);
>  }