[FFmpeg-devel] [PATCH 2/2 v2] lavf: Add function for accessing Matroska Codec <-> CodecID table

[Derek Buitenhuis]

On 30/05/2012 3:54 PM, Derek Buitenhuis wrote:
> On 30/05/2012 3:47 PM, Michael Niedermayer wrote:
>> Maybe iam misunderstanding the problem this is supposed to solve but
>> why do you not set the AVCodecTag arrays in AVIn/OutputFormat ?
>>
>> if the issue is to fit the arbitrary strings in AVCodecTag one way
>> to do it is to consider the 32bit codec_tag a index into an public
>> array of strings from libavformat.
>> a char* could also be added to AVCodecTag but this may cause ABI
>> issues
>>
>> This would not need API extensions per (de)muxer
> 
> I agree that needing to do this on a per-demuxer basis is really ugly.
> 
> I think it's worth looking into changing AVCodecTag, but of course
> this would break ABI, as you said.

In addition to this, I've been informed of some other issues on IRC
(naming, struct opaqueness), so this will definitely get a rewrite.

- Derek


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 898 bytes
Desc: OpenPGP digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120530/c2830751/attachment.asc>