Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-11 Thread Michael Niedermayer
On Thu, Nov 09, 2017 at 04:11:22PM -0800, Sasi Inguva wrote: > Partially fixes t/6699. > --- > libavformat/mov.c | 32 +++--- > tests/fate/mov.mak| 4 > tests/ref/fate/mov-elst-ends-betn-b-and-i | 33 >

[FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-09 Thread Sasi Inguva
Partially fixes t/6699. --- libavformat/mov.c | 32 +++--- tests/fate/mov.mak| 4 tests/ref/fate/mov-elst-ends-betn-b-and-i | 33 +++ 3 files changed, 58 insertions(+), 11 deletions(-)

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-09 Thread Sasi Inguva
Sorry, about this. i had uncommitted changes. Attaching the patch. On Thu, Nov 9, 2017 at 12:22 PM, Michael Niedermayer wrote: > On Wed, Nov 08, 2017 at 04:13:46PM -0800, Sasi Inguva wrote: > > Partially fixes t/6699. > > --- > > libavformat/mov.c

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-09 Thread Michael Niedermayer
On Wed, Nov 08, 2017 at 04:13:46PM -0800, Sasi Inguva wrote: > Partially fixes t/6699. > --- > libavformat/mov.c | 32 > --- > tests/fate/mov.mak| 4 > tests/ref/fate/mov-elst-ends-betn-b-and-i | 31

[FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-08 Thread Sasi Inguva
Partially fixes t/6699. --- libavformat/mov.c | 32 --- tests/fate/mov.mak| 4 tests/ref/fate/mov-elst-ends-betn-b-and-i | 31 ++ 3 files changed, 56 insertions(+), 11 deletions(-)

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-08 Thread Sasi Inguva
Updated the fate test after http://git.videolan.org/?p=ffmpeg.git;a=commit;h=c2a8f0fcbe57ea9ccaa864130f078af10516c3c1 . Sending the updated patch. On Thu, Nov 2, 2017 at 4:23 PM, Michael Niedermayer wrote: > On Wed, Nov 01, 2017 at 02:27:50PM -0700, Sasi Inguva wrote: >

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-02 Thread Michael Niedermayer
On Wed, Nov 01, 2017 at 02:27:50PM -0700, Sasi Inguva wrote: > Pls find attached, the FATE sample. uplaoded [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB There will always be a question for which you do not know the correct answer. signature.asc

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-01 Thread Sasi Inguva
This patch is not an alternative. Both patches are required to fix t/6699 . Once one of the patches is submitted, i will update the fate tests in the other patch. On Wed, Nov 1, 2017 at 5:11 PM, Michael Niedermayer wrote: > On Wed, Nov 01, 2017 at 02:26:45PM -0700, Sasi

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-01 Thread Michael Niedermayer
On Wed, Nov 01, 2017 at 02:26:45PM -0700, Sasi Inguva wrote: > Partially fixes t/6699. > --- > libavformat/mov.c | 32 > --- > tests/fate/mov.mak| 4 > tests/ref/fate/mov-elst-ends-betn-b-and-i | 31

[FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-01 Thread Sasi Inguva
Partially fixes t/6699. --- libavformat/mov.c | 32 +--- tests/fate/mov.mak | 4 2 files changed, 25 insertions(+), 11 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 60f0228e2d..e716e5ba9e 100644 --- a/libavformat/mov.c +++

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-01 Thread Sasi Inguva
Pls find attached, the FATE sample. On Wed, Nov 1, 2017 at 2:26 PM, Sasi Inguva wrote: > Partially fixes t/6699. > --- > libavformat/mov.c | 32 > --- > tests/fate/mov.mak| 4 >

[FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-01 Thread Sasi Inguva
Partially fixes t/6699. --- libavformat/mov.c | 32 --- tests/fate/mov.mak| 4 tests/ref/fate/mov-elst-ends-betn-b-and-i | 31 ++ 3 files changed, 56 insertions(+), 11 deletions(-)

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Parse upto 2 keyframes after the edit list end in mov_fix_index.

2017-11-01 Thread Sasi Inguva
Forgot to attach the FATE ref file. sending the patch again. On Wed, Nov 1, 2017 at 2:23 PM, Sasi Inguva wrote: > Partially fixes t/6699. > --- > libavformat/mov.c | 32 +--- > tests/fate/mov.mak | 4 > 2 files changed, 25 insertions(+), 11