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

2014-12-11 Thread Michael Niedermayer
On Thu, Dec 11, 2014 at 01:22:28AM +, Vittorio Giovara wrote: > On Tue, Dec 9, 2014 at 10:15 PM, wm4 wrote: > > On Tue, 9 Dec 2014 21:33:41 +0100 > > Reimar Döffinger wrote: > > > >> On Tue, Dec 09, 2014 at 12:59:24PM +, Vittorio Giovara wrote: > >> > On Tue, Dec 9, 2014 at 8:04 AM, wm4

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

2014-12-10 Thread Vittorio Giovara
On Tue, Dec 9, 2014 at 10:15 PM, wm4 wrote: > On Tue, 9 Dec 2014 21:33:41 +0100 > Reimar Döffinger wrote: > >> On Tue, Dec 09, 2014 at 12:59:24PM +, Vittorio Giovara wrote: >> > On Tue, Dec 9, 2014 at 8:04 AM, wm4 wrote: >> > > On Tue, 9 Dec 2014 02:57:01 +0100 >> > > Michael Niedermayer w

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

2014-12-09 Thread wm4
On Tue, 9 Dec 2014 21:33:41 +0100 Reimar Döffinger wrote: > On Tue, Dec 09, 2014 at 12:59:24PM +, Vittorio Giovara wrote: > > On Tue, Dec 9, 2014 at 8:04 AM, wm4 wrote: > > > On Tue, 9 Dec 2014 02:57:01 +0100 > > > Michael Niedermayer wrote: > > > > > > It seems very wrong to export this i

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

2014-12-09 Thread Reimar Döffinger
On Tue, Dec 09, 2014 at 12:59:24PM +, Vittorio Giovara wrote: > On Tue, Dec 9, 2014 at 8:04 AM, wm4 wrote: > > On Tue, 9 Dec 2014 02:57:01 +0100 > > Michael Niedermayer wrote: > > > > It seems very wrong to export this info optionally just because > > ffmpeg.c is too stupid to handle it corr

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

2014-12-09 Thread Vittorio Giovara
On Tue, Dec 9, 2014 at 8:04 AM, wm4 wrote: > On Tue, 9 Dec 2014 02:57:01 +0100 > Michael Niedermayer 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 allo

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

2014-12-09 Thread wm4
On Tue, 9 Dec 2014 02:57:01 +0100 Michael Niedermayer wrote: > This was suggested by cbsrobot, ubitux and koda > > There are files with huge amounts of XMP data, which would otherwise > be displayed in the terminal output of FFmpeg > > Signed-off-by: Michael Niedermayer > --- > libavformat/m

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

2014-12-08 Thread Clément Bœsch
On Tue, Dec 09, 2014 at 02:57:01AM +0100, Michael Niedermayer wrote: > This was suggested by cbsrobot, ubitux and koda > > There are files with huge amounts of XMP data, which would otherwise > be displayed in the terminal output of FFmpeg > Can you just mention that in the behaviour changes sec

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

2014-12-08 Thread Michael Niedermayer
This was suggested by cbsrobot, ubitux and koda There are files with huge amounts of XMP data, which would otherwise be displayed in the terminal output of FFmpeg Signed-off-by: Michael Niedermayer --- libavformat/mov.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/liba