[vlc-devel] [PATCH] hqdn3d video filter, ported from mplayer

[Jean-Baptiste Kempf]

On Wed, Nov 23, 2011 at 07:40:10AM +0100, Rémi Denis-Courmont wrote :
> On Tue, 22 Nov 2011 18:48:35 -0500, Rafaël Carré <funman at videolan.org>
> wrote:
> >> +    deNoise(src->p[0].p_pixels, dst->p[0].p_pixels,
> >> +            cfg->Line, &cfg->Frame[0], sys->w[0], sys->h[0],
> >> +            src->p[0].i_pitch, dst->p[0].i_pitch,
> >> +            cfg->Coefs[0],
> >> +            cfg->Coefs[0],
> >> +            cfg->Coefs[1]);
> >> +    deNoise(src->p[1].p_pixels, dst->p[1].p_pixels,
> >> +            cfg->Line, &cfg->Frame[1], sys->w[1], sys->h[1],
> >> +            src->p[1].i_pitch, dst->p[1].i_pitch,
> >> +            cfg->Coefs[2],
> >> +            cfg->Coefs[2],
> >> +            cfg->Coefs[3]);
> >> +    deNoise(src->p[2].p_pixels, dst->p[2].p_pixels,
> >> +            cfg->Line, &cfg->Frame[2], sys->w[2], sys->h[2],
> >> +            src->p[2].i_pitch, dst->p[2].i_pitch,
> >> +            cfg->Coefs[2],
> >> +            cfg->Coefs[2],
> >> +            cfg->Coefs[3]);
> > 
> > can this be a for loop?
> 
> Indeed. And does it really work with the same Line for each plane? That
> seems very suspicious.

How do you do a loop when it isn't the same Coefs in each line?


-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device