[FFmpeg-devel] [PATCH] ogg: Fix seek to zero and packet pts after read through.

[Michael Niedermayer]

On Fri, May 04, 2012 at 03:42:00PM -0700, Dale Curtis wrote:
> On Fri, May 4, 2012 at 3:11 PM, Michael Niedermayer <michaelni at gmx.at>wrote:
> 
> >
> > i dont see a difference between
> > seek-test sync1.ogg -frames 20  | head
> > and
> > seek-test sync1.ogg -frames 20  -seekback 0| head
> >
> > ogg-ts sync1.ogg
> > also shows no errors
> >
> > so maybe my last few commits fixed that too or iam testing the wrong
> > thing.
> > Can you confirm if this issue is still reproduceable
> >
> >
> Awesome! Looks fixed for sync1.ogg, however libre.ogg is still messed up
> (maybe a bad test case?):
> 
> $ libavformat/seek-test /var/www/libre.ogg -frames 10  2>/dev/null | head
> -n 10
> ret: 0         st: 0 flags:0 dts: 1796.120000 pts: 1796.120000 pos:   6183
> size:  3964
> ret: 0         st: 1 flags:1 dts: 1796.179592 pts: 1796.179592 pos:  10190
> size:    91
> ret: 0         st: 0 flags:1 dts: 1796.160000 pts: 1796.160000 pos:  10309
> size: 16266
> ret: 0         st: 1 flags:1 dts: 1796.179592 pts: 1796.179592 pos:  26666

fixed the duplicate timestamp


[...]

> $ libavformat/seek-test /var/www/libre.ogg -frames 10 -seekback 0
> 2>/dev/null | head -n 10
> ret: 0         st: 0 flags:1 dts: 1796.160000 pts: 1796.160000 pos:  10309
> size: 16266

from a quick look it seems the first frame is not a keyframe
thus the seek seeks to the second frame.

thanks for the detailed bug reports!

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120505/4da0baa3/attachment.asc>