[MPlayer-dev-eng] [PATCH] fix realaudio in mkv

[Roberto Togni]

On Fri, 24 Mar 2006 11:42:41 +0100
Moritz Bunkus <m.bunkus at linet-services.de> wrote:

> Hey,
> 
> On Friday 24 March 2006 08:56, Diego Biurrun wrote:
> 
> > I suppose this is why I get
> >
> >   A:   inf
> >
> > on the status line?  Other than that the patch appears to work.
> 
> I noticed this as well, and this completely destroys A/V sync. Therefore
> unacceptable.
> 

The cause of it is the missing i_bps. Unfortunately the data is not
available in the type specific data block of the MDPR, but it's in the
general part of it (average bit rate).

> Why was that change to the decoder necessary in the first place? Seems
> like it created a lot of work and shifted code from the decoder to the
> demuxers... What's the benefit?
> 

Only the demuxer have all the correct info to reorder audio packet. If
a packet was missing with the old method you got a permanent desync in
the descrambling matrix. The old method also required to pass
container-related data to the decoder to perform the reordering process.

The i_bps problem is not caused by the reordering patch, it worked only
with the binary codecs (native cook does not provide any
flavor-related functions)

I see some track->fix_i_bps in mkv demuxer, how does it work exactly?
can it be used to solve this problem? Else the only solution is to
create a flavor2bps array and use it.


Ciao,
 Roberto