avviddec: Remove unneeded check
authorEdward Hervey <edward@centricular.com>
Fri, 21 Sep 2018 14:01:53 +0000 (16:01 +0200)
committerEdward Hervey <bilboed@bilboed.com>
Fri, 21 Sep 2018 14:01:53 +0000 (16:01 +0200)
frame is always valid in this function (but wasn't before the
refactoring from a few months ago).

CID #1439540

ext/libav/gstavviddec.c

index de23954..8c9a794 100644 (file)
@@ -1800,14 +1800,12 @@ gst_ffmpegviddec_handle_frame (GstVideoDecoder * decoder,
   if (!packet.size)
     goto done;
 
-  if (frame) {
-    /* save reference to the timing info */
-    ffmpegdec->context->reordered_opaque = (gint64) frame->system_frame_number;
-    ffmpegdec->picture->reordered_opaque = (gint64) frame->system_frame_number;
+  /* save reference to the timing info */
+  ffmpegdec->context->reordered_opaque = (gint64) frame->system_frame_number;
+  ffmpegdec->picture->reordered_opaque = (gint64) frame->system_frame_number;
 
-    GST_DEBUG_OBJECT (ffmpegdec, "stored opaque values idx %d",
-        frame->system_frame_number);
-  }
+  GST_DEBUG_OBJECT (ffmpegdec, "stored opaque values idx %d",
+      frame->system_frame_number);
 
   /* This might call into get_buffer() from another thread,
    * which would cause a deadlock. Release the lock here