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

Michael Niedermayer michaelni at gmx.at
Sat May 5 00:11:03 CEST 2012

On Fri, May 04, 2012 at 10:58:45AM -0700, Dale Curtis wrote:
> On Fri, May 4, 2012 at 5:52 AM, Michael Niedermayer <michaelni at gmx.at>wrote:
> > On Thu, May 03, 2012 at 08:09:21PM -0700, Dale Curtis wrote:
> > > Sorry for the delay, I've been out on vacation the past week. The pts/dts
> > > for the first frame on the first pass are 0,0. Why is it incorrect to
> > have
> > > a pts/dts of 0,0 on the second pass as well? Thanks in advance for your
> > > patience.
> >
> > 0,0 is incorrect, its wrong on the first AND second pass.
> >
> > Ive fixed it now.
> >
> Thanks Michael. I'm unable to reproduce the issues you're seeing though.
> Can you send me the test program and video you're using? I just reran my
> test case with our patch applied to ffmpeg HEAD and the timestamps are
> coming out correct:

I suspect the issue cant be reproduced with git HEAD anymore.


> Curiously using libre.ogg and seeking on the audio index w/o the patch
> yields the correct results too. Unfortunately this doesn't seem to work
> with sync1.ogg.

i dont see a difference between 
seek-test sync1.ogg -frames 20  | head
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
Can you confirm if this issue is still reproduceable 

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- 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/5e6cfcb4/attachment.asc>

More information about the ffmpeg-devel mailing list