[MPlayer-dev-eng] Colors on mplayer 1.0 SPARC/Solaris

[Michael Niedermayer]

Hi

On Thursday 22 January 2004 11:00, Alex Beregszaszi wrote:
> Hi,
>
> > > > > Hmm... I recon - is what should be removed and + is what should
> > > > > be inserted instead, right? Anyway, what should be removed was
> > > > > never there, but what should be inserted was, so I tried
> > > > > switching the two(removed the two + lines and inserted the
> > > > > minus) and now the colors look correctly.
> > > >
> > > > argh :(
> > >
> > > pre3 never contained my change, and the report says that the change
> > > has sense, now it's working
> >
> > yes, on bigendian, but little endian is broken, just try -vo x11 -zoom
> > -xy 2 and look at the last 2 lines, there red and blue are exchanged
> > since your change
>
> Imho it's broken elsewhere if only the 2 lines are bad..
no, its entirely broken on little endian, its just the mmx code which doesnt 
use the c yuv->rgb tables

-- 
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