[FFmpeg-cvslog] r26260 - in trunk: doc/APIchanges ffplay.c libavcodec/avcodec.h libavcodec/utils.c

[Michael Niedermayer]

On Wed, Feb 02, 2011 at 10:07:35PM +0100, Nicolas George wrote:
> Le nonidi 19 niv?se, an CCXIX, michael a ?crit?:
> > @@ -344,6 +344,8 @@ int avcodec_default_get_buffer(AVCodecCo
> >      }
> >      s->internal_buffer_count++;
> >  
> > +    if(s->pkt) pic->pkt_pts= s->pkt->pts;
> > +    else       pic->pkt_pts= AV_NOPTS_VALUE;
> >      pic->reordered_opaque= s->reordered_opaque;
> 
> I believe you forgot the same lines in _reget_buffer. This is a part of the
> code I really do not understand well, but when working on using pkt_pts in
> ffmpeg, I noticed that fate-vsynth1-flashsv fails because the flashsv
> decoder do not update pkt_pts when decoding vsynth1/flashsv.flv. The
> attached patch seems to fix it.
> 
> Regards,
> 
> -- 
>   Nicolas George

>  utils.c |    2 ++
>  1 file changed, 2 insertions(+)
> ad0bfaf6b671232aadc583bc4867329bd8c016fb  0001-Reorder-pts-in-avcodec_default_reget_buffer.patch
> From 245118f72c54819daf7c99d43a9934f14f83cd12 Mon Sep 17 00:00:00 2001
> From: Nicolas George <nicolas.george at normalesup.org>
> Date: Wed, 2 Feb 2011 21:54:14 +0100
> Subject: [PATCH] Reorder pts in avcodec_default_reget_buffer.

LGTM if tested

can you commit to videolan? or should i?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20110203/df2399a7/attachment.pgp>