[FFmpeg-devel] [PATCH v6 3/3] avformat/tee: Handling slave failure in tee muxer

Michael Niedermayer michael at niedermayer.cc
Thu Apr 14 02:57:27 CEST 2016

On Wed, Apr 13, 2016 at 10:15:32AM +0300, sebechlebskyjan at gmail.com wrote:
> From: Jan Sebechlebsky <sebechlebskyjan at gmail.com>
> Adds per slave option 'onfail' to the tee muxer allowing an output to
> fail,so other slave outputs can continue.
> Signed-off-by: Jan Sebechlebsky <sebechlebskyjan at gmail.com>
> ---
>  Thanks for pointing on the problem with testing enum for negative value.
>  I've fixed that, so parse_slave_failure_policy_option returns only error value
>  and sets value of enum by using pointer - I think it also looks nicer to have it this way.

thanks, no more comments from me

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160414/15b78446/attachment.sig>

More information about the ffmpeg-devel mailing list