[MPlayer-dev-eng] "D" for kerndeint & pp

[Paul TT]

On Tue, 21 Nov 2006 13:45:02 +0100
"Guillaume POIRIER" <poirierg at gmail.com> wrote:

> Hi,
> 
> On 11/21/06, Diego Biurrun <diego at biurrun.de> wrote:
> > On Tue, Nov 21, 2006 at 12:53:54AM +0100, Carl Eugen Hoyos wrote:
> > > Attached are patches to de- and reactivate kerndeint & pp with
> > > "D".
> >
> > Please mark patches with [PATCH] in the subject line.
> >
> > > --- libmpcodecs/vf_kerndeint.c        (Revision 21115)
> > > +++ libmpcodecs/vf_kerndeint.c        (Arbeitskopie)
> > > @@ -43,6 +43,7 @@
> > >       int     thresh;
> > >       int     sharp;
> > >       int     twoway;
> > > +        int     do_deinterlace;
> >
> > Please match the surrounding indentation.
> >
> > So how is the choice for one deinterlacer or the other made?
> 
> You jut put whatever deint filter you wanna use on your chain of
> filter, and "D" will (de)active it, provided that that filter supports
> being disabled on the fly.

should it not be better off to enable a config keyword to say what
filters (try to) enable/disable on the fly, not only deinterlacing
one???????