[FFmpeg-devel] [PATCH] lavf/mov.c: Set st->start_time for video streams explicitly.

[Michael Niedermayer]

On Thu, May 31, 2018 at 04:26:24PM -0700, Sasi Inguva wrote:
> Pls find attached, the correctly signed patch
> Thanks.
> 
> On Thu, May 31, 2018 at 4:14 PM Sasi Inguva <isasi at google.com> wrote:
> 
> > Sorry. Forgot to attach the file. Pls find it attached. Also resending the
> > patch signed with correct email address
> >
> > On Thu, May 31, 2018 at 2:14 PM Michael Niedermayer <michael at niedermayer.cc>
> > wrote:
> >
> >> On Tue, May 29, 2018 at 03:39:40PM -0700, Sasi Inguva wrote:
> >> > If start_time is not set, ffmpeg takes the duration from the global
> >> > movie instead of the per stream duration.
> >> > Signed-off-by: Sasi Inguva <isasi at rodete-desktop-imager.corp.google.com
> >> >
> >> > ---
> >> >  libavformat/mov.c                            | 20 +++++++++---
> >> >  tests/fate/mov.mak                           |  4 +++
> >> >  tests/ref/fate/mov-neg-firstpts-discard      |  2 +-
> >> >  tests/ref/fate/mov-stream-shorter-than-movie | 33 ++++++++++++++++++++
> >> >  4 files changed, 54 insertions(+), 5 deletions(-)
> >> >  create mode 100644 tests/ref/fate/mov-stream-shorter-than-movie
> >>
> >> i may have missed a link to a sample file
> >>
> >> fate-suite//mov/mov_stream_shorter_than_movie.mov: No such file or
> >> directory
> >> make: *** [fate-mov-stream-shorter-than-movie] Error 1
> >>
> >>
> >>
> >>
> >> [...]
> >> --
> >> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> >>
> >> It is dangerous to be right in matters on which the established
> >> authorities
> >> are wrong. -- Voltaire
> >> _______________________________________________
> >> ffmpeg-devel mailing list
> >> ffmpeg-devel at ffmpeg.org
> >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >>
> >

>  libavformat/mov.c                            |   20 +++++++++++++---
>  tests/fate/mov.mak                           |    4 +++
>  tests/ref/fate/mov-neg-firstpts-discard      |    2 -
>  tests/ref/fate/mov-stream-shorter-than-movie |   33 +++++++++++++++++++++++++++
>  4 files changed, 54 insertions(+), 5 deletions(-)
> 5d8c7d670192a5f785f22bc79e8bf02be950c9f4  0001-lavf-mov.c-Set-st-start_time-for-video-streams-expli.patch
> From ba2feb04ef1ca94f7e1972e23c84c2acde93bf68 Mon Sep 17 00:00:00 2001
> From: Sasi Inguva <isasi at google.com>
> Date: Tue, 29 May 2018 15:36:07 -0700
> Subject: [PATCH] lavf/mov.c: Set st->start_time for video streams explicitly.
> 
> If start_time is not set, ffmpeg takes the duration from the global
> movie instead of the per stream duration.
> Signed-off-by: Sasi Inguva <isasi at google.com>
> ---
>  libavformat/mov.c                            | 20 +++++++++---
>  tests/fate/mov.mak                           |  4 +++
>  tests/ref/fate/mov-neg-firstpts-discard      |  2 +-
>  tests/ref/fate/mov-stream-shorter-than-movie | 33 ++++++++++++++++++++
>  4 files changed, 54 insertions(+), 5 deletions(-)
>  create mode 100644 tests/ref/fate/mov-stream-shorter-than-movie

will apply

thanks


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180606/a18d2676/attachment.sig>