Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-11-13 Thread Jan Ekstrom
On Fri, Oct 21, 2016 at 6:32 PM, Derek Buitenhuis wrote: > The design one fist: This approach is fundamentally wrong. Edit lists are > meant to be applied at presentation level, not packet level. The current > implementation will cause problems in a number of cases: > > * Audio packets. Especi

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-11-13 Thread Paul B Mahol
On 11/12/16, Sasi Inguva wrote: > The above file has an edit list with duration zero . > > * > Even quick time player doesn't play the video. > > Enabling the old code to populate skip_samples by looking at first edit > list duration, when -ignore_editlist is true is a good fallback. I'll send >

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-11-11 Thread Sasi Inguva
The above file has an edit list with duration zero . ​ Even quick time player doesn't play the video. Enabling the old code to populate skip_samples by looking at first edit list duration, when -ignore_editlist is true is a good fallback. I'll send a commit soon. On Fri, Nov 11, 2016 at 11:08 P

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-11-11 Thread wm4
On Fri, 5 Aug 2016 17:18:39 -0700 Sasi Inguva wrote: > In YouTube we have long been receiving MOV files from users, which have > non-trivial edit lists (Those edit Found a file that does not show video at all with edit lists enabled: https://0x0.st/2Ep.mp4 You can test e.g. in ffplay. There'

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-11-02 Thread Derek Buitenhuis
On 10/24/2016 6:43 AM, Sasi Inguva wrote: > Just sent a patch, correcting a bug in the edit list code. PTAL. Hi again, it's me. Back with more files this patch set broke :). This time it's very easy to reproduce: $ ffmpeg -i sample.mp4 -f null - This works before your patchset, and also if

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-23 Thread Sasi Inguva
Just sent a patch, correcting a bug in the edit list code. PTAL. On Fri, Oct 21, 2016 at 11:08 AM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 10/21/2016 6:47 PM, Sasi Inguva wrote: > > * Audio packets. Especially audio packets with a large number of > samples. > >

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-21 Thread Derek Buitenhuis
On 10/21/2016 6:47 PM, Sasi Inguva wrote: > * Audio packets. Especially audio packets with a large number of > samples. > It's extremely likely that edits will not fall on packet > boundaries, and > depending on the number of samples per packet, audio sync issues > ca

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-21 Thread Sasi Inguva
On Fri, Oct 21, 2016 at 8:32 AM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > > In YouTube we have long been receiving MOV files from users, which have > non-trivial edit lists (Those edit lists which are not just used to offset > video start from audio start) and multiple edit lists. R

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-21 Thread wm4
On Fri, 21 Oct 2016 16:59:14 +0100 Derek Buitenhuis wrote: > On 10/21/2016 4:32 PM, Derek Buitenhuis wrote: > > * Audio packets. Especially audio packets with a large number of > > samples. > > It's extremely likely that edits will not fall on packet boundaries, > > and > > depe

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-21 Thread Derek Buitenhuis
On 10/21/2016 4:32 PM, Derek Buitenhuis wrote: > * Audio packets. Especially audio packets with a large number of samples. > It's extremely likely that edits will not fall on packet boundaries, and > depending on the number of samples per packet, audio sync issues can and > wi

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-21 Thread Derek Buitenhuis
> In YouTube we have long been receiving MOV files from users, which have > non-trivial edit lists (Those edit lists which are not just used to offset > video start from audio start) and multiple edit lists. Recently the uploads > of such files has increased with the introduction of apps that al

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-19 Thread Robert Krüger
On Tue, Oct 18, 2016 at 10:10 PM, Sasi Inguva < isasi-at-google@ffmpeg.org> wrote: > I am also happy to fix any breaking files, if they don't fall under the > above category. > > > The change seems to have caused resulting files to have incorrect duration in some cases (remuxing XDCAM material

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-19 Thread wm4
On Tue, 18 Oct 2016 13:08:07 -0700 Sasi Inguva wrote: > With the current state of code, we will only encounterdemuxer > outputting non-monotonic DTS when there are multiple edit lists . Avoiding > non-monotonic DTS for mulitple edit lists will not be possible with how the > edit list support

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-18 Thread Sasi Inguva
I am also happy to fix any breaking files, if they don't fall under the above category. On Tue, Oct 18, 2016 at 1:08 PM, Sasi Inguva wrote: > With the current state of code, we will only encounterdemuxer > outputting non-monotonic DTS when there are multiple edit lists . Avoiding > non-mono

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-18 Thread Sasi Inguva
With the current state of code, we will only encounterdemuxer outputting non-monotonic DTS when there are multiple edit lists . Avoiding non-monotonic DTS for mulitple edit lists will not be possible with how the edit list support is currently implemented. That would only be possible if the ed

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-18 Thread wm4
On Fri, 5 Aug 2016 17:18:39 -0700 Sasi Inguva wrote: > In YouTube we have long been receiving MOV files from users, which have > non-trivial edit lists (Those edit lists which are not just used to offset > video start from audio start) and multiple edit lists. Recently the uploads > of such f

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-08-10 Thread Sasi Inguva
Oops. I messed up with git send-email, and my message didn't occur in reply to this thread. So, here I am typing in Gmail.. Hi Clement and Michael, Sorry that I hadn't started a discussion on the patch before sending it for review. However this patch has been there for years in our ffmpeg versio

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-08-08 Thread Clément Bœsch
Hi, On Fri, Aug 05, 2016 at 05:18:39PM -0700, Sasi Inguva wrote: > In YouTube we have long been receiving MOV files from users, which have > non-trivial edit lists (Those edit lists which are not just used to offset > video start from audio start) and multiple edit lists. Recently the uploads >

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-08-06 Thread Michael Niedermayer
On Fri, Aug 05, 2016 at 05:18:40PM -0700, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavcodec/avcodec.h | 6 + > libavcodec/utils.c | 6 + > libavformat/avformat.h | 3 + > libavformat/mov.c

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-08-06 Thread Michael Niedermayer
On Fri, Aug 05, 2016 at 05:18:40PM -0700, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavcodec/avcodec.h | 6 + > libavcodec/utils.c | 6 + > libavformat/avformat.h | 3 + > libavformat/mov.c