videoparsers: adjust to standardized baseparse frame state tracking
authorMark Nauwelaerts <mark.nauwelaerts@collabora.co.uk>
Wed, 15 Feb 2012 10:59:37 +0000 (11:59 +0100)
committerMark Nauwelaerts <mark.nauwelaerts@collabora.co.uk>
Wed, 15 Feb 2012 16:15:07 +0000 (17:15 +0100)
gst/videoparsers/gsth264parse.c
gst/videoparsers/gstmpeg4videoparse.c
gst/videoparsers/gstmpegvideoparse.c

index 4ba67e133b5122af50e09972b441dba2e41daf70..f8d33fb6049964605d133d3094dda4b1827452ad 100644 (file)
@@ -639,10 +639,6 @@ gst_h264_parse_collect_nal (GstH264Parse * h264parse, const guint8 * data,
   return complete;
 }
 
-/* FIXME move into baseparse, or anything equivalent;
- * see https://bugzilla.gnome.org/show_bug.cgi?id=650093 */
-#define GST_BASE_PARSE_FRAME_FLAG_PARSING   0x10000
-
 static GstFlowReturn
 gst_h264_parse_handle_frame (GstBaseParse * parse,
     GstBaseParseFrame * frame, gint * skipsize)
@@ -675,10 +671,9 @@ gst_h264_parse_handle_frame (GstBaseParse * parse,
     gst_h264_parse_negotiate (h264parse, NULL);
 
   /* avoid stale cached parsing state */
-  if (!(frame->flags & GST_BASE_PARSE_FRAME_FLAG_PARSING)) {
+  if (frame->flags & GST_BASE_PARSE_FRAME_FLAG_NEW_FRAME) {
     GST_LOG_OBJECT (h264parse, "parsing new frame");
     gst_h264_parse_reset_frame (h264parse);
-    frame->flags |= GST_BASE_PARSE_FRAME_FLAG_PARSING;
   } else {
     GST_LOG_OBJECT (h264parse, "resuming frame parsing");
   }
index aae6a8a4a073c14dd772c26d4645f46a676945bf..d359439e43bfa296f515b2265f7f6afa17fbebff 100644 (file)
@@ -357,10 +357,6 @@ gst_mpeg4vparse_process_sc (GstMpeg4VParse * mp4vparse, GstMpeg4Packet * packet,
   return FALSE;
 }
 
-/* FIXME move into baseparse, or anything equivalent;
- * see https://bugzilla.gnome.org/show_bug.cgi?id=650093 */
-#define GST_BASE_PARSE_FRAME_FLAG_PARSING   0x10000
-
 static GstFlowReturn
 gst_mpeg4vparse_handle_frame (GstBaseParse * parse,
     GstBaseParseFrame * frame, gint * skipsize)
@@ -386,10 +382,9 @@ retry:
   }
 
   /* avoid stale cached parsing state */
-  if (!(frame->flags & GST_BASE_PARSE_FRAME_FLAG_PARSING)) {
+  if (frame->flags & GST_BASE_PARSE_FRAME_FLAG_NEW_FRAME) {
     GST_LOG_OBJECT (mp4vparse, "parsing new frame");
     gst_mpeg4vparse_reset_frame (mp4vparse);
-    frame->flags |= GST_BASE_PARSE_FRAME_FLAG_PARSING;
   } else {
     GST_LOG_OBJECT (mp4vparse, "resuming frame parsing");
   }
index dca3df5551a4fc6a7c62a2fec049f28c4edad83d..f0726b9606a4f8b495a2bbf2f8e550b2419952bf 100644 (file)
@@ -472,19 +472,14 @@ get_frame_size (GstMpegvParse * mpvparse, GstBuffer * buf, GList * l_codoffsz)
   return -1;
 }
 
-/* FIXME move into baseparse, or anything equivalent;
- * see https://bugzilla.gnome.org/show_bug.cgi?id=650093 */
-#define GST_BASE_PARSE_FRAME_FLAG_PARSING   0x10000
-
 static inline void
 update_frame_parsing_status (GstMpegvParse * mpvparse,
     GstBaseParseFrame * frame)
 {
   /* avoid stale cached parsing state */
-  if (!(frame->flags & GST_BASE_PARSE_FRAME_FLAG_PARSING)) {
+  if (frame->flags & GST_BASE_PARSE_FRAME_FLAG_NEW_FRAME) {
     GST_LOG_OBJECT (mpvparse, "parsing new frame");
     gst_mpegv_parse_reset_frame (mpvparse);
-    frame->flags |= GST_BASE_PARSE_FRAME_FLAG_PARSING;
   } else {
     GST_LOG_OBJECT (mpvparse, "resuming frame parsing");
   }