[FFmpeg-devel] [PATCH 1/6] ffmpeg.c: copy global metadata by default
Michael Niedermayer
michaelni
Thu Nov 4 01:41:52 CET 2010
On Tue, Nov 02, 2010 at 10:04:00PM +0100, Anton Khirnov wrote:
> ---
> ffmpeg.c | 12 ++++++++++++
> 1 files changed, 12 insertions(+), 0 deletions(-)
>
> diff --git a/ffmpeg.c b/ffmpeg.c
> index fd3b404..26237a2 100644
> --- a/ffmpeg.c
> +++ b/ffmpeg.c
> @@ -129,6 +129,7 @@ static int nb_stream_maps;
> /* first item specifies output metadata, second is input */
> static AVMetaDataMap (*meta_data_maps)[2] = NULL;
> static int nb_meta_data_maps;
> +static int metadata_global_autocopy = 1;
> static int metadata_streams_autocopy = 1;
> static int metadata_chapters_autocopy = 1;
>
> @@ -2380,6 +2381,15 @@ static int transcode(AVFormatContext **output_files,
> av_metadata_set2(meta[0], mtag->key, mtag->value, AV_METADATA_DONT_OVERWRITE);
> }
>
> + /* copy global metadata by default */
> + if (metadata_global_autocopy) {
> + AVMetadataTag *t = NULL;
> +
> + while ((t = av_metadata_get(input_files[0]->metadata, "", t, AV_METADATA_IGNORE_SUFFIX)))
> + for (i = 0; i < nb_output_files; i++)
> + av_metadata_set2(&output_files[i]->metadata, t->key, t->value, AV_METADATA_DONT_OVERWRITE);
> + }
is that copying code seperate from the one used to copy what the user specifies?
that seems a bit messy
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- 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/20101104/a15c7f19/attachment.pgp>
More information about the ffmpeg-devel
mailing list