[FFmpeg-devel] [PATCH 1/2] lavfi: rename decimate to mpdecimate.

[Clément Bœsch]

On Sun, Mar 24, 2013 at 11:53:56AM +0100, Stefano Sabatini wrote:
> On date Sunday 2013-03-24 07:40:46 +0100, Clément Bœsch encoded:
> > The next commit will introduce a proper decimation filter to be used
> > along with the field matching filter. To avoid confusion with this
> > filter which has currently a very limited usage (and will not work
> > properly with the fieldmatching filter), the new decimation filter will
> > take the decimate name, and this filter is renamed to mpdecimate.
> 
> No strong opinion on this, but this is technically a compatibility
> break (even if I'm not sure there are actually people using decimate,
> but who can tell?). Thus a possible solution would be to assign a new
> name to the new decimate filter (and maybe create an alias "mpdecimate"
> for this, and drop it at the next bump).
> 

Sounds quite overkill: if some people are using current decimate,
switching to the new filter should be mostly transparent, and likely to
get better results.

> > 
> > TODO: bump minor
> > ---
> >  configure                   |   2 +-
> >  doc/filters.texi            |   2 +-
> >  libavfilter/Makefile        |   2 +-
> >  libavfilter/allfilters.c    |   2 +-
> >  libavfilter/vf_decimate.c   | 259 --------------------------------------------
> >  libavfilter/vf_mpdecimate.c | 259 ++++++++++++++++++++++++++++++++++++++++++++
> >  6 files changed, 263 insertions(+), 263 deletions(-)
> >  delete mode 100644 libavfilter/vf_decimate.c
> >  create mode 100644 libavfilter/vf_mpdecimate.c
> > 
> > diff --git a/configure b/configure
> > index 5565c9e..6a35185 100755
> > --- a/configure
> > +++ b/configure
> > @@ -2083,7 +2083,7 @@ blackframe_filter_deps="gpl"
> >  boxblur_filter_deps="gpl"
> >  colormatrix_filter_deps="gpl"
> >  cropdetect_filter_deps="gpl"
> > -decimate_filter_deps="gpl avcodec"
> > +mpdecimate_filter_deps="gpl avcodec"
> >  delogo_filter_deps="gpl"
> >  deshake_filter_deps="avcodec"
> >  drawtext_filter_deps="libfreetype"
> > diff --git a/doc/filters.texi b/doc/filters.texi
> > index 74a682a..a30f030 100644
> 
> Order.
> 

Will move.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130406/b46ad64a/attachment.asc>