[MPlayer-dev-eng] [patch] libvorbis support matrsoka style headers

Nico Sabbi nsabbi at email.it
Sat May 6 19:08:09 CEST 2006


Balatoni Denes wrote:

>Hi!
>
>szombat 06 május 2006 17.55-kor Nico Sabbi ezeket a bolcs gondolatokat 
>fogalmazta meg:
>  
>
>>has it changed?
>>I remember that I changed this file to make it accept only the layout of
>>extradata
>>used at the time by matroska (and I had to change all other demuxers
>>accordingly)
>>    
>>
>
>It didn't change, but libvorbis doesn't accept the two different kinds of 
>header format that ffvorbis does. I thought the new one was the one used by 
>matroska, but anyhow it is used by
>http://www.mplayerhq.hu/MPlayer/samples/MPEG-4/vorbis-in-mp4/mi2_vorbis51.mp4
>
>bye
>Denes
>
>  
>

IMO the decoder should accept only 1 format, and all demuxers should 
adapt to it
(that is what I did last year).
I would consider more appropriate patching demux_m[ok]v accordingly.

    Nico




More information about the MPlayer-dev-eng mailing list