[FFmpeg-devel] [PATCH] In get_video_frame(), use frame->pkt_pts rather than the deprecated reordered_opaque API, which is deprecated for this specific use.

Stefano Sabatini stefano.sabatini-lala
Sun Jan 30 19:18:59 CET 2011


On date Sunday 2011-01-30 18:02:11 +0000, M?ns Rullg?rd encoded:
> Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
> 
> > From e967ddb90f76e3a5e8a310c5d6d844374aab63ed Mon Sep 17 00:00:00 2001
> > From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > Date: Sun, 16 Jan 2011 18:39:34 +0100
> > Subject: [PATCH] In ffplay:get_video_frame(), use frame->pkt_pts rather than
> >  reordered_opaque.
> >
> > AVCodecContext.reordered_opaque is deprecated for this specific use.
> > ---
> >  ffplay.c |    5 ++---
> >  1 files changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/ffplay.c b/ffplay.c
> > index edc6097..1df3824 100644
> > --- a/ffplay.c
> > +++ b/ffplay.c
> > @@ -1570,16 +1570,15 @@ static int get_video_frame(VideoState *is, AVFrame *frame, int64_t *pts, AVPacke
> >          return 0;
> >      }
> >  
> > -    is->video_st->codec->reordered_opaque = pkt->pts;
> >      len1 = avcodec_decode_video2(is->video_st->codec,
> >                                   frame, &got_picture,
> >                                   pkt);
> >  
> >      if (got_picture) {
> >          if (decoder_reorder_pts == -1) {
> > -            *pts = guess_correct_pts(&is->pts_ctx, frame->reordered_opaque, pkt->dts);
> > +            *pts = guess_correct_pts(&is->pts_ctx, frame->pkt_pts, pkt->dts);
> >          } else if (decoder_reorder_pts) {
> > -            *pts = frame->reordered_opaque;
> > +            *pts = frame->pkt_pts;
> >          } else {
> >              *pts = pkt->dts;
> >          }
> > -- 
> > 1.7.2.3
> 
> Looks good.  Is this ready to push now, or did I miss some patch that
> needs to go in first?

Ready to go (and already approved by Michael).
-- 
FFmpeg = Frenzy Foolish Mortal Porno Egregious Ghost



More information about the ffmpeg-devel mailing list