Re: [PATCH spice-gtk 3/4] Fix video playback with GStreamer backend

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 08, 2012 at 12:13:28PM +0200, Marc-André Lureau wrote:
> The playback audio delay is not correctly adjusted, we should take
> min_latency, set by gst_bin_do_latency_func ().

No clue about this one, I'll have to trust you on that one, at least it
does not look wrong, ACK

> ---
>  gtk/spice-gstaudio.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/gtk/spice-gstaudio.c b/gtk/spice-gstaudio.c
> index 080169b..6dd250e 100644
> --- a/gtk/spice-gstaudio.c
> +++ b/gtk/spice-gstaudio.c
> @@ -279,7 +279,10 @@ static gboolean update_mmtime_timeout_cb(gpointer data)
>          gboolean live;
>          GstClockTime minlat, maxlat;
>          gst_query_parse_latency(q, &live, &minlat, &maxlat);
> -        spice_playback_channel_set_delay(SPICE_PLAYBACK_CHANNEL(p->pchannel), GST_TIME_AS_MSECONDS(maxlat));
> +        SPICE_DEBUG("got min latency %" GST_TIME_FORMAT ", max latency %"
> +                    GST_TIME_FORMAT ", live %d", GST_TIME_ARGS (minlat),
> +                    GST_TIME_ARGS (maxlat), live);
> +        spice_playback_channel_set_delay(SPICE_PLAYBACK_CHANNEL(p->pchannel), GST_TIME_AS_MSECONDS(minlat));
>      }
>      gst_query_unref (q);
>  
> -- 
> 1.7.10.2
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: pgpBmVNlKi12a.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]