[FFmpeg-devel] [PATCH] avfilter/signalstats: add NULL_IF_CONFIG_SMALL

Alexander Strasser eclipse7 at gmx.net
Thu Jan 23 23:01:24 EET 2025


Hi Abdulrahman!

I fixed up and tested your patch locally and your change looks fine :)

Will include when it when I push my next batch of changes soon'ish.
Depending on how it goes it will take a couple of days from now.

For your information I will outline the broken parts inline below:

On 2025-01-22 20:31 +0200, Abdulrahman Saber wrote:
> On Tue, Jan 14, 2025 at 2:02 PM Abdulrahman Saber <abdsaber000 at gmail.com> wrote:
> >
> > Signed-off-by: abdo <abdsaber000 at gmail.com>
> > ---
> >   libavfilter/vf_signalstats.c | 3 ++-
> >   1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/libavfilter/vf_signalstats.c b/libavfilter/vf_signalstats.c
> > index b1451cc465..9b6af8becd 100644
> > --- a/libavfilter/vf_signalstats.c
> > +++ b/libavfilter/vf_signalstats.c
> > @@ -20,6 +20,7 @@

> >    * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
> > 02110-1301 USA

The above line was broken into 2 in your mail.

> >    */
> >   +#include "libavutil/internal.h"
> >   #include "libavutil/intreadwrite.h"
> >   #include "libavutil/mem.h"
> >   #include "libavutil/opt.h"
> > @@ -829,7 +830,7 @@ static const AVFilterPad signalstats_outputs[] = {
> >    const FFFilter ff_vf_signalstats = {
> >       .p.name        = "signalstats",
> > -    .p.description = "Generate statistics from video analysis.",

> > +    .p.description = NULL_IF_CONFIG_SMALL("Generate statistics from
> > video analysis."),

Same here.

> >       .p.priv_class  = &signalstats_class,
> >       .p.flags       = AVFILTER_FLAG_SLICE_THREADS,
> >       .init          = init,
> > --
> > 2.25.1
> >
>
> Ping

Seems like your mail client/editor does it (automatically?) to avoid
overly long lines, which might be fine for email texts but isn't in
patches.

Hope you can find the setting for easier contribution in the future.
Another approach for single patches or small series is to attach them.
Usually mailers don't mess with attachments.


Best regards,
  Alexander


More information about the ffmpeg-devel mailing list