Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-13 Thread Michael Niedermayer
On Thu, Oct 12, 2017 at 10:43:54AM +0200, Thomas Mundt wrote: > Hi Michael, > > 2017-10-12 1:28 GMT+02:00 Michael Niedermayer : > > > On Tue, Oct 10, 2017 at 10:26:13PM +0200, Thomas Mundt wrote: > > > 2017-10-10 19:36 GMT+02:00 Sasi Inguva : > > > > > > > This is required for FLV files, for whic

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-12 Thread Sasi Inguva
I can initialize duration_pts to -1, and check against duration_pts < 0 , if you think that's better. On Thu, Oct 12, 2017 at 3:51 AM, wm4 wrote: > On Thu, 12 Oct 2017 01:28:01 +0200 > Michael Niedermayer wrote: > > > On Tue, Oct 10, 2017 at 10:26:13PM +0200, Thomas Mundt wrote: > > > 2017-10-1

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-12 Thread wm4
On Thu, 12 Oct 2017 01:28:01 +0200 Michael Niedermayer wrote: > On Tue, Oct 10, 2017 at 10:26:13PM +0200, Thomas Mundt wrote: > > 2017-10-10 19:36 GMT+02:00 Sasi Inguva : > > > > > This is required for FLV files, for which duration_pts comes out to be > > > zero. > > > > > > Signed-off-by: Sas

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-12 Thread Nicolas George
L'octidi 18 vendémiaire, an CCXXVI, Sasi Inguva a écrit : > This is required for FLV files, for which duration_pts comes out to be zero. > > Signed-off-by: Sasi Inguva > --- > fftools/ffmpeg.c | 3 +++ > 1 file changed, 3 insertions(+) Could you make a FATE test so that it stays fixed? Regards

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-12 Thread Thomas Mundt
Hi Michael, 2017-10-12 1:28 GMT+02:00 Michael Niedermayer : > On Tue, Oct 10, 2017 at 10:26:13PM +0200, Thomas Mundt wrote: > > 2017-10-10 19:36 GMT+02:00 Sasi Inguva : > > > > > This is required for FLV files, for which duration_pts comes out to be > > > zero. > > > > > > Signed-off-by: Sasi Ing

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-11 Thread Michael Niedermayer
On Tue, Oct 10, 2017 at 10:26:13PM +0200, Thomas Mundt wrote: > 2017-10-10 19:36 GMT+02:00 Sasi Inguva : > > > This is required for FLV files, for which duration_pts comes out to be > > zero. > > > > Signed-off-by: Sasi Inguva > > --- > > fftools/ffmpeg.c | 9 +++-- > > 1 file changed, 7 ins

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-10 Thread Thomas Mundt
2017-10-10 19:36 GMT+02:00 Sasi Inguva : > This is required for FLV files, for which duration_pts comes out to be > zero. > > Signed-off-by: Sasi Inguva > --- > fftools/ffmpeg.c | 9 +++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-10 Thread Thomas Mundt
2017-10-10 19:44 GMT+02:00 wm4 : > On Tue, 10 Oct 2017 10:36:58 -0700 > Sasi Inguva wrote: > > > This is required for FLV files, for which duration_pts comes out to be > zero. > > > > Signed-off-by: Sasi Inguva > > --- > > fftools/ffmpeg.c | 9 +++-- > > 1 file changed, 7 insertions(+), 2 d

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-10 Thread wm4
On Tue, 10 Oct 2017 10:36:58 -0700 Sasi Inguva wrote: > This is required for FLV files, for which duration_pts comes out to be zero. > > Signed-off-by: Sasi Inguva > --- > fftools/ffmpeg.c | 9 +++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/fftools/ffmpeg.c b/fft

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-10 Thread Sasi Inguva
True. Thanks! Sending the corrected patch. On Tue, Oct 10, 2017 at 5:40 AM, Thomas Mundt wrote: > 2017-10-10 3:27 GMT+02:00 Sasi Inguva : > >> This is required for FLV files, for which duration_pts comes out to be >> zero. >> >> Signed-off-by: Sasi Inguva >> --- >> fftools/ffmpeg.c | 3 +++ >>

[FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-10 Thread Sasi Inguva
This is required for FLV files, for which duration_pts comes out to be zero. Signed-off-by: Sasi Inguva --- fftools/ffmpeg.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 6d64bc1043..3ee31473dc 100644 --- a/fftools/ffmpeg.c

Re: [FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-10 Thread Thomas Mundt
2017-10-10 3:27 GMT+02:00 Sasi Inguva : > This is required for FLV files, for which duration_pts comes out to be > zero. > > Signed-off-by: Sasi Inguva > --- > fftools/ffmpeg.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c > index 6d64bc1043..5f3

[FFmpeg-devel] [PATCH] ffmpeg.c: Fallback to duration_dts, when duration_pts can't be determined.

2017-10-09 Thread Sasi Inguva
This is required for FLV files, for which duration_pts comes out to be zero. Signed-off-by: Sasi Inguva --- fftools/ffmpeg.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index 6d64bc1043..5f373db847 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.