[FFmpeg-devel] Problem with initial dts in h264

Michael Niedermayer michaelni
Wed Dec 9 13:17:48 CET 2009


On Wed, Dec 09, 2009 at 12:07:54AM +0000, Carl Eugen Hoyos wrote:
> Hi!
> 
> Michael Niedermayer <michaelni <at> gmx.at> writes:
> 
> > > --- libavformat/utils.c	(revision 20757)
> > > +++ libavformat/utils.c	(working copy)
> > > @@ -1177,7 +1177,7 @@
> > >  void av_read_frame_flush(AVFormatContext *s)
> > >  {
> > >      AVStream *st;
> > > -    int i;
> > > +    int i, j;
> > >  
> > >      flush_packet_queue(s);
> > >  
> > > @@ -1200,6 +1200,9 @@
> > >          st->cur_len = 0;
> > >  
> > >          st->probe_packets = MAX_PROBE_PACKETS;
> > > +
> > > +        for(j=0; j<MAX_REORDER_DELAY+1; j++)
> > > +            st->pts_buffer[j]= AV_NOPTS_VALUE;
> > >      }
> > >  }
> > >  
> > 
> > 2 hunks above ok (if tested), (and maybe with FF_ARRAY_ELEMS())
> 
> I believe you over-estimate the number of people on this list who are
> a) able to understand what they should test before applying this and

regression tests & timestamps after seeking h264


> b) apply other people's patches at all.
> 
> I will happily apply both hunks if you want (and I think we will find out soon
> if they do any harm), but I have no idea what to test.
> 
> Carl Eugen
> 
> PS: Could you look at the regression issue 1550 (sample in incoming/issue1550)?
> And could one of the H264 experts look at issue 1585? I suspect an important
> issue, but maybe it is just a broken sample.

ill try to look at 1550

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

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091209/59e26da6/attachment.pgp>



More information about the ffmpeg-devel mailing list