[FFmpeg-devel] [PATCH] new audio filter and misc improvements

[Michael Niedermayer]

On Thu, Aug 10, 2023 at 12:10:49PM +0200, Paul B Mahol wrote:
> On Thu, Aug 10, 2023 at 11:48 AM Michael Niedermayer <michael at niedermayer.cc>
> wrote:
> 
> > On Wed, Aug 09, 2023 at 11:36:16PM +0200, Paul B Mahol wrote:
> > > Patches attached.
> >
> > >  af_adeclick.c |    7 ++++---
> > >  1 file changed, 4 insertions(+), 3 deletions(-)
> > > b19bd0c3f05d5ebce92f7500dab61bd85d9b3661
> > 0003-avfilter-af_adeclick-do-not-output-pointless-message.patch
> > > From 2d8c330f543397642fa1afe0a01a67155008d4e1 Mon Sep 17 00:00:00 2001
> > > From: Paul B Mahol <onemda at gmail.com>
> > > Date: Wed, 9 Aug 2023 21:53:04 +0200
> > > Subject: [PATCH 3/3] avfilter/af_adeclick: do not output pointless
> > message
> > >
> > > Signed-off-by: Paul B Mahol <onemda at gmail.com>
> > > ---
> > >  libavfilter/af_adeclick.c | 7 ++++---
> > >  1 file changed, 4 insertions(+), 3 deletions(-)
> > [...]
> >
> > >  af_adeclick.c |  136
> > ++++++++++++++++++++++++++++------------------------------
> > >  1 file changed, 67 insertions(+), 69 deletions(-)
> > > cfc7bc440f2f408f8570501f3afb2de235860221
> > 0002-avfilter-af_adeclick-refactor-and-cleanup.patch
> > > From 0a6de9b1482ece4402b1c7274501b43e43f5f56a Mon Sep 17 00:00:00 2001
> > > From: Paul B Mahol <onemda at gmail.com>
> > > Date: Wed, 9 Aug 2023 21:49:19 +0200
> > > Subject: [PATCH 2/3] avfilter/af_adeclick: refactor and cleanup
> > >
> > > Signed-off-by: Paul B Mahol <onemda at gmail.com>
> > > ---
> > >  libavfilter/af_adeclick.c | 136 +++++++++++++++++++-------------------
> > >  1 file changed, 67 insertions(+), 69 deletions(-)
> >
> > [...]
> >
> > >  doc/filters.texi          |   51 +++++++++++++++++++++++++
> > >  libavfilter/Makefile      |    1
> > >  libavfilter/af_adeclick.c |   91
> > +++++++++++++++++++++++++++++++++++++++++++---
> > >  libavfilter/allfilters.c  |    1
> > >  4 files changed, 140 insertions(+), 4 deletions(-)
> > > 61d03514ba9fa85d1ec308ce37037a0c07ad9a7f
> > 0001-avfilter-add-adesurge-filter.patch
> > > From af565f57f733af327edc1e1724e31a3c5f1fe44f Mon Sep 17 00:00:00 2001
> > > From: Paul B Mahol <onemda at gmail.com>
> > > Date: Wed, 9 Aug 2023 18:21:25 +0200
> > > Subject: [PATCH 1/3] avfilter: add adesurge filter
> > >
> > > Signed-off-by: Paul B Mahol <onemda at gmail.com>
> > > ---
> > >  doc/filters.texi          | 51 ++++++++++++++++++++++
> > >  libavfilter/Makefile      |  1 +
> > >  libavfilter/af_adeclick.c | 91 +++++++++++++++++++++++++++++++++++++--
> > >  libavfilter/allfilters.c  |  1 +
> > >  4 files changed, 140 insertions(+), 4 deletions(-)
> >
> > It seems you forgot adding fate tests, please make sure fate tests are
> > added
> > when this or a future variant is applied
> >
> 
> Sorry, Sir!,
> 
> The filter modules are not mention for fate testing,

doc/developer.texi:
 807 @item
 808 Consider adding a regression test for your code. All new modules
 809 should be covered by tests. That includes demuxers, muxers, decoders, encoders
 810 filters, bitstream filters, parsers. If its not possible to do that, add
 811 an explanation why to your patchset, its ok to not test if theres a reason.

so yes, filters should be tested too


> and for obvious
> reasons like usage of non-integer math.

A wide range of non integer modules are tested.

thx

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

If you fake or manipulate statistics in a paper in physics you will never
get a job again.
If you fake or manipulate statistics in a paper in medicin you will get
a job for life at the pharma industry.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20230810/81428799/attachment.sig>