[FFmpeg-cvslog] yadif: support 10bit YUV

[Jethro Walters]

On 13 Dec 2011, at 22:33, git at videolan.org (Luca Barbato) wrote:

> ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Sun Dec  4 15:56:41 2011 +0100| [d32eed5c73744c25625a9fb66c48e57f2d0b433d] | committer: Luca Barbato
> 
> yadif: support 10bit YUV
> 
> Signed-off-by: Luca Barbato <lu_zero at gentoo.org>
> 
>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d32eed5c73744c25625a9fb66c48e57f2d0b433d
> ---
> 
> libavfilter/vf_yadif.c |    7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/libavfilter/vf_yadif.c b/libavfilter/vf_yadif.c
> index 64b2561..ca2adb2 100644
> --- a/libavfilter/vf_yadif.c
> +++ b/libavfilter/vf_yadif.c
> @@ -145,7 +145,7 @@ static void filter(AVFilterContext *ctx, AVFilterBufferRef *dstpic,
>         int w = dstpic->video->w;
>         int h = dstpic->video->h;
>         int refs = yadif->cur->linesize[i];
> -        int df = (yadif->csp->comp[i].depth_minus1+1) / 8;
> +        int df = (yadif->csp->comp[i].depth_minus1 + 8) / 8;
> 
>         if (i == 1 || i == 2) {
>         /* Why is this not part of the per-plane description thing? */
> @@ -212,7 +212,7 @@ static void return_frame(AVFilterContext *ctx, int is_second)
> 
>     if (!yadif->csp)
>         yadif->csp = &av_pix_fmt_descriptors[link->format];
> -    if (yadif->csp->comp[0].depth_minus1 == 15)
> +    if (yadif->csp->comp[0].depth_minus1 / 8 == 1)

Aren't multiplications faster than divisions on pretty much all platform? / 8 should be *0.25 IMHO.

Regards,

Jethro