[Ffmpeg-devel] [PATCH] cook compatible decoder
Rich Felker
dalias
Tue Nov 8 19:04:43 CET 2005
On Tue, Nov 08, 2005 at 06:41:46PM +0100, mkhodor7 at yahoo.com wrote:
> Benjamin Larsson wrote:
>
> > The real problem is the way cook is stored in rm. They store subpackets
> > out of order. Realplayer reorders the subpackets in the demuxer. I
> > did the reordering in the codec to simplify the demuxer layer. But to be
> > able to know how to reorder I need 2 variables from the ra header. Those
> > variables are added to the extradata and passed to the decoder. Currently
> > 5 extra variables are passed, it could be reduced to 2 but it doesn't solve
> > the problem of being able to mux cook into something else without
> > cook specific code in the (de)muxer, if the codec reorders the subpackets.
> >
> > We could do it the way Realplayer does it, then we would only pass real
> > extradata and let the demuxer reorder the subpackets. This way it might
> > be possible to mux into another container without cook specific code.
> > The rm demuxer would need a rewrite for it to work though.
>
> This is a general problem with RealMedia. The byte order in the
> container is not always the same as what the codec expects.
>
> For example, with AC3 every pair of bytes is swapped. The demuxer
> swaps the bytes in rm_read_packet before sending the data to liba52.
> With G.728 there is a more complicated rearrangement of the packets
> (I don't think the current code actually works right.)
>
> Anyway, the code should be consistent in where we do this. If we're
> going to reorder the dnet data in the demuxer to output standard AC3,
> then we should reorder the 28_8 data in the demuxer to output standard
> G.728, and we should reorder the cook data in the demuxer to output
> whatever the cook codec expects.
>
> I'm not sure exactly what Matroska does with this. I believe that AC3
> is converted to standard order and given the generic A_AC3 tag.
> They don't have an ID for G728 so presumably this is stored in the
> original realnetworks format with the ID A_REAL/28_8. It may be better
> to just leave it like that, since unlike AC3, I don't think anyone
> actually uses G.728 outside of RealAudio. Someone should write a spec
> for the cook extradata in Matroska and decide what the canonical packet
> ordering is.
I don't care which the demuxer does as long as it uses a correct codec
tag (G728 if the order is corrected, 28_8 if it's not).
Rich
More information about the ffmpeg-devel
mailing list