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.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel