[MPlayer-dev-eng] [PATCH] fixing rgb24toyv12

Michael Niedermayer michaelni at gmx.at
Tue Aug 10 18:37:37 CEST 2004


Hi

On Tuesday 10 August 2004 15:48, Reimar Döffinger wrote:
> Hi,
>
> >> 2) I made the C version use all four pixels for chroma calculation,
> >> just like the MMX version (can be switched off by defining
> >> FAST_BGR2YV12)
> >
> > Now what did I do there? That part is completely broken.... Expect an
> > updated version.
>
> Here is the reduced version, without that part.
the width -> chromWidth change looks ok, feel free to commit if u tested it 
(no segfault, colors look ok, ...)
the height-2 -> height (and related) change is wrong, it will segfault if the 
width has the wrong value

[...]
-- 
Michael
level[i]= get_vlc(); i+=get_vlc();		(violates patent EP0266049)
median(mv[y-1][x], mv[y][x-1], mv[y+1][x+1]);	(violates patent #5,905,535)
buf[i]= qp - buf[i-1];				(violates patent #?)
for more examples, see http://mplayerhq.hu/~michael/patent.html
stop it, see http://petition.eurolinux.org & http://petition.ffii.org/eubsa/en




More information about the MPlayer-dev-eng mailing list