[Mplayer-cvslog] CVS: main/libmpcodecs ad_liba52.c,1.10,1.11

[Alex Beregszaszi]

Hi,

> > if it isnt a vlid header, and it has to read one more byte, the
> > header stuff will be swapped again, messing it all up.
> Just reread the code, and it is braindamaged imho, swipswapping all
> the times the 64bit sync tag, but only for original AC3 and not for
> Real's DNET. Also later in a52_printinfo it calls a52_syncinfo without
> byteswapping...
Ah, forget the above!

Someone f*cked up the whole dnet handling, leaving the buffer unchanged
in the demuxers and passing it away with the fourcc DNET, so all the
decoders should care about it, in case of liba52, it byteswapps only the
first 8 bytes (the syncinfo tag). Imho dnet is broken on BE systems now,
but who knows if it works on LE?

-- 
Alex Beregszaszi <alex at fsn.hu>
(MPlayer Core Developer -- http://www.mplayerhq.hu/)