On Fri, Mar 13, 2015 at 05:16:53PM +0100, Stefano Sabatini wrote:
> ---
>  libavfilter/vf_eq.c | 56 
> ++++++++++++++---------------------------------------
>  1 file changed, 15 insertions(+), 41 deletions(-)

i would be more in favor of a function than a macro but LGTM either
way, macros are harder to debug and all kind of line number based
outputs are basically useless with multiline macros, be that
static analyzers of dynamic ...



set_param(cmd, "gamma_g", &eq->gamma_g_pexp, args, ctx, eq, &ret);
set_param(cmd, "gamma_r", &eq->gamma_r_pexp, args, ctx, eq, &ret);
if (ret < 0)


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to