[Ffmpeg-devel] Re: Re: Re: [theora-dev] Theora encoding in FFmpeg

Michael Niedermayer michaelni
Sun Dec 31 21:17:52 CET 2006


On Sun, Dec 31, 2006 at 08:09:54PM +0000, Paul Richards wrote:
> On 31/12/06, Michael Niedermayer <michaelni at gmx.at> wrote:
> >On Sun, Dec 31, 2006 at 05:56:11PM +0000, Paul Richards wrote:
> >> And on a deeper level..
> >>
> >> Why is all this special packing of the output from theora_encode_***()
> >> into the extradata required?  I naively assumed that any packing like
> >> this would be the responsibility of the muxer...
> >
> >no of course not, just think about this for a moment, should the avi muxer
> >contain code to repackage theora, vorbis, ... headers? then what about
> >matroska, nut, asf, mov, mp4, ... ? should every muxer contain (duplicated)
> >code to pack all the codec specific xiph headers into a single packet?
> >
> >and then after you duplicated that code in every muxer, you will have to
> >add code to every corresponding demuxer to undo it otherwise you cannot
> >copy a stream between 2 containers witout reencoding ...
> >
> >simply putting the (de)packaging code into the 2 or 3 xiph codecs is much
> >easier ...
> >
> In that case shouldn't there be a function in libtheora to concatenate
> all of the "extradata" packets together in a container-insensitive
> way?

yes, and this container-insensitive way should be documented in the 
theora/vorbis/... spec
iam sure many people would be happy if this would happen ...

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20061231/0730da5a/attachment.pgp>

More information about the ffmpeg-devel mailing list