[FFmpeg-cvslog] In retry_transfer_wrapper, do not check url_interrupt_cb,

[Baptiste Coudurier]

Hi Martin,

On 3/3/11 11:00 PM, Martin Storsj? wrote:
> On Fri, 4 Mar 2011, Nicolas George wrote:
> 
>> Le quartidi 14 vent?se, an CCXIX, Baptiste Coudurier a ?crit :
>>> In retry_transfer_wrapper, do not check url_interrupt_cb,
>>> causes problems when writing and pressing q during encoding.
>>
>> I think you just broke url_interrupt_cb for raw TCP.
> 
> Perhaps a fix which would match the old behaviour more closely wold be to 
> move the url_interrupt_cb() check down, either into the if (EAGAIN) 
> branch, or at least to after transfer_func has been called. That way, 
> url_interrupt_cb() triggering won't make writes to local files fail, which 
> wouldn't have blocked anyway.

Can you please check ? Many users reported the problem, ffmpeg was
basically broken.

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