[FFmpeg-devel] [PATCH] avformat/mov: Disable XMP metadata by default

Vittorio Giovara vittorio.giovara at gmail.com
Thu Dec 11 02:22:28 CET 2014


On Tue, Dec 9, 2014 at 10:15 PM, wm4 <nfxjfg at googlemail.com> wrote:
> On Tue, 9 Dec 2014 21:33:41 +0100
> Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:
>
>> On Tue, Dec 09, 2014 at 12:59:24PM +0000, Vittorio Giovara wrote:
>> > On Tue, Dec 9, 2014 at 8:04 AM, wm4 <nfxjfg at googlemail.com> wrote:
>> > > On Tue,  9 Dec 2014 02:57:01 +0100
>> > > Michael Niedermayer <michaelni at gmx.at> wrote:
>> > >
>> > > It seems very wrong to export this info optionally just because
>> > > ffmpeg.c is too stupid to handle it correctly.
>> >
>> > Not exporting saves a little bit of memory too, since you have to
>> > allocate the space where to export this metadata to too.
>> > We're often talking about more than hundreds of megabytes.
>
> WTF, > 100 MBs of XML? In media files?

Yep, it's not uncommon that xmp contains the all the save states of
the project files (eg the list of undo action), imports, outputs
proxies and other stuff. It gets to ridiculous dimensions pretty
quickly.

>> From that aspect it might make more sense to have a generic
>> option to not export metadata larger than a certain size for example.
>> Or some other system.
>
> Until now, the method for exporting large metadata-like blobs was
> adding new streams. E.g. picture attachments and fonts.

That would be neat. Until then, can we have this patch committed?
-- 
Vittorio


More information about the ffmpeg-devel mailing list