On date Tuesday 2015-03-17 12:46:32 +0100, Michael Niedermayer encoded: > On Sun, Mar 15, 2015 at 02:59:58PM +0100, Stefano Sabatini wrote: > > On date Friday 2015-03-13 20:05:35 +0100, Michael Niedermayer encoded: > > > 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) > > > > Up. > > -- > > FFmpeg = Fantastic Fundamental Miracolous Patchable Elitarian Gargoyle > > > vf_eq.c | 66 > > ++++++++++++++++++++++------------------------------------------ > > 1 file changed, 23 insertions(+), 43 deletions(-) > > 3adede32119e8d758297b2a29a92b98cb17d1010 > > 0001-lavfi-eq-factorize-code-in-process_command.patch > > From ee2157da84a30e043dfa55cf25a86ef751451bba Mon Sep 17 00:00:00 2001 > > From: Stefano Sabatini <stefa...@gmail.com> > > Date: Fri, 13 Mar 2015 16:45:08 +0100 > > Subject: [PATCH] lavfi/eq: factorize code in process_command() > > LGTM
Applied, thanks. -- FFmpeg = Faithless & Free Magnificient Portable Explosive Gadget _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
