[gstreamer-bugs] [Bug 640859] basesink default drift-tolerance causes glitches on some clips

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #8 from Mark Nauwelaerts <mnauw at users.sourceforge.net> 2011-01-30 15:42:18 UTC ---
(In reply to comment #4)
> So, if virtually every player out there plays this clip (at least all the ones
> I've tried, and I've tried a lot), that means they all are wrong? GStreamer is
> the only one right? I don't think many people out there would agree with that.

Have you considered the possibility that it's not about being right or wrong,
and that both in fact may be "right", albeit approaching matters with a
different focus, use-cases etc, and that in GStreamer's case it is at least
configurable.

In particular, this clip (or any one with broken [*] input timestamps), along
with the fact that tolerance may have been 500ms at one point in time only
suggests that one (= application) might consider increasing the tolerance to
100ms or 500ms or for that matter "infinity" if it really wants to support
playback of broken _audio-only_ clips.

Other considerations and available facts, such as all other non-broken stuff
playing just fine with 40ms (= default) and a/v lip-sync sensitive to (afaik)
100ms variations indicate the present default is fine.

In summary, the default, which by definition should Just Work (safely) for
majority (-> not necessarily all) of use-cases and inputs is best to stay as it
is, but it could be considered to have playbin2 tweak the drift-tolerance in
some circumstances (e.g. single audio stream playback) (rather than relaying
this to application using it, which can already configure so at present if
desired).


[*] definition of broken: providing an imperfect stream of timestamps, with
imperfection in order of magnitude way beyond (single sample) rounding errors.

-- 
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.