[FFmpeg-devel] [PATCH] lavf/mov.c: Set start_time for audio too (in case of edit lists).

[Sasi Inguva]

i intended it only for audio, but i don't see any harm if it gets applied
to subtitle tracks also .

On Tue, Aug 7, 2018 at 3:23 PM Michael Niedermayer <michael at niedermayer.cc>
wrote:

> On Mon, Aug 06, 2018 at 04:32:29PM -0700, isasi-at-google.com at ffmpeg.org
> wrote:
> > From: Sasi Inguva <isasi at google.com>
> >
> > Fixes vorbis mp4 audio files, with edit list specified. Since
> > st->skip_samples is not set in case of vorbis , ffmpeg computes the
> > start_time as negative.
> >
> > Signed-off-by: Sasi Inguva <isasi at google.com>
> > ---
> >  libavformat/mov.c                              | 4 ++--
> >  tests/fate/mov.mak                             | 5 +++++
> >  tests/ref/fate/mov-neg-firstpts-discard-vorbis | 3 +++
> >  3 files changed, 10 insertions(+), 2 deletions(-)
> >  create mode 100644 tests/ref/fate/mov-neg-firstpts-discard-vorbis
>
> The commit message speaks of audio, this changes subtitles too
> is this intended ?
>
> thx
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> I have often repented speaking, but never of holding my tongue.
> -- Xenocrates
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>