[FFmpeg-devel] [PATCH] add yuv444p support to mcdeint

[Michael Niedermayer]

Hi Ethan

On Sat, Jun 07, 2025 at 12:15:06PM -0500, Ethan Halsall wrote:
> Hi Michael,
[...]
> @@ -201,8 +203,14 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *inpic)
>  
>      for (i = 0; i < 3; i++) {
>          int is_chroma = !!i;
> -        int w = AV_CEIL_RSHIFT(inlink->w, is_chroma);
> -        int h = AV_CEIL_RSHIFT(inlink->h, is_chroma);
> +        int w, h;
> +        if (is_chroma) {
> +            w = inlink->w >> pix_fmt_desc->log2_chroma_w;
> +            h = inlink->h >> pix_fmt_desc->log2_chroma_h;

why is this not using AV_CEIL_RSHIFT() ?

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

What is money laundering? Its paying someone and not telling the government.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250608/0030af2c/attachment.sig>