avviddec: libav will already copy the reordered_opaque pointer for us
authorSebastian Dröge <sebastian@centricular.com>
Fri, 26 Jun 2015 13:38:38 +0000 (15:38 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Fri, 26 Jun 2015 14:58:55 +0000 (16:58 +0200)
If we do it ourselves, it might get the wrong value if our assumptions are
broken by libav at a later time.

ext/libav/gstavviddec.c

index 13a8d30..e7111bb 100644 (file)
@@ -623,7 +623,6 @@ gst_ffmpegviddec_get_buffer (AVCodecContext * context, AVFrame * picture)
   /* apply the last info we have seen to this picture, when we get the
    * picture back from ffmpeg we can use this to correctly timestamp the output
    * buffer */
-  picture->reordered_opaque = context->reordered_opaque;
   GST_DEBUG_OBJECT (ffmpegdec, "opaque value SN %d",
       (gint32) picture->reordered_opaque);
 
@@ -794,8 +793,6 @@ gst_ffmpegviddec_reget_buffer (AVCodecContext * context, AVFrame * picture)
 
   GST_DEBUG_OBJECT (ffmpegdec, "regetting buffer picture %p", picture);
 
-  picture->reordered_opaque = context->reordered_opaque;
-
   /* if there is no opaque, we didn't yet attach any frame to it. What usually
    * happens is that avcodec_default_reget_buffer will call the getbuffer
    * function. */