Remove useless video filters

Most of these have very limited actual use, or are even entirely
useless. They only serve to bloat the codebase and to make life harder.
Drowning users in tons of barely useful filters isn't exactly helpful
either. Some of these filters were redundant or marked as obsolete.

The dlopen and lua (to be added soon) video filters provide ways to add
custom filters.

Detailed listing for each filter with reasons (with contributions from
divVerent and lachs0r):

1bpp:
    Replaced by "scale".
2xsai:
    Pixel art scaling algorithm, useless with lossy video.
blackframe:
    Not very useful. Apparently one use is combining it with scripts,
    that pass the
bmovl:
    Weirdly complex and insane (using FIFO commands), questionable use.
cropdetect:
    Only sort-of useful when used with scripts, and then it will be
    very fragile.
    It's probably better to use the dlopen rectangle filter, or to
    implement the common use-case in a better way.
decimate:
    Not needed/useful with modern video codecs, is an
    encoding-only filter.
denoise3d:
    "hqdn3d" is better.
detc:
    Some of the worse deteleciners.
dint:
    Useless, actually crashes. (On an assert in vf.c that is disabled
    by default in mplayer-svn.)
dvbscale:
    Not even practical, and the same effect can be achieved through
    other means.
eq:
    Worse/older version of eq2.
field:
    Limited use, available as dlopen filter.
fil:
    Quoting the manpage:
        This filter is very similar to the il filter but much faster,
        the main disadvantage is that it does not always work.
        Especially if combined with other filters it may produce
        randomly messed up images, so be happy if it works but do not
        complain if it does not for your combination of filters.
filmdint:
    Kind of redundant with pullup, and slightly worse.
fixpts:
    Never useful. (Most if not all filters have been fixed for PTS.)
framestep:
    Questionable use. For things like creating thumbnails, ffmpeg or
    --sstep should be used.
geq:
    Limited use, will be redundant with the "lua" filter.
halfpack:
    Useless, probably redundant with "scale".
harddup:
    Useless.
hue:
    Most VOs support this.
il:
    Useless.
ivtc:
    Another of the worse deteleciners.
kerndeint:
    A bad deinterlacer.
lavc:
    For DVB output devices. We removed that support.
lavcdeint:
    A bad deinterlacer, was already deprecated.
    Still available as --vf=pp=fd.
mcdeint:
    A broken deinterlacer that uses lavc internals.
ow:
    Very slow, barely any quality benefit over "hqdn3d".
palette:
    Done by "scale".
perspective:
    Files with incorrect perspective are extremely rare. About the
    only real-world use for this is keystone correction, which is
    usually done in hardware by the projector or by graphics
    drivers/compositors.
pp7:
    Another useless postprocessing filter with bad and complicated code.
    Use libpostprocess with "pp" instead.
qp:
    Useless.
remove-logo:
    Redundant with delogo, which is better and more practical.
rgbtest:
    Useless.
sab, smartblur, boxblur:
    Blur filters, redundant to "unsharp".
softskip:
    Does nothing.
spp, fspp, uspp:
    Useless postprocessing filters. "spp" needs ffmpeg internals.
    "fspp" is the optimized version of the "spp" filter (???), while
    "uspp" is the slow version (????).
    Use libpostprocess with "pp" instead.
telecine:
    Evil and useless. Available as dlopen filter for testing
    purposes.
test:
    Useless.
tfields:
    Useless, probably.
tile:
    Questionable use. Available as dlopen filter.
tinterlace:
    Evil and useless.
yuvcsp:
    Probably useless.
yvu9:
    Redundant with "scale".

Also remove the following left-over files: vd_null.c, vqf.h

1 files changed, 0 insertions, 235 deletions

