[Bug 700537] theoradec drops frames because it needs a keyframe after a seek (bug in oggdemux from what I gathered)

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #13 from Mathieu Duponchelle <mathieu.duponchelle at epitech.eu> 2013-07-04 19:49:48 UTC ---
(In reply to comment #12)
> Mathieu?

Sorry :)

(In reply to comment #11)
> I agree with the first part you mention, it seems that this commit can be
> reverted nowadays... for exactly that reasoning. And I don't think it was
> correct at all ;)
> 

cool.

> For the second part, your new patch, I think that will cause A/V sync issues.
> So when doing a seek we will jump to the earliest position that contains the
> seek position for all streams. This means that all but the earliest stream will
> have non-keyframe packets for some time, that should be dropped. Which happens
> because current_granule==-1 after a seek until the packet actually contains a
> new granule position (which will be the case for a keyframe).
> 
> Is there any problem with the packets that are dropped after a seek without
> your patch? Is it enough to just revert
> b41cd0428956f3ade9b428149e38be8e788556fe and then everything is fine for this
> bug here?

Unfortunately not. Without that patch, we still get the issue, also I don't
think that would cause stream sync issues afaiu, I can not prove it but from
testing I can say I didn't experience unsyncing issues, manual testing is
obviously not enough but it's a good hint nevertheless.

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