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

[Rafaël Carré]

Le Tue, 22 Nov 2011 22:29:07 +0000,
Cheng Sun <chengsun9 at gmail.com> a écrit :

> >> +/*
> >> + * Copyright (C) 2003 Daniel Moreno <comac at comac.darktech.org>
> >> + *
> >> + * This file is part of MPlayer.
> >> + *
> >> + * MPlayer is free software; you can redistribute it and/or modify
> > 
> > Should we change this to "This program" ?
> > 
> > Also why do you keep code in the header, to reduce difference with MPlayer code?
> > 
> Yes, I don't think we should remove it, because the file is almost
> identical to the copy in MPlayer (only a few lines removed).

ok

> +    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?

> +
> +    return CopyInfoAndRelease(dst, src);
> +}

-- 
Rafaël Carré