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

[Michael Niedermayer]

Hi

On Wed, May 30, 2012 at 02:17:35PM -0400, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
[...]
> @@ -1925,6 +1926,11 @@ const struct AVCodecTag *avformat_get_riff_audio_tags(void);
>   */
>  
>  /**
> + * @return the table mapping Matroska Codec IDs to libavcodec CodecIDs.
> + */
> +const struct CodecTags *avformat_get_mkv_codec_tags(void);

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

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

What does censorship reveal? It reveals fear. -- Julian Assange
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120530/46f0f6bc/attachment.asc>