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

Howard Chu hyc
Thu Jun 3 11:03:45 CEST 2010


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.

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?

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: dif.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100603/b0fed38d/attachment.txt>



More information about the ffmpeg-devel mailing list