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

Nicolas George nicolas.george
Sun Mar 6 11:48:22 CET 2011


Le sextidi 16 vent?se, an CCXIX, Michael Niedermayer a ?crit?:
> a 2 or 3 level abort should be used. Like unix TERM & KILL

It is an interesting solution, but is it necessary?

With the check for url_interrupt_cb at the end of the loop, the code works,
just as it has worked for a long time. It just needs a little clarification
in the documentation.

Adding this multi-level interrupt system would not solve any problem
currently known. And when a problem will actually arise, we do not know
whether this will be enough to solve it.

Designing a solution without a problem is rarely a good idea.

For now, I think it would be best to just restore the check of
url_interrupt_cb at the end of the loop and document that url_interrupt_cb
makes the protocol it has interrupted unusable.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20110306/ead69c21/attachment.pgp>



More information about the ffmpeg-cvslog mailing list