diff --git a/libmpcodecs/vf_tinterlace.c b/libmpcodecs/vf_tinterlace.c
deleted file mode 100644
index 2583d431e9..0000000000
--- a/libmpcodecs/vf_tinterlace.c
+++ /dev/null
@@ -1,235 +0,0 @@
-/*
- * Copyright (C) 2003 Michael Zucchi <notzed@ximian.com>
- *
- * This file is part of MPlayer.
- *
- * MPlayer is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * MPlayer is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License along
- * with MPlayer; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
- */
-
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-
-#include "config.h"
-#include "mp_msg.h"
-
-#include "img_format.h"
-#include "mp_image.h"
-#include "vf.h"
-
-#include "libvo/fastmemcpy.h"
-
-struct vf_priv_s {
-	int mode;
-	int frame;
-	mp_image_t *dmpi;
-};
-
-static int put_image(struct vf_instance *vf, mp_image_t *mpi, double pts)
-{
-	int ret = 0;
-	mp_image_t *dmpi;
-
-	switch (vf->priv->mode) {
-	case 0:
-		dmpi = vf->priv->dmpi;
-		if (dmpi == NULL) {
-			dmpi = vf_get_image(vf->next, mpi->imgfmt,
-					    MP_IMGTYPE_STATIC, MP_IMGFLAG_ACCEPT_STRIDE |
-					    MP_IMGFLAG_PRESERVE,
-					    mpi->width, mpi->height*2);
-
-			vf->priv->dmpi = dmpi;
-
-			memcpy_pic(dmpi->planes[0], mpi->planes[0], mpi->w, mpi->h,
-				   dmpi->stride[0]*2, mpi->stride[0]);
-			if (mpi->flags & MP_IMGFLAG_PLANAR) {
-				memcpy_pic(dmpi->planes[1], mpi->planes[1],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[1]*2, mpi->stride[1]);
-				memcpy_pic(dmpi->planes[2], mpi->planes[2],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[2]*2, mpi->stride[2]);
-			}
-		} else {
-			vf->priv->dmpi = NULL;
-
-			memcpy_pic(dmpi->planes[0]+dmpi->stride[0], mpi->planes[0], mpi->w, mpi->h,
-				   dmpi->stride[0]*2, mpi->stride[0]);
-			if (mpi->flags & MP_IMGFLAG_PLANAR) {
-				memcpy_pic(dmpi->planes[1]+dmpi->stride[1], mpi->planes[1],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[1]*2, mpi->stride[1]);
-				memcpy_pic(dmpi->planes[2]+dmpi->stride[2], mpi->planes[2],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[2]*2, mpi->stride[2]);
-			}
-			ret = vf_next_put_image(vf, dmpi, pts);
-		}
-		break;
-	case 1:
-		if (vf->priv->frame & 1)
-			ret = vf_next_put_image(vf, mpi, pts);
-		break;
-	case 2:
-		if ((vf->priv->frame & 1) == 0)
-			ret = vf_next_put_image(vf, mpi, pts);
-		break;
-	case 3:
-		dmpi = vf_get_image(vf->next, mpi->imgfmt,
-				    MP_IMGTYPE_TEMP, MP_IMGFLAG_ACCEPT_STRIDE,
-				    mpi->width, mpi->height*2);
-		/* fixme, just clear alternate lines */
-		vf_mpi_clear(dmpi, 0, 0, dmpi->w, dmpi->h);
-		if ((vf->priv->frame & 1) == 0) {
-			memcpy_pic(dmpi->planes[0], mpi->planes[0], mpi->w, mpi->h,
-				   dmpi->stride[0]*2, mpi->stride[0]);
-			if (mpi->flags & MP_IMGFLAG_PLANAR) {
-				memcpy_pic(dmpi->planes[1], mpi->planes[1],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[1]*2, mpi->stride[1]);
-				memcpy_pic(dmpi->planes[2], mpi->planes[2],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[2]*2, mpi->stride[2]);
-			}
-		} else {
-			memcpy_pic(dmpi->planes[0]+dmpi->stride[0], mpi->planes[0], mpi->w, mpi->h,
-				   dmpi->stride[0]*2, mpi->stride[0]);
-			if (mpi->flags & MP_IMGFLAG_PLANAR) {
-				memcpy_pic(dmpi->planes[1]+dmpi->stride[1], mpi->planes[1],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[1]*2, mpi->stride[1]);
-				memcpy_pic(dmpi->planes[2]+dmpi->stride[2], mpi->planes[2],
-					   mpi->chroma_width, mpi->chroma_height,
-					   dmpi->stride[2]*2, mpi->stride[2]);
-			}
-		}
-		ret = vf_next_put_image(vf, dmpi, pts);
-		break;
-	case 4:
-		// Interleave even lines (only) from Frame 'i' with odd
-		// lines (only) from Frame 'i+1', halving the Frame
-		// rate and preserving image height.
-
-		dmpi = vf->priv->dmpi;
-
-		// @@ Need help:  Should I set dmpi->fields to indicate
-		// that the (new) frame will be interlaced!?  E.g. ...
-		// dmpi->fields |= MP_IMGFIELD_INTERLACED;
-		// dmpi->fields |= MP_IMGFIELD_TOP_FIRST;
-		// etc.
-
-		if (dmpi == NULL) {
-			dmpi = vf_get_image(vf->next, mpi->imgfmt,
-					    MP_IMGTYPE_STATIC, MP_IMGFLAG_ACCEPT_STRIDE |
-					    MP_IMGFLAG_PRESERVE,
-					    mpi->width, mpi->height);
-
-			vf->priv->dmpi = dmpi;
-
-			my_memcpy_pic(dmpi->planes[0], mpi->planes[0], mpi->w, mpi->h/2,
-				      dmpi->stride[0]*2, mpi->stride[0]*2);
-			if (mpi->flags & MP_IMGFLAG_PLANAR) {
-				my_memcpy_pic(dmpi->planes[1], mpi->planes[1],
-					      mpi->chroma_width, mpi->chroma_height/2,
-					      dmpi->stride[1]*2, mpi->stride[1]*2);
-				my_memcpy_pic(dmpi->planes[2], mpi->planes[2],
-					      mpi->chroma_width, mpi->chroma_height/2,
-					      dmpi->stride[2]*2, mpi->stride[2]*2);
-			}
-		} else {
-			vf->priv->dmpi = NULL;
-
-			my_memcpy_pic(dmpi->planes[0]+dmpi->stride[0],
-				      mpi->planes[0]+mpi->stride[0],
-				      mpi->w, mpi->h/2,
-				      dmpi->stride[0]*2, mpi->stride[0]*2);
-			if (mpi->flags & MP_IMGFLAG_PLANAR) {
-				my_memcpy_pic(dmpi->planes[1]+dmpi->stride[1],
-					      mpi->planes[1]+mpi->stride[1],
-					      mpi->chroma_width, mpi->chroma_height/2,
-					      dmpi->stride[1]*2, mpi->stride[1]*2);
-				my_memcpy_pic(dmpi->planes[2]+dmpi->stride[2],
-					      mpi->planes[2]+mpi->stride[2],
-					      mpi->chroma_width, mpi->chroma_height/2,
-					      dmpi->stride[2]*2, mpi->stride[2]*2);
-			}
-			ret = vf_next_put_image(vf, dmpi, pts);
-		}
-		break;
-	}
-
-	vf->priv->frame++;
-
-	return ret;
-}
-
-static int query_format(struct vf_instance *vf, unsigned int fmt)
-{
-	/* FIXME - figure out which other formats work */
-	switch (fmt) {
-	case IMGFMT_YV12:
-	case IMGFMT_IYUV:
-	case IMGFMT_I420:
-		return vf_next_query_format(vf, fmt);
-	}
-	return 0;
-}
-
-static int config(struct vf_instance *vf,
-        int width, int height, int d_width, int d_height,
-	unsigned int flags, unsigned int outfmt)
-{
-	switch (vf->priv->mode) {
-	case 0:
-	case 3:
-		return vf_next_config(vf,width,height*2,d_width,d_height*2,flags,outfmt);
-	case 1:			/* odd frames */
-	case 2:			/* even frames */
-	case 4:			/* alternate frame (height-preserving) interlacing */
-		return vf_next_config(vf,width,height,d_width,d_height,flags,outfmt);
-	}
-	return 0;
-}
-
-static void uninit(struct vf_instance *vf)
-{
-	free(vf->priv);
-}
-
-static int vf_open(vf_instance_t *vf, char *args)
-{
-	struct vf_priv_s *p;
-	vf->config = config;
-	vf->put_image = put_image;
-	vf->query_format = query_format;
-	vf->uninit = uninit;
-	vf->default_reqs = VFCAP_ACCEPT_STRIDE;
-	vf->priv = p = calloc(1, sizeof(struct vf_priv_s));
-	vf->priv->mode = 0;
-	if (args)
-	  sscanf(args, "%d", &vf->priv->mode);
-	vf->priv->frame = 0;
-	return 1;
-}
-
-const vf_info_t vf_info_tinterlace = {
-	"temporal field interlacing",
-	"tinterlace",
-	"Michael Zucchi",
-	"",
-	vf_open,
-	NULL
-};