[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gstreamer-bugs
Subject:    [Bug 700537] theoradec drops frames because it needs a keyframe after a seek (bug in oggdemux from w
From:       "GStreamer" (bugzilla.gnome.org) <bugzilla () gnome ! org>
Date:       2013-06-30 16:30:31
Message-ID: 20130630163031.568E0EE663 () bugzilla-web ! gnome ! org
[Download RAW message or body]

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

--- Comment #11 from Sebastian Dröge <slomo@circular-chaos.org> 2013-06-30 16:30:23 UTC ---
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 ;)

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?

-- 
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.
_______________________________________________
gstreamer-bugs mailing list
gstreamer-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/gstreamer-bugs

[prev in list] [next in list] [prev in thread] [next in thread]