[FFmpeg-devel] [PATCH] attachments support in matroska demuxer

[Michael Niedermayer]

On Sun, Jan 20, 2008 at 04:26:36PM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Sun, Jan 20, 2008 at 10:10:50AM +0100, Reimar D?ffinger wrote:
> > Hello,
> > On Sun, Jan 20, 2008 at 03:42:09AM +0100, Michael Niedermayer wrote:
> > [...]
> > > > > iam against some undocumented char *type
> > > > > please document precissely what it repressents and how it differs from
> > > > > codec_tag, stream_codec_tag and codec_id
> > > > > or even better get rid of it and use codec_tag or explain why the type
> > > > > here should be special cased relative to these funny codec id strings in
> > > > > matroska
> > > > 
> > > > Currently, *type stores the standard mime type of the attachment.
> > > 
> > > so it idetifies what the attachment is ...
> > > thats what codec_tag does alraedy ...
> > 
> > Sorry to be so flameish, but no, maybe codec_id does that, but codec_tag
> > certainly does not. E.g. for mp4 files all kind of audio has mp4a as
> > codec_tag and some other things like that.
> 
> the mp4 demuxer is buggy, it intentionally violates the API as baptiste doesnt
> agree with the API. And choose to set codec_tag to something else than what
> it is supposed to be
> (not that mpeg-ps/ts and matroska would set codec_tag correctly ...)

and if mpeg-ps where setting it, mplayer could use it do identify hddvd/h264
even if our mpeg-ps maintainer chooses to follow the mpeg-1/2 standard of
mapping it to CODEC_ID_MPEGVIDEO

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is not what we do, but why we do it that matters.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080120/48cf7b02/attachment.pgp>