[Bug 640859] basesink incorrectly categorizes timestamp jitter as drift

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=640859
  GStreamer | gst-plugins-base | git

--- Comment #33 from Felipe Contreras <felipe.contreras at gmail.com> 2011-05-01 21:13:55 UTC ---
(In reply to comment #32)
> We (Tandberg/Cisco) recently changed this value from 125ms to 225ms based on
> simulations of real-life data from users experiencing glitches in their audio.

Did you try my patch in attachment #179691?

> However, our use case is strictly live streaming which is much more
> timestamp-jitter-prone.

Yeah, but jitter should not cause glitches based on drift-tolerance; that's for
drift. Which is precisely my point.

> As long as this value is configurable, I find the
> discussion of a default-value a bit meaningless?

The point is that you shouldn't need to configure it based the jitter. In fact,
no configuration value should cause such horrible glitches as they happen right
now. My patch improves that.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.