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

[Michael Niedermayer]

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

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