[FFmpeg-devel] [RFC][PATCH] Rudimentary support for id3v2 APIC tags.
Adrian Drzewiecki
adrian.drzewiecki at gmail.com
Sun Dec 25 20:25:28 CET 2011
On Dec 24, 2011, at 12:00 PM, Nicolas George wrote:
> Le quartidi 4 nivôse, an CCXX, Adrian Drzewiecki a écrit :
>> When parsing id3 tags, if an APIC tag is encountered, add it
>> to the tag dictionary encoded as "<mime type>\0<4 byte data len>\0<data>".
>
> I did not read the rest, but I think it would be better to have proper
> support for MIME type (or any kind of type description) and binary data
> directly in the data structures.
>
> Regards,
>
> --
> Nicolas George
Thanks. How about this then: extend AVDictionaryEntry with two fields:
AVDictionaryEntry {
char *key;
char *value;
void *data;
int size;
};
And a new function av_dict_set_data().
Thoughts?
-Adrian
More information about the ffmpeg-devel
mailing list