[FFmpeg-devel] [PATCH] metadata API is now ready to be part of public API

[Baptiste Coudurier]

On 3/16/2009 5:09 PM, Baptiste Coudurier wrote:
> On 3/16/2009 5:01 PM, Michael Niedermayer wrote:
>> On Mon, Mar 16, 2009 at 04:48:35PM -0700, Baptiste Coudurier wrote:
>>
>> [...]
>>
>> * Metadata is data that is not directly part of the presentation but
>>   data about the presentation
> 
> IMHO "presentation" in case of audio is the "sound" produced. not the cover.

Also, I think this is not libavformat's job to take care about
presentation, but application's job. libavformat should export the data
in the most _convenient_ way for the application, and IMHO this is not
necessarly char*, especially if the application wants to do some
treatment on the data.

>> [...]
>> 
>> Also if images are put in AVMetadata we also need full support for ffmpeg
>> to encode, decode and "-vcodec copy" them as well as ffplay to display them
>> i suspect this will be uglier than 2 lines of code to create a seperate
>> stream for the cover.

Besides, code needed to create separatetracks in demuxers is ugly too.

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