On Wed, Apr 13, 2016 at 10:15:32AM +0300, sebechlebsky...@gmail.com wrote: > From: Jan Sebechlebsky <sebechlebsky...@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 <sebechlebsky...@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.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
