[Bug 736397] flvdemux: Per-stream resyncing behavior causes A/V-desyncs

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736397
  GStreamer | gst-plugins-good | 1.4.1

Vincent Penquerc'h <vincent.penquerch> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vincent.penquerch at collabora
                   |                            |.co.uk

--- Comment #8 from Vincent Penquerc'h <vincent.penquerch at collabora.co.uk> 2014-11-19 15:56:47 UTC ---
I think the reasoning for not resyncing does make sense, but it would also make
sense if the discont was negative.

I'm also not convinced the original file that this fixed was made for was not
buggy in the first place.

I guess it boils down to "does the flv format allow discontinuities in
timestamps". If yes, then resyncing makes more sense, and if not, then keeping
timestamps as they are makes more sense.In the latter case, if we still find a
discont (so we have a buggy file), then resyncing anyway seems like not a bad
option.

Rereading the original bug, it looks like the file was originally buggy on the
server, then replaced with one with correct timestamps, so it makes sense to
apply this new patch.

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