[MPlayer-dev-eng] [PATCH] Improve NV12/NV21 support

[Michael Niedermayer]

Hi

On Sunday 13 February 2005 02:50, Ville Syrjälä wrote:
> The attached patch tries to make NV12/NV21 support a bit more functional.
>
> My aim was just to get it working with DirectFB + Matrox BES so I'm not
> really interested in software scaling.
>
> Changes:
> - Fixed PlanarToNV12Wrapper() and made it handle NV21.
ok

> - Removed NV12 from vf_hue and vf_spp as they don't look like they can
>   actually handle it.
ok


> - Added vDest != NULL tests to swscale since it can't actually handle
>   NV12/NV21. So only luma will be scaled but at least it won't crash.

rejected, fix the code so it handles chroma or leave it as it is


>+       case IMGFMT_NV12:
>+       case IMGFMT_NV21:
>+               *h=1;
>+               *v=1;
>+               break;

ugh, add these to the already existing case for h/v=1


[...]

-- 
Michael

"nothing is evil in the beginning. Even Sauron was not so." -- Elrond