[FFmpeg-devel] [PATCH] avformat/mov: compute dts_shift with trun cts

[Clément Bœsch]

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>