[MPlayer-dev-eng] De- and reactivate yadif

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sat Nov 18 12:27:24 CET 2006


Hello,
On Sat, Nov 18, 2006 at 11:32:36AM +0100, Michael Niedermayer wrote:
> [...]
> > @@ -469,6 +472,21 @@
> >      return 0;
> >  }
> > 
> > +static int control(struct vf_instance_s* vf, int request, void* data){
> > +    switch (request){
> > +      case VFCTRL_GET_DEINTERLACE:
> > +        *(int*)data = do_deinterlace;
> > +        return CONTROL_OK;
> > +      case VFCTRL_SET_DEINTERLACE:
> > +        if (*(int*)data == -1)
> > +            do_deinterlace = !do_deinterlace;
> > +        else
> > +            do_deinterlace = *(int*)data;
> 
> set, get ok but do_deinterlace = !do_deinterlace; is redundant, considering
> that this would have to be added to every deinterlacing filter set(!get) at
> one central spot is better 

My fault, I suggested doing it like this. For some reason it seemed
nicer to me. I guess it was a misguided idea to avoid the overhead of
one extra control...

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list