BaseTransform: Add a query vfunc
authorSjoerd Simons <sjoerd.simons@collabora.co.uk>
Thu, 25 Aug 2011 10:02:16 +0000 (11:02 +0100)
committerSebastian Dröge <sebastian.droege@collabora.co.uk>
Fri, 26 Aug 2011 08:39:44 +0000 (10:39 +0200)
libs/gst/base/gstbasetransform.c
libs/gst/base/gstbasetransform.h

index a46642e..debeccb 100644 (file)
@@ -350,6 +350,8 @@ static gboolean gst_base_transform_setcaps (GstPad * pad, GstCaps * caps);
 static GstFlowReturn gst_base_transform_buffer_alloc (GstPad * pad,
     guint64 offset, guint size, GstCaps * caps, GstBuffer ** buf);
 static gboolean gst_base_transform_query (GstPad * pad, GstQuery * query);
+static gboolean gst_base_transform_default_query (GstBaseTransform * trans,
+    GstPad * pad, GstQuery * query);
 static const GstQueryType *gst_base_transform_query_type (GstPad * pad);
 
 /* static guint gst_base_transform_signals[LAST_SIGNAL] = { 0 }; */
@@ -397,6 +399,7 @@ gst_base_transform_class_init (GstBaseTransformClass * klass)
   klass->src_event = GST_DEBUG_FUNCPTR (gst_base_transform_src_eventfunc);
   klass->accept_caps =
       GST_DEBUG_FUNCPTR (gst_base_transform_acceptcaps_default);
+  klass->query = GST_DEBUG_FUNCPTR (gst_base_transform_default_query);
 }
 
 static void
@@ -1234,15 +1237,12 @@ failed_configure:
 }
 
 static gboolean
-gst_base_transform_query (GstPad * pad, GstQuery * query)
+gst_base_transform_default_query (GstBaseTransform * trans,
+    GstPad * pad, GstQuery * query)
 {
   gboolean ret = FALSE;
-  GstBaseTransform *trans;
   GstPad *otherpad;
 
-  trans = GST_BASE_TRANSFORM (gst_pad_get_parent (pad));
-  if (G_UNLIKELY (trans == NULL))
-    return FALSE;
   otherpad = (pad == trans->srcpad) ? trans->sinkpad : trans->srcpad;
 
   switch (GST_QUERY_TYPE (query)) {
@@ -1274,7 +1274,29 @@ gst_base_transform_query (GstPad * pad, GstQuery * query)
       break;
   }
 
+  return ret;
+}
+
+static gboolean
+gst_base_transform_query (GstPad * pad, GstQuery * query)
+{
+  GstBaseTransform *trans;
+  GstBaseTransformClass *bclass;
+  gboolean ret;
+
+  trans = GST_BASE_TRANSFORM (gst_pad_get_parent (pad));
+  if (G_UNLIKELY (trans == NULL))
+    return FALSE;
+
+  bclass = GST_BASE_TRANSFORM_GET_CLASS (trans);
+
+  if (bclass->query)
+    ret = bclass->query (trans, pad, query);
+  else
+    ret = gst_pad_query_default (pad, query);
+
   gst_object_unref (trans);
+
   return ret;
 }
 
index 72c2b2a..8914260 100644 (file)
@@ -190,6 +190,10 @@ struct _GstBaseTransform {
  *               Subclasses can override this method to check if @caps can be
  *               handled by the element. The default implementation might not be
  *               the most optimal way to check this in all cases.
+ * @query: Optional Since 0.10.36
+ *                Handle a requested query. Subclasses that implement this
+ *                should must chain up to the parent if they didn't handle the
+ *                query
  *
  * Subclasses can override any of the available virtual methods or not, as
  * needed. At minimum either @transform or @transform_ip need to be overridden.
@@ -243,9 +247,10 @@ struct _GstBaseTransformClass {
 
   gboolean      (*accept_caps)  (GstBaseTransform *trans, GstPadDirection direction,
                                          GstCaps *caps);
+  gboolean      (*query)  (GstBaseTransform *trans, GstPad *pad, GstQuery *query);
 
   /*< private >*/
-  gpointer       _gst_reserved[GST_PADDING_LARGE - 3];
+  gpointer       _gst_reserved[GST_PADDING_LARGE - 4];
 };
 
 GType           gst_base_transform_get_type         (void);