[FFmpeg-devel] [PATCH 3/3] Export metadata in the generic format.

[Baptiste Coudurier]

Hi Aurel,

On 10/05/2010 01:05 PM, Aurelien Jacobs wrote:
> On Tue, Oct 05, 2010 at 07:57:06PM +0200, Anton Khirnov wrote:
>> deprecate old conversion API.
>> [...]
>
> I'm not totally sure after all those changes, so maybe you could refresh
> my mind, but to me, it looks like the only problem this patch is fixing
> is supporting demuxer which could make use of different conversion
> tables depending on the actual metadata format present in the file
> (ie. supporting id3v1 append at the end of any kind of format).
>
> Maybe we could solve this problem in a simpler way by just moving
> AV(In/Out)putFormat.metadata_conv to AVMetadata.conv.
> When reading metadata out of a file, the demuxer would set
> AVMetadata.conv to the appropriate conversion table.
> Then we just have to change to av_metadata_conv() calls to read this new
> pointer instead of the old one.

Woot I love this idea.

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org