[FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata. Also updates fate ref for rgb24-mkv and fate-mov-zombie as the output will contain the new metadata field.

2016-11-14 Thread Zhenni Huang
--- libavformat/mov.c | 23 +-- tests/ref/fate/mov-zombie | 2 +- tests/ref/fate/rgb24-mkv | 4 ++-- 3 files changed, 24 insertions(+), 5 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9ec7d03..6176415 100644 --- a/libavformat/mov.c +++ b/liba

[FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata. Also updates fate ref for rgb24-mkv and fate-mov-zombie as the output will contain the new metadata field.

2016-11-11 Thread Zhenni Huang
--- libavformat/mov.c | 19 +-- tests/ref/fate/mov-zombie | 2 +- tests/ref/fate/rgb24-mkv | 4 ++-- 3 files changed, 20 insertions(+), 5 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9ec7d03..b547832 100644 --- a/libavformat/mov.c +++ b/libavfor

[FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata.

2016-11-09 Thread Zhenni Huang
Also updates fate ref for rgb24-mkv as the output video will contain the new metadata field and have different md5sum and file size. --- libavformat/mov.c| 18 -- tests/ref/fate/rgb24-mkv | 4 ++-- 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/libavformat

Re: [FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata.

2016-11-07 Thread Zhenni Huang
On Mon, Nov 7, 2016 at 4:30 PM, Zhenni Huang wrote: > Also updates fate ref for rgb24-mkv as the output video will contain the > new metadata field and have different md5sum and file size. > --- > libavformat/mov.c| 25 +++-- > tests/ref/fate/rgb24-mkv

[FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata.

2016-11-07 Thread Zhenni Huang
Also updates fate ref for rgb24-mkv as the output video will contain the new metadata field and have different md5sum and file size. --- libavformat/mov.c| 25 +++-- tests/ref/fate/rgb24-mkv | 4 ++-- 2 files changed, 25 insertions(+), 4 deletions(-) diff --git a/liba

[FFmpeg-devel] [PATCH] mov: extract stsd vendor field in metadata.

2016-11-03 Thread Zhenni Huang
--- libavformat/mov.c | 25 +++-- 1 file changed, 23 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 4222088..2cb041f 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1824,6 +1824,8 @@ static void mov_parse_stsd_video(MOVContext

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? >

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

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
, 2016 at 2:54 PM, Carl Eugen Hoyos wrote: > 2016-10-24 23:11 GMT+02:00 Zhenni Huang ffmpeg.org>: > > [...] > > Does the following inlined patch help you? > > Carl Eugen > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index 357d800..ed099fc 100644

[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

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms.

2016-10-13 Thread Zhenni Huang
Sorry about the second patch I sent. This option is for flexibility since some apps may not want to use the hidden atoms. On Thu, Oct 13, 2016 at 1:44 PM, Robert Krüger wrote: > On Thu, Oct 13, 2016 at 9:54 PM, Dave Rice wrote: > > > > > > On Oct 13, 2016, at 1:56 PM, Hendrik Leppkes > > wrot