[FFmpeg-devel] [PATCH] add AV_METADATA_DONT_OVERWRITE flag

Anton Khirnov wyskas
Fri Apr 2 21:28:26 CEST 2010


On Wed, Mar 31, 2010 at 11:26:12PM +0200, Michael Niedermayer wrote:
> On Wed, Mar 31, 2010 at 07:10:04PM +0200, Anton Khirnov wrote:
> > Hi,
> > these patches a) add a flag for av_metadata_set2() with which it doesn't
> > overwrite existing tags and b) use this flag in ffmpeg.c
> 
> this patchset looks ok but it doesnt fix the problem with
> convertion, that is tags are stored from various sources with various
> convertion tables in the destination and then all the convertion tables
> are applied more or less. This is just not correct
> 
yeah, i know, these patches weren't supposed to fix that. i'm not sure
yet what the proper fix would be.

anyway, updated patches against HEAD attached, can somebody apply them
please? (don't forget to put correct revision number to doc/APIChanges)

Anton Khirnov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-metadata-API-add-a-flag-to-not-overwrite-existing-ta.patch
Type: text/x-diff
Size: 1972 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100402/0c1f1216/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-ffmpeg.c-don-t-overwrite-user-supplied-metadata-with.patch
Type: text/x-diff
Size: 959 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100402/0c1f1216/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-ffmpeg.c-simplify-writing-stream-metadata.patch
Type: text/x-diff
Size: 1490 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100402/0c1f1216/attachment-0002.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100402/0c1f1216/attachment.pgp>



More information about the ffmpeg-devel mailing list