[FFmpeg-devel] [PATCH] avformat/mov: compute dts_shift with trun cts
Clément Bœsch
ubitux at gmail.com
Fri Jun 7 16:31:08 CEST 2013
On Fri, Jun 07, 2013 at 03:12:56PM +0200, Alexandre Sicard wrote:
> Le jeudi 6 juin 2013 19:45:33 Clément Bœsch a écrit :
> > Patch should be OK. The hack could be isolated in a function and shared
> > with the one in mov_read_ctts(), but I don't consider this blocking.
> >
> > Another nitpick would be to use an intermediate duration variable to avoid
> > requesting 3 times sc->ctts_data[sc->ctts_count].duration, but that's not
> > really important as well.
>
> Updated patch following your first suggestion.
>
> --
> Alexandre SICARD - Intern, Research & Engineering
> SmartJog | www.smartjog.com | a TDF Group company
> From 6bd0b035d4122491cf7c3031aaf1fcb71ed24331 Mon Sep 17 00:00:00 2001
> From: Alexandre Sicard <alexandre.sicard at smartjog.com>
> Date: Fri, 7 Jun 2013 14:56:16 +0200
> Subject: [PATCH] avformat/mov: compute dts_shift with trun cts
>
> Some movies have negative composition time offsets in their trun, causing pts <
> dts errors. This patch makes use of dts_shift to handle them.
>
> Signed-off-by: Alexandre Sicard <alexandre.sicard at smartjog.com>
> ---
> libavformat/mov.c | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
>
Applied, thank you.
[...]
--
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130607/f8323066/attachment.asc>
More information about the ffmpeg-devel
mailing list