[FFmpeg-devel] [PATCH] Metadata

[Baptiste Coudurier]

Hi Michael,

Michael Niedermayer wrote:
> [...]
 >
> + * 3. A tag whichs value is translated has the ISO 639 3-letter language code
> + *    with a '-' between appended. So for example Author-ger=Michael, Author-eng=Mike
> + *    the original/default language is in the unqualified "Author"
> + *    A demuxer should set a default if it sets any translated tag.
 >
> [...]
 >
> +typedef struct {
> +    char *key;
> +    char *value;
> +}AVMetaDataTag;

Maybe it would be simpler and more extensible to have a "const char 
**attributes" field where to store language, or anything else related to 
the AVMetaDataTag entry. This would avoid parsing the '-'.

What do people think ?

[...]

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org