Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-11-02 Thread Zhenni Huang
On Tue, Nov 1, 2016 at 5:45 PM, Carl Eugen Hoyos wrote: > 2016-10-31 21:33 GMT+01:00 Zhenni Huang ffmpeg.org>: > > > Thanks Carl, I think setting strict_std_compliance is fine. Could please > > you apply the change? > > Done. > > Carl Eugen > ___ > ffm

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-11-01 Thread Carl Eugen Hoyos
2016-10-31 21:33 GMT+01:00 Zhenni Huang : > Thanks Carl, I think setting strict_std_compliance is fine. Could please > you apply the change? Done. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpe

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-10-31 Thread Zhenni Huang
On Mon, Oct 24, 2016 at 4:48 PM, Carl Eugen Hoyos wrote: > 2016-10-25 1:23 GMT+02:00 Zhenni Huang peg.org>: > > > Thanks for your reply. Setting strict_std_compliance to 2 could > > help in this case. However, as it is a global flag, it could influence > > other parts in demuxers. > > Yes, this

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-10-24 Thread Carl Eugen Hoyos
2016-10-25 1:23 GMT+02:00 Zhenni Huang : > Thanks for your reply. Setting strict_std_compliance to 2 could > help in this case. However, as it is a global flag, it could influence > other parts in demuxers. Yes, this is intended: If you don't want to read invalid mov files, it seems logical that

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-10-24 Thread Zhenni Huang
Hi Carl, Thanks for your reply. Setting strict_std_compliance to 2 could help in this case. However, as it is a global flag, it could influence other parts in demuxers. It is preferable if we can have control of whether to use moov in free with one separate flag. Thanks, Zhenni On Mon, Oct 24, 2

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-10-24 Thread Carl Eugen Hoyos
2016-10-24 23:11 GMT+02:00 Zhenni Huang : [...] Does the following inlined patch help you? Carl Eugen diff --git a/libavformat/mov.c b/libavformat/mov.c index 357d800..ed099fc 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -4884,6 +4884,7 @@ a.type = avio_rl32(pb);

[FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-10-24 Thread Zhenni Huang
From: liangsi --- libavformat/isom.h | 1 + libavformat/mov.c | 5 - 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/isom.h b/libavformat/isom.h index 2246fed..6824f7e 100644 --- a/libavformat/isom.h +++ b/libavformat/isom.h @@ -214,6 +214,7 @@ typedef struct MOVCo