[FFmpeg-devel] [PATCH] Metadata

[Baptiste Coudurier]

Baptiste Coudurier wrote:
> 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 ?
> 

After rethinking a bit about it, it would be more like:
"AVMetadataTag **attributes" field, to contain as key/value(lang/eng)

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