[Mplayer-cvslog] CVS: main/libmpcodecs vf_lavcdeint.c,1.5,1.6

Arpi arpi at thot.banki.hu
Sat Jan 4 23:47:07 CET 2003


Hi,

> Found another 10l :-), but the filter is still broken :-(

>    /* The deinterlacer will fail if this is false */
> -  if ((width & 1) != 0 || (height & 3) != 0)
> +  if ((width & 3) != 0 || (height & 3) != 0)
>      return 0;

why is it a bug? imho it was right before you changed it.

why?  yv12 pixels are 2x2 blocks (4 Y, 1 U, 1 V), but since it
operates on pairs of lines (deinterlaces) it requires 2x4 pixel blocks.
so, width&1 and heigt&3 checks were correct.

ok to reverse? :)


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu


More information about the MPlayer-cvslog mailing list