[FFmpeg-devel] [PATCH] When PMT is found, we have found mpegts header information

[Baptiste Coudurier]

On 04/15/2010 08:34 AM, Joakim Plate wrote:
> M?ns Rullg?rd<mans<at>  mansr.com>  writes:
>>>
>>> While i agree, should we always let av_find_stream_info search the whole
>>> file to sure we find all stream??? That doesn't make sence either.
>>
>> Of course not.  The correct solution would be a design allowing
>> dynamic addition and removal of streams.  Unfortunately, FFmpeg isn't
>> designed like that.
>>
>
> Thought i'd just mention that current mpegts demuxer already stops
> searching for pmt after the first one is found... At the end of
> pmt_cb you have "mpegts_close_filter(ts, filter);", but
> av_find_stream_info keeps on going even if nothing will ever happen...

Yes, as soon as you got the PMT referred by the PAT it is closed, but 
the PAT filter is kept open and will relaunch the PMT filter if PMT pid 
changes.

> Soo, can we please accept this patch, since it's only effect is to
> reduce startup time.. :)

Does probesize reduce start time ? That's the way to go IMHO.

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org