[FFmpeg-devel] [PATCH] stop av_find_stream_info after reading a certain amount of frames

[Baptiste Coudurier]

Baptiste Coudurier wrote:
> Marc Mason wrote:
>> Baptiste Coudurier wrote:
>>
>>> Index: libavformat/utils.c
>>> ===================================================================
>>> --- libavformat/utils.c	(revision 17505)
>>> +++ libavformat/utils.c	(working copy)
>>> @@ -2019,7 +2019,7 @@
>>>  
>>>      count = 0;
>>>      read_size = 0;
>>> -    for(;;) {
>>> +    for(; count < 200; ) {
>> Just wondering. Why not "while (count < 200)" ?
> 
> Yes, why not.
> 

Michael, can you please comment and propose a better solution if you see
one.

It seems EAGAIN thing broke FFserver, since there is an infinite loop in
av_find_stream_info.

Many users are complaining now, since in fact, people actually _use_
FFserver.

I know the design is not elegant nor efficient, but it was working, and
I think it would be better to have it working in the release.

Thanks.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org