[FFmpeg-devel] [PATCH 6/6] mxfdec: Set DTS and PTS on video packets

[Tomas Härdin]

On Mon, 2011-12-12 at 18:33 +0100, Michael Niedermayer wrote:
> On Mon, Dec 12, 2011 at 10:22:28AM +0100, Tomas Härdin wrote:
> > Only video because I suspect that computing timestamps for compressed
> > audio is going to be tricky. PCM would of course be easier, but I think
> > the timestamp generation works fine for that?
> >
> > /Tomas
> 
> >  libavformat/mxfdec.c    |   96 +++++++++++++++++++++++++++++++++++++++++++++++-
> >  tests/ref/seek/lavf_mxf |   44 +++++++++++-----------
> >  2 files changed, 117 insertions(+), 23 deletions(-)
> > 4b7eb8848ec1961fd3ecf24b5b1e036c49c0e22e  0006-mxfdec-Set-DTS-and-PTS-on-video-packets.patch
> > From 742c637907a8285df879a1092d5c92ab87f4e065 Mon Sep 17 00:00:00 2001
> > From: =?UTF-8?q?Tomas=20H=C3=A4rdin?= <tomas.hardin at codemill.se>
> > Date: Fri, 9 Dec 2011 17:56:46 +0100
> > Subject: [PATCH 6/6] mxfdec: Set DTS and PTS on video packets
> >
> > This involves computing DTS/PTS from the index tables.
> 
> Applied, thanks

Cool, thanks.

> also should anything of this patchset be backported for 0.9.1 ?

I'd say the first two.
Since it unbreaks a few samples: "[PATCH 1/6] mxfdec: Bring back
accumulated_offset that was removed in 5e67e3e"
Since it fixes a potential SIGSEGV: "[PATCH 2/6] mxfdec: Check
slice_count before freeing slice_offset_entries"

/Tomas