[FFmpeg-devel] [PATCH] Make sure AVFormatContext->start_time is initialized after the first patch

Martin Storsjö martin
Thu Jun 3 22:59:26 CEST 2010


On Thu, 3 Jun 2010, Howard Chu wrote:

> Martin Storsj? wrote:
> > On Wed, 2 Jun 2010, Martin Storsj? wrote:
> > 
> > > On Wed, 2 Jun 2010, Baptiste Coudurier wrote:
> > > 
> > > > 
> > > > So, it should be ok, and I think I agree that c->first_pts could be
> > > > always
> > > > used.
> > > 
> > > Ok, so like in the attached, then.
> > > 
> > > Howard, can you test run this and see if it seems to work fine for you,
> > > too? I'll apply it if you don't find any issues with it.
> > 
> > Hrm, here's the attachment, too...
> 
> Works for me.

Applied this one then.

> The patch I've attached here isn't necessary for your patch to work, but I
> still have a feeling it's more correct. Why seek if you don't need to?

Baptiste, any comments on this? Seems sensible to me.

// Martin



More information about the ffmpeg-devel mailing list