baseparse: remove checks for buffer==NULL
authorStefan Kost <ensonic@users.sf.net>
Thu, 26 Feb 2009 09:02:06 +0000 (11:02 +0200)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Fri, 8 Apr 2011 17:06:57 +0000 (18:06 +0100)
Accordifn to docs for GstPadChainFunction buffer cannot be NULL. If we would
leave the check, we would also need more such check below.

gst/aacparse/gstbaseparse.c

index baa56826c00418a949e626c1882f443ccb217813..5f9a949ee02a96ad01e47870c35545dba23b8d2d 100644 (file)
@@ -838,7 +838,7 @@ gst_base_parse_chain (GstPad * pad, GstBuffer * buffer)
   parse = GST_BASE_PARSE (GST_OBJECT_PARENT (pad));
   bclass = GST_BASE_PARSE_GET_CLASS (parse);
 
-  if (parse->pending_segment) {
+  if (G_UNLIKELY (parse->pending_segment)) {
     GST_DEBUG_OBJECT (parse, "chain pushing a pending segment");
     gst_pad_push_event (parse->srcpad, parse->pending_segment);
     parse->pending_segment = NULL;
@@ -854,7 +854,7 @@ gst_base_parse_chain (GstPad * pad, GstBuffer * buffer)
     gst_adapter_clear (parse->adapter);
   }
 
-  if (parse->priv->pending_events) {
+  if (G_UNLIKELY (parse->priv->pending_events)) {
     GList *l;
 
     for (l = parse->priv->pending_events; l != NULL; l = l->next) {
@@ -864,13 +864,9 @@ gst_base_parse_chain (GstPad * pad, GstBuffer * buffer)
     parse->priv->pending_events = NULL;
   }
 
-  if (buffer) {
-    GST_LOG_OBJECT (parse, "buffer size: %d, offset = %lld",
-        GST_BUFFER_SIZE (buffer), GST_BUFFER_OFFSET (buffer));
-
-
-    gst_adapter_push (parse->adapter, buffer);
-  }
+  GST_LOG_OBJECT (parse, "buffer size: %d, offset = %lld",
+      GST_BUFFER_SIZE (buffer), GST_BUFFER_OFFSET (buffer));
+  gst_adapter_push (parse->adapter, buffer);
 
   /* Parse and push as many frames as possible */
   /* Stop either when adapter is empty or we are flushing */