* Boston, MA 02111-1307, USA.
*/
-/* #define GST_DEBUG_ENABLED */
#include "gst_private.h"
#include "gstpad.h"
#include "gstbin.h"
#include "gstscheduler.h"
#include "gstevent.h"
-#include "gstlog.h"
+#include "gstinfo.h"
enum {
TEMPL_PAD_CREATED,
/***** Start with the base GstPad class *****/
static void gst_pad_class_init (GstPadClass *klass);
static void gst_pad_init (GstPad *pad);
+static void gst_pad_dispose (GObject *object);
static gboolean gst_pad_try_relink_filtered_func (GstRealPad *srcpad, GstRealPad *sinkpad,
GstCaps *caps, gboolean clear);
+static void gst_pad_set_pad_template (GstPad *pad, GstPadTemplate *templ);
#ifndef GST_DISABLE_LOADSAVE
static xmlNodePtr gst_pad_save_thyself (GstObject *object, xmlNodePtr parent);
static void
gst_pad_class_init (GstPadClass *klass)
{
+ GObjectClass *gobject_class;
+
+ gobject_class = (GObjectClass*) klass;
+
pad_parent_class = g_type_class_ref (GST_TYPE_OBJECT);
+
+ gobject_class->dispose = GST_DEBUG_FUNCPTR (gst_pad_dispose);
}
static void
gst_pad_init (GstPad *pad)
{
- pad->element_private = NULL;
+ /* all structs are initialized to NULL by glib */
+}
+static void
+gst_pad_dispose (GObject *object)
+{
+ GstPad *pad = GST_PAD (object);
- pad->padtemplate = NULL;
+ gst_pad_set_pad_template (pad, NULL);
+
+ G_OBJECT_CLASS (pad_parent_class)->dispose (object);
}
static void gst_real_pad_class_init (GstRealPadClass *klass);
static void gst_real_pad_init (GstRealPad *pad);
+static void gst_real_pad_dispose (GObject *object);
static void gst_real_pad_set_property (GObject *object, guint prop_id,
const GValue *value,
GValue *value,
GParamSpec *pspec);
-static void gst_real_pad_dispose (GObject *object);
-
GType _gst_real_pad_type = 0;
static GstPad *real_pad_parent_class = NULL;
g_return_val_if_fail (templ != NULL, NULL);
pad = gst_pad_new (name, templ->direction);
-
- gst_object_ref (GST_OBJECT (templ));
- GST_PAD_PAD_TEMPLATE (pad) = templ;
+ gst_pad_set_pad_template (pad, templ);
- g_signal_emit (G_OBJECT (templ), gst_pad_template_signals[TEMPL_PAD_CREATED], 0, pad);
-
return pad;
}
realpad = GST_PAD_REALIZE (pad);
if (active) {
- GST_DEBUG (GST_CAT_PADS, "activating pad %s:%s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "activating pad %s:%s",
GST_DEBUG_PAD_NAME (realpad));
GST_FLAG_UNSET (realpad, GST_PAD_DISABLED);
} else {
- GST_DEBUG (GST_CAT_PADS, "de-activating pad %s:%s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "de-activating pad %s:%s",
GST_DEBUG_PAD_NAME (realpad));
GST_FLAG_SET (realpad, GST_PAD_DISABLED);
}
{
g_return_if_fail (pad != NULL);
g_return_if_fail (GST_IS_REAL_PAD (pad));
+ g_return_if_fail (GST_RPAD_DIRECTION (pad) == GST_PAD_SINK);
GST_RPAD_CHAINFUNC (pad) = chain;
- GST_DEBUG (GST_CAT_PADS, "chainfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "chainfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (chain));
}
{
g_return_if_fail (pad != NULL);
g_return_if_fail (GST_IS_REAL_PAD (pad));
+ g_return_if_fail (GST_RPAD_DIRECTION (pad) == GST_PAD_SRC);
GST_RPAD_GETFUNC (pad) = get;
- GST_DEBUG (GST_CAT_PADS, "getfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "getfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (get));
}
GstPadEventFunction event)
{
g_return_if_fail (GST_IS_REAL_PAD (pad));
+ g_return_if_fail (GST_RPAD_DIRECTION (pad) == GST_PAD_SRC);
GST_RPAD_EVENTFUNC (pad) = event;
- GST_DEBUG (GST_CAT_PADS, "eventfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "eventfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (event));
}
GST_RPAD_EVENTMASKFUNC (pad) = mask_func;
- GST_DEBUG (GST_CAT_PADS, "eventmaskfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "eventmaskfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (mask_func));
}
GST_RPAD_CONVERTFUNC (pad) = convert;
- GST_DEBUG (GST_CAT_PADS, "convertfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "convertfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (convert));
}
GST_RPAD_QUERYFUNC (pad) = query;
- GST_DEBUG (GST_CAT_PADS, "queryfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "queryfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (query));
}
GST_RPAD_QUERYTYPEFUNC (pad) = type_func;
- GST_DEBUG (GST_CAT_PADS, "querytypefunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "querytypefunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (type_func));
}
g_return_if_fail (GST_IS_REAL_PAD (pad));
GST_RPAD_INTLINKFUNC (pad) = intlink;
- GST_DEBUG (GST_CAT_PADS, "internal link for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "internal link for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (intlink));
}
g_return_if_fail (GST_IS_REAL_PAD (pad));
GST_RPAD_FORMATSFUNC (pad) = formats;
- GST_DEBUG (GST_CAT_PADS, "formats function for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "formats function for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (formats));
}
g_return_if_fail (GST_IS_REAL_PAD (pad));
GST_RPAD_LINKFUNC (pad) = link;
- GST_DEBUG (GST_CAT_PADS, "linkfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "linkfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (link));
}
/**
+ * gst_pad_set_unlink_function:
+ * @pad: a #GstPad to set the unlink function for.
+ * @unlink: the #GstPadUnlinkFunction to set.
+ *
+ * Sets the given unlink function for the pad. It will be called
+ * when the pad is unlinked.
+ */
+void
+gst_pad_set_unlink_function (GstPad *pad,
+ GstPadUnlinkFunction unlink)
+{
+ g_return_if_fail (pad != NULL);
+ g_return_if_fail (GST_IS_REAL_PAD (pad));
+
+ GST_RPAD_UNLINKFUNC (pad) = unlink;
+ GST_CAT_DEBUG (GST_CAT_PADS, "unlinkfunc for %s:%s set to %s",
+ GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (unlink));
+}
+
+/**
* gst_pad_set_getcaps_function:
* @pad: a #GstPad to set the getcaps function for.
* @getcaps: the #GstPadGetCapsFunction to set.
g_return_if_fail (GST_IS_REAL_PAD (pad));
GST_RPAD_GETCAPSFUNC (pad) = getcaps;
- GST_DEBUG (GST_CAT_PADS, "getcapsfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "getcapsfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (getcaps));
}
/**
g_return_if_fail (GST_PAD_IS_SINK (pad));
GST_RPAD_BUFFERPOOLFUNC (pad) = bufpool;
- GST_DEBUG (GST_CAT_PADS, "bufferpoolfunc for %s:%s set to %s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "bufferpoolfunc for %s:%s set to %s",
GST_DEBUG_PAD_NAME (pad), GST_DEBUG_FUNCPTR_NAME (bufpool));
}
g_return_if_fail (sinkpad != NULL);
g_return_if_fail (GST_IS_PAD (sinkpad));
- GST_INFO (GST_CAT_ELEMENT_PADS, "unlinking %s:%s(%p) and %s:%s(%p)",
+ GST_CAT_INFO (GST_CAT_ELEMENT_PADS, "unlinking %s:%s(%p) and %s:%s(%p)",
GST_DEBUG_PAD_NAME (srcpad), srcpad,
GST_DEBUG_PAD_NAME (sinkpad), sinkpad);
g_return_if_fail ((GST_RPAD_DIRECTION (realsrc) == GST_PAD_SRC) &&
(GST_RPAD_DIRECTION (realsink) == GST_PAD_SINK));
+ if (GST_RPAD_UNLINKFUNC (srcpad)) {
+ GST_RPAD_UNLINKFUNC (srcpad) (srcpad);
+ }
+ if (GST_RPAD_UNLINKFUNC (sinkpad)) {
+ GST_RPAD_UNLINKFUNC (sinkpad) (sinkpad);
+ }
+
/* get the schedulers before we unlink */
src_sched = gst_pad_get_scheduler (GST_PAD_CAST (realsrc));
sink_sched = gst_pad_get_scheduler (GST_PAD_CAST (realsink));
/* reset the filters, both filters are refcounted once */
if (GST_RPAD_FILTER (realsrc)) {
- gst_caps_unref (GST_RPAD_FILTER (realsrc));
- GST_RPAD_FILTER (realsink) = NULL;
- GST_RPAD_FILTER (realsrc) = NULL;
+ gst_caps_replace (&GST_RPAD_FILTER (realsink), NULL);
+ gst_caps_replace (&GST_RPAD_FILTER (realsrc), NULL);
}
/* now tell the scheduler */
if (src_sched && src_sched == sink_sched) {
gst_scheduler_pad_unlink (src_sched,
- GST_PAD_CAST (realsrc), GST_PAD_CAST (realsink));
+ GST_PAD_CAST (realsrc),
+ GST_PAD_CAST (realsink));
}
/* hold a reference, as they can go away in the signal handlers */
g_signal_emit (G_OBJECT (realsink), gst_real_pad_signals[REAL_UNLINKED],
0, realsrc);
- GST_INFO (GST_CAT_ELEMENT_PADS, "unlinked %s:%s and %s:%s",
+ GST_CAT_INFO (GST_CAT_ELEMENT_PADS, "unlinked %s:%s and %s:%s",
GST_DEBUG_PAD_NAME (srcpad), GST_DEBUG_PAD_NAME (sinkpad));
gst_object_unref (GST_OBJECT (realsrc));
*/
gboolean
gst_pad_can_link_filtered (GstPad *srcpad, GstPad *sinkpad,
- GstCaps *filtercaps)
+ GstCaps *filtercaps)
{
GstRealPad *realsrc, *realsink;
* @filtercaps: the filter #GstCaps.
*
* Links the source pad and the sink pad, constrained
- * by the given filter caps.
+ * by the given filter caps. This function sinks the caps.
*
* Returns: TRUE if the pads have been linked, FALSE otherwise.
*/
g_return_val_if_fail (sinkpad != NULL, FALSE);
g_return_val_if_fail (GST_IS_PAD (sinkpad), FALSE);
- GST_INFO (GST_CAT_PADS, "trying to link %s:%s and %s:%s",
+ GST_CAT_INFO (GST_CAT_PADS, "trying to link %s:%s and %s:%s",
GST_DEBUG_PAD_NAME (srcpad), GST_DEBUG_PAD_NAME (sinkpad));
/* now we need to deal with the real/ghost stuff */
realsink = GST_PAD_REALIZE (sinkpad);
if ((GST_PAD (realsrc) != srcpad) || (GST_PAD (realsink) != sinkpad)) {
- GST_INFO (GST_CAT_PADS, "*actually* linking %s:%s and %s:%s",
+ GST_CAT_INFO (GST_CAT_PADS, "*actually* linking %s:%s and %s:%s",
GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink));
}
+ /* FIXME: shouldn't we convert this to g_return_val_if_fail? */
if (GST_RPAD_PEER (realsrc) != NULL) {
- GST_INFO (GST_CAT_PADS, "Real source pad %s:%s has a peer, failed",
+ GST_CAT_INFO (GST_CAT_PADS, "Real source pad %s:%s has a peer, failed",
GST_DEBUG_PAD_NAME (realsrc));
return FALSE;
}
if (GST_RPAD_PEER (realsink) != NULL) {
- GST_INFO (GST_CAT_PADS, "Real sink pad %s:%s has a peer, failed",
+ GST_CAT_INFO (GST_CAT_PADS, "Real sink pad %s:%s has a peer, failed",
GST_DEBUG_PAD_NAME (realsink));
return FALSE;
}
if (GST_PAD_PARENT (realsrc) == NULL) {
- GST_INFO (GST_CAT_PADS, "Real src pad %s:%s has no parent, failed",
+ GST_CAT_INFO (GST_CAT_PADS, "Real src pad %s:%s has no parent, failed",
GST_DEBUG_PAD_NAME (realsrc));
return FALSE;
}
if (GST_PAD_PARENT (realsink) == NULL) {
- GST_INFO (GST_CAT_PADS, "Real src pad %s:%s has no parent, failed",
+ GST_CAT_INFO (GST_CAT_PADS, "Real sink pad %s:%s has no parent, failed",
GST_DEBUG_PAD_NAME (realsrc));
return FALSE;
}
realsink = temppad;
}
if (GST_RPAD_DIRECTION (realsrc) != GST_PAD_SRC) {
- GST_INFO (GST_CAT_PADS, "Real src pad %s:%s is not a source pad, failed",
+ GST_CAT_INFO (GST_CAT_PADS, "Real src pad %s:%s is not a source pad, failed",
GST_DEBUG_PAD_NAME (realsrc));
return FALSE;
}
if (GST_RPAD_DIRECTION (realsink) != GST_PAD_SINK) {
- GST_INFO (GST_CAT_PADS, "Real sink pad %s:%s is not a sink pad, failed",
+ GST_CAT_INFO (GST_CAT_PADS, "Real sink pad %s:%s is not a sink pad, failed",
GST_DEBUG_PAD_NAME (realsink));
return FALSE;
}
/* try to negotiate the pads, we don't need to clear the caps here */
if (!gst_pad_try_relink_filtered_func (realsrc, realsink,
- filtercaps, FALSE)) {
- GST_DEBUG (GST_CAT_CAPS, "relink_filtered_func failed, can't link");
+ filtercaps, FALSE)) {
+ GST_CAT_DEBUG (GST_CAT_CAPS, "relink_filtered_func failed, can't link");
GST_RPAD_PEER (realsrc) = NULL;
GST_RPAD_PEER (realsink) = NULL;
/* now tell the scheduler */
if (src_sched && src_sched == sink_sched) {
gst_scheduler_pad_link (src_sched,
- GST_PAD_CAST (realsrc), GST_PAD_CAST (realsink));
+ GST_PAD_CAST (realsrc), GST_PAD_CAST (realsink));
+ }
+ else {
+ GST_CAT_INFO (GST_CAT_PADS, "not telling link to scheduler %s:%s and %s:%s, %p %p",
+ GST_DEBUG_PAD_NAME (srcpad), GST_DEBUG_PAD_NAME (sinkpad),
+ src_sched, sink_sched);
}
- GST_INFO (GST_CAT_PADS, "linked %s:%s and %s:%s, successful",
+ GST_CAT_INFO (GST_CAT_PADS, "linked %s:%s and %s:%s, successful",
GST_DEBUG_PAD_NAME (srcpad), GST_DEBUG_PAD_NAME (sinkpad));
- gst_caps_debug (gst_pad_get_caps (GST_PAD_CAST (realsrc)),
- "caps of newly linked src pad");
return TRUE;
}
return GST_PAD_PARENT (pad);
}
+static void
+gst_pad_set_pad_template (GstPad *pad, GstPadTemplate *templ)
+{
+ /* this function would need checks if it weren't static */
+
+ gst_object_replace ((GstObject **) &pad->padtemplate, (GstObject *) templ);
+
+ if (templ)
+ g_signal_emit (G_OBJECT (templ), gst_pad_template_signals[TEMPL_PAD_CREATED], 0, pad);
+}
/**
* gst_pad_get_pad_template:
* @pad: a #GstPad to get the pad template of.
{
GstRealPad *realpad;
- g_return_if_fail (pad != NULL);
g_return_if_fail (GST_IS_PAD (pad));
- g_return_if_fail (ghostpad != NULL);
g_return_if_fail (GST_IS_GHOST_PAD (ghostpad));
-
realpad = GST_PAD_REALIZE (pad);
+ g_return_if_fail (GST_GPAD_REALPAD (ghostpad) == realpad);
realpad->ghostpads = g_list_remove (realpad->ghostpads, ghostpad);
+ GST_GPAD_REALPAD (ghostpad) = NULL;
}
/**
static GstPadLinkReturn
gst_pad_try_set_caps_func (GstRealPad *pad, GstCaps *caps, gboolean notify)
{
- GstCaps *oldcaps, *allowed = NULL;
+ GstCaps *allowed = NULL;
GstPadTemplate *template;
GstElement *parent = GST_PAD_PARENT (pad);
* negotiation */
if (parent && GST_STATE (parent) < GST_STATE_READY)
{
- GST_DEBUG (GST_CAT_CAPS, "parent %s of pad %s:%s is not READY",
+ GST_CAT_DEBUG (GST_CAT_CAPS, "parent %s of pad %s:%s is not READY",
GST_ELEMENT_NAME (parent), GST_DEBUG_PAD_NAME (pad));
return GST_PAD_LINK_DELAYED;
}
- GST_INFO (GST_CAT_CAPS, "trying to set caps %p on pad %s:%s",
+ GST_CAT_INFO (GST_CAT_CAPS, "trying to set caps %p on pad %s:%s",
caps, GST_DEBUG_PAD_NAME (pad));
- /* first see if we have to check against a filter */
- if (!(allowed = GST_RPAD_FILTER (pad))) {
+ /* first see if we have to check against a filter, we ref the caps here as we're
+ * going to unref it later on */
+ if (!(allowed = gst_caps_ref (GST_RPAD_FILTER (pad)))) {
/* no filter, make sure we check against the padtemplate then */
if ((template = gst_pad_get_pad_template (GST_PAD_CAST (pad)))) {
allowed = gst_pad_template_get_caps (template);
/* oops, empty intersection, caps don"t have anything in common */
if (!intersection) {
- GST_INFO (GST_CAT_CAPS, "caps did not intersect with %s:%s's allowed caps",
+ GST_CAT_INFO (GST_CAT_CAPS, "caps did not intersect with %s:%s's allowed caps",
GST_DEBUG_PAD_NAME (pad));
gst_caps_debug (caps, "caps themselves (attemped to set)");
gst_caps_debug (allowed,
"allowed caps that did not agree with caps");
+ gst_caps_unref (allowed);
return GST_PAD_LINK_REFUSED;
}
/* caps checks out fine, we can unref the intersection now */
gst_caps_unref (intersection);
+ gst_caps_unref (allowed);
/* given that the caps are fixed, we know that their intersection with the
* padtemplate caps is the same as caps itself */
}
gchar *debug_string;
gboolean negotiating;
- GST_INFO (GST_CAT_CAPS, "calling link function on pad %s:%s",
+ GST_CAT_INFO (GST_CAT_CAPS, "calling link function on pad %s:%s",
GST_DEBUG_PAD_NAME (pad));
negotiating = GST_FLAG_IS_SET (pad, GST_PAD_NEGOTIATING);
return GST_PAD_LINK_REFUSED;
}
- GST_INFO (GST_CAT_CAPS,
+ GST_CAT_INFO (GST_CAT_CAPS,
"got reply %s (%d) from link function on pad %s:%s",
debug_string, res, GST_DEBUG_PAD_NAME (pad));
/* done means the link function called another caps negotiate function
* on this pad that succeeded, we dont need to continue */
if (res == GST_PAD_LINK_DONE) {
- GST_INFO (GST_CAT_CAPS, "pad %s:%s is done", GST_DEBUG_PAD_NAME (pad));
+ GST_CAT_INFO (GST_CAT_CAPS, "pad %s:%s is done", GST_DEBUG_PAD_NAME (pad));
return GST_PAD_LINK_DONE;
}
if (res == GST_PAD_LINK_REFUSED) {
- GST_INFO (GST_CAT_CAPS, "pad %s:%s doesn't accept caps",
+ GST_CAT_INFO (GST_CAT_CAPS, "pad %s:%s doesn't accept caps",
GST_DEBUG_PAD_NAME (pad));
return GST_PAD_LINK_REFUSED;
}
/* we can only set caps on the pad if they are fixed */
if (GST_CAPS_IS_FIXED (caps)) {
- GST_INFO (GST_CAT_CAPS, "setting caps on pad %s:%s",
+ GST_CAT_INFO (GST_CAT_CAPS, "setting caps on pad %s:%s",
GST_DEBUG_PAD_NAME (pad));
/* if we got this far all is ok, remove the old caps, set the new one */
- oldcaps = GST_PAD_CAPS (pad);
- if (caps) gst_caps_ref (caps);
- GST_PAD_CAPS (pad) = caps;
- if (oldcaps) gst_caps_unref (oldcaps);
+ gst_caps_replace_sink (&GST_PAD_CAPS (pad), caps);
g_object_notify (G_OBJECT (pad), "caps");
}
else {
- GST_INFO (GST_CAT_CAPS,
+ GST_CAT_INFO (GST_CAT_CAPS,
"caps are not fixed on pad %s:%s, not setting them yet",
GST_DEBUG_PAD_NAME (pad));
+
+ return GST_PAD_LINK_DELAYED;
}
return GST_PAD_LINK_OK;
}
* @pad: a #GstPad to try to set the caps on.
* @caps: the #GstCaps to set.
*
- * Tries to set the caps on the given pad.
+ * Tries to set the caps on the given pad. Ownership is always taken
+ * of the caps, so you will need to unref non-floating caps.
*
* Returns: A #GstPadLinkReturn value indicating whether the caps
* could be set.
realpad = GST_PAD_REALIZE (pad);
peer = GST_RPAD_PEER (realpad);
- GST_INFO (GST_CAT_CAPS, "trying to set caps %p on pad %s:%s",
+ GST_CAT_INFO (GST_CAT_CAPS, "trying to set caps %p on pad %s:%s",
caps, GST_DEBUG_PAD_NAME (realpad));
gst_caps_debug (caps, "caps that we are trying to set");
+ /* try to take ownership */
+ gst_caps_ref (caps);
+ gst_caps_sink (caps);
+
/* setting non fixed caps on a pad is not allowed */
if (!GST_CAPS_IS_FIXED (caps)) {
- GST_INFO (GST_CAT_CAPS,
- "trying to set unfixed caps on pad %s:%s, not allowed",
- GST_DEBUG_PAD_NAME (realpad));
+ GST_CAT_INFO (GST_CAT_CAPS,
+ "trying to set unfixed caps on pad %s:%s, not allowed",
+ GST_DEBUG_PAD_NAME (realpad));
g_warning ("trying to set non fixed caps on pad %s:%s, not allowed",
GST_DEBUG_PAD_NAME (realpad));
+
gst_caps_debug (caps, "unfixed caps");
- return GST_PAD_LINK_DELAYED;
+ set_retval = GST_PAD_LINK_DELAYED;
+ goto done;
}
/* if we have a peer try to set the caps, notifying the peerpad
* if it has a link function */
if (peer && ((set_retval = gst_pad_try_set_caps_func (peer, caps, TRUE)) <= 0))
{
- GST_INFO (GST_CAT_CAPS, "tried to set caps on peerpad %s:%s but couldn't, return value %d",
+ GST_CAT_INFO (GST_CAT_CAPS, "tried to set caps on peerpad %s:%s but couldn't, return value %d",
GST_DEBUG_PAD_NAME (peer), set_retval);
- return set_retval;
+ goto done;
}
/* then try to set our own caps, we don't need to be notified */
if ((set_retval = gst_pad_try_set_caps_func (realpad, caps, FALSE)) <= 0)
{
- GST_INFO (GST_CAT_CAPS, "tried to set own caps on pad %s:%s but couldn't, return value %d",
+ GST_CAT_INFO (GST_CAT_CAPS, "tried to set own caps on pad %s:%s but couldn't, return value %d",
GST_DEBUG_PAD_NAME (realpad), set_retval);
- return set_retval;
+ goto done;
}
- GST_INFO (GST_CAT_CAPS, "succeeded setting caps %p on pad %s:%s, return value %d",
+ GST_CAT_INFO (GST_CAT_CAPS, "succeeded setting caps %p on pad %s:%s, return value %d",
caps, GST_DEBUG_PAD_NAME (realpad), set_retval);
g_assert (GST_PAD_CAPS (pad));
-
+
+done:
+ /* if we took ownership, the caps will be freed */
+ gst_caps_unref (caps);
+
return set_retval;
}
*/
static gboolean
gst_pad_try_relink_filtered_func (GstRealPad *srcpad, GstRealPad *sinkpad,
- GstCaps *filtercaps, gboolean clear)
+ GstCaps *filtercaps, gboolean clear)
{
GstCaps *srccaps, *sinkcaps;
GstCaps *intersection = NULL;
/* optinally clear the caps */
if (clear) {
- GST_INFO (GST_CAT_PADS,
+ GST_CAT_INFO (GST_CAT_PADS,
"start relink filtered %s:%s and %s:%s, clearing caps",
GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink));
- GST_PAD_CAPS (GST_PAD (realsrc)) = NULL;
- GST_PAD_CAPS (GST_PAD (realsink)) = NULL;
+ gst_caps_replace (&GST_PAD_CAPS (GST_PAD (realsrc)), NULL);
+ gst_caps_replace (&GST_PAD_CAPS (GST_PAD (realsink)), NULL);
+ gst_caps_replace (&GST_RPAD_FILTER (realsrc), NULL);
+ gst_caps_replace (&GST_RPAD_FILTER (realsink), NULL);
}
else {
- GST_INFO (GST_CAT_PADS, "start relink filtered %s:%s and %s:%s",
+ GST_CAT_INFO (GST_CAT_PADS, "start relink filtered %s:%s and %s:%s",
GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink));
}
srccaps = gst_pad_get_caps (GST_PAD (realsrc));
- GST_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
GST_DEBUG_PAD_NAME (realsrc));
gst_caps_debug (srccaps, "caps of src pad (pre-relink)");
sinkcaps = gst_pad_get_caps (GST_PAD (realsink));
- GST_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
GST_DEBUG_PAD_NAME (realsink));
gst_caps_debug (sinkcaps, "caps of sink pad (pre-relink)");
if (!intersection && (srccaps || sinkcaps)) {
/* the intersection is NULL but the pad caps were not both NULL,
* this means they have no common format */
- GST_INFO (GST_CAT_PADS, "pads %s:%s and %s:%s have no common type",
+ GST_CAT_INFO (GST_CAT_PADS, "pads %s:%s and %s:%s have no common type",
GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink));
+ /* make sure any floating caps from gst_pad_get_caps are freed here */
+ gst_caps_sink (srccaps);
+ gst_caps_sink (sinkcaps);
return FALSE;
} else {
- GST_INFO (GST_CAT_PADS, "pads %s:%s and %s:%s intersected to %s caps",
+ GST_CAT_INFO (GST_CAT_PADS, "pads %s:%s and %s:%s intersected to %s caps",
GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink),
(intersection ?
(GST_CAPS_IS_FIXED (intersection) ? "fixed" : "variable") :
"NULL"));
+ /* we don't need those anymore, as the caps can be floating */
+ gst_caps_sink (srccaps);
+ gst_caps_sink (sinkcaps);
+
/* then filter this against the app filter */
if (filtercaps) {
- GstCaps *filtered_intersection = gst_caps_intersect (intersection,
- filtercaps);
+ GstCaps *filtered_intersection;
+
+ filtered_intersection = gst_caps_intersect (intersection,
+ filtercaps);
- /* get rid of the old intersection here */
- gst_caps_unref (intersection);
+ gst_caps_sink (intersection);
if (!filtered_intersection) {
- GST_INFO (GST_CAT_PADS,
+ GST_CAT_INFO (GST_CAT_PADS,
"filtered link between pads %s:%s and %s:%s is empty",
GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink));
return FALSE;
intersection = filtered_intersection;
/* keep a reference to the app caps */
- GST_RPAD_APPFILTER (realsink) = filtercaps;
- GST_RPAD_APPFILTER (realsrc) = filtercaps;
+ gst_caps_replace_sink (&GST_RPAD_APPFILTER (realsink), filtercaps);
+ gst_caps_replace_sink (&GST_RPAD_APPFILTER (realsrc), filtercaps);
}
}
- GST_DEBUG (GST_CAT_CAPS, "setting filter for link to:");
+ GST_CAT_DEBUG (GST_CAT_CAPS, "setting filter for link to:");
gst_caps_debug (intersection, "filter for link");
/* both the app filter and the filter, while stored on both peer pads,
* are equal to the same thing on both */
- GST_RPAD_FILTER (realsrc) = intersection;
- GST_RPAD_FILTER (realsink) = intersection;
+ gst_caps_replace_sink (&GST_RPAD_FILTER (realsrc), intersection);
+ gst_caps_replace_sink (&GST_RPAD_FILTER (realsink), intersection);
+ gst_caps_sink (intersection);
return gst_pad_perform_negotiate (GST_PAD (realsrc), GST_PAD (realsink));
}
GstCaps *intersection, *filtered_intersection;
GstRealPad *realsrc, *realsink;
GstCaps *srccaps, *sinkcaps, *filter;
+ gboolean res = TRUE;
+ GstElement *parent;
+
g_return_val_if_fail (srcpad != NULL, FALSE);
g_return_val_if_fail (sinkpad != NULL, FALSE);
g_return_val_if_fail (GST_RPAD_PEER (realsrc) != NULL, FALSE);
g_return_val_if_fail (GST_RPAD_PEER (realsink) == realsrc, FALSE);
+ /* shortcut negotiation */
+ parent = GST_PAD_PARENT (realsrc);
+ if (parent && GST_STATE (parent) < GST_STATE_READY) {
+ GST_CAT_DEBUG (GST_CAT_CAPS, "parent %s of pad %s:%s is not READY",
+ GST_ELEMENT_NAME (parent), GST_DEBUG_PAD_NAME (realsrc));
+ return TRUE;
+ }
+ parent = GST_PAD_PARENT (realsink);
+ if (parent && GST_STATE (parent) < GST_STATE_READY) {
+ GST_CAT_DEBUG (GST_CAT_CAPS, "parent %s of pad %s:%s is not READY",
+ GST_ELEMENT_NAME (parent), GST_DEBUG_PAD_NAME (realsink));
+ return TRUE;
+ }
+
+ GST_CAT_INFO (GST_CAT_PADS, "perform negotiate for link %s:%s-%s:%s",
+ GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink));
+
filter = GST_RPAD_APPFILTER (realsrc);
if (filter) {
- GST_INFO (GST_CAT_PADS, "dumping filter for link %s:%s-%s:%s",
+ GST_CAT_INFO (GST_CAT_PADS, "dumping filter for link %s:%s-%s:%s",
GST_DEBUG_PAD_NAME (realsrc), GST_DEBUG_PAD_NAME (realsink));
gst_caps_debug (filter, "link filter caps");
}
/* calculate the new caps here */
srccaps = gst_pad_get_caps (GST_PAD (realsrc));
- GST_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
GST_DEBUG_PAD_NAME (realsrc));
gst_caps_debug (srccaps,
"src caps, awaiting negotiation, after applying filter");
sinkcaps = gst_pad_get_caps (GST_PAD (realsink));
- GST_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
+ GST_CAT_DEBUG (GST_CAT_PADS, "dumping caps of pad %s:%s",
GST_DEBUG_PAD_NAME (realsink));
gst_caps_debug (sinkcaps,
"sink caps, awaiting negotiation, after applying filter");
intersection = gst_caps_intersect (srccaps, sinkcaps);
filtered_intersection = gst_caps_intersect (intersection, filter);
- if (filtered_intersection) {
- gst_caps_unref (intersection);
- intersection = filtered_intersection;
- }
+ gst_caps_unref (intersection);
/* no negotiation is performed if the pads have filtercaps */
- if (intersection) {
- GstPadLinkReturn res;
+ if (filtered_intersection) {
+ GstPadLinkReturn link_res;
- res = gst_pad_try_set_caps_func (realsrc, intersection, TRUE);
- if (res == GST_PAD_LINK_REFUSED)
- return FALSE;
- if (res == GST_PAD_LINK_DONE)
- return TRUE;
+ link_res = gst_pad_try_set_caps_func (realsrc, filtered_intersection, TRUE);
+ if (link_res == GST_PAD_LINK_REFUSED)
+ goto error;
+ if (link_res == GST_PAD_LINK_DONE)
+ goto success;
- res = gst_pad_try_set_caps_func (realsink, intersection, TRUE);
- if (res == GST_PAD_LINK_REFUSED)
- return FALSE;
- if (res == GST_PAD_LINK_DONE)
- return TRUE;
+ link_res = gst_pad_try_set_caps_func (realsink, filtered_intersection, TRUE);
+ if (link_res == GST_PAD_LINK_REFUSED)
+ goto error;
+ if (link_res == GST_PAD_LINK_DONE)
+ goto success;
}
- return TRUE;
+ /* no filtered_intersection, some pads had caps and ther was a filter */
+ else if ((srccaps || sinkcaps) && filter) {
+ goto error;
+ }
+
+success:
+cleanup:
+ gst_caps_sink (srccaps);
+ gst_caps_sink (sinkcaps);
+ gst_caps_unref (filtered_intersection);
+ return res;
+
+error:
+ res = FALSE;
+ goto cleanup;
}
/**
g_return_val_if_fail (GST_RPAD_PEER (realsink) == realsrc, FALSE);
return gst_pad_try_relink_filtered_func (realsrc, realsink,
- filtercaps, TRUE);
+ filtercaps, TRUE);
}
/**
g_return_val_if_fail (GST_RPAD_PEER (realsrc) != NULL, FALSE);
g_return_val_if_fail (GST_RPAD_PEER (realsink) == realsrc, FALSE);
- if (! gst_pad_try_relink_filtered_func (realsrc, realsink,
- filtercaps, TRUE)) {
+ if (!gst_pad_try_relink_filtered_func (realsrc, realsink,
+ filtercaps, TRUE))
+ {
gst_pad_unlink (srcpad, GST_PAD (GST_PAD_PEER (srcpad)));
return FALSE;
}
peer = GST_RPAD_PEER (realpad);
- GST_INFO (GST_CAT_CAPS, "proxy link to pad %s:%s",
+ GST_CAT_INFO (GST_CAT_CAPS, "proxy link to pad %s:%s",
GST_DEBUG_PAD_NAME (realpad));
if (peer && gst_pad_try_set_caps_func (peer, caps, TRUE) < 0)
if (gst_pad_try_set_caps_func (realpad, caps, FALSE) < 0)
return GST_PAD_LINK_REFUSED;
+ if (peer) {
+ gst_caps_debug (caps, "proxy link filter");
+
+ GST_CAT_INFO (GST_CAT_CAPS, "setting filter on %s:%s and %s:%s",
+ GST_DEBUG_PAD_NAME (peer), GST_DEBUG_PAD_NAME (realpad));
+
+ gst_caps_replace_sink (&GST_RPAD_FILTER (peer), caps);
+ gst_caps_replace_sink (&GST_RPAD_FILTER (realpad), caps);
+ }
+
return GST_PAD_LINK_OK;
}
*
* Gets the capabilities of this pad.
*
- * Returns: the #GstCaps of this pad.
+ * Returns: the #GstCaps of this pad. This function potentially
+ * returns a floating caps, so use gst_caps_sink to get rid of
+ * it.
*/
GstCaps*
gst_pad_get_caps (GstPad *pad)
realpad = GST_PAD_REALIZE (pad);
- GST_DEBUG (GST_CAT_CAPS, "get pad caps of %s:%s (%p)",
+ GST_CAT_DEBUG (GST_CAT_CAPS, "get pad caps of %s:%s (%p)",
GST_DEBUG_PAD_NAME (realpad), realpad);
+ /* note that we will not _ref the caps here as this function might be
+ * called recursively */
if (GST_PAD_CAPS (realpad)) {
- GST_DEBUG (GST_CAT_CAPS, "using pad real caps %p", GST_PAD_CAPS (realpad));
+ GST_CAT_DEBUG (GST_CAT_CAPS, "using pad real caps %p", GST_PAD_CAPS (realpad));
return GST_PAD_CAPS (realpad);
}
else if GST_RPAD_GETCAPSFUNC (realpad) {
- GST_DEBUG (GST_CAT_CAPS, "using pad get function");
- return GST_RPAD_GETCAPSFUNC (realpad) (GST_PAD_CAST (realpad), NULL);
+ GstCaps *caps;
+
+ GST_CAT_DEBUG (GST_CAT_CAPS, "using pad get function");
+ caps = GST_RPAD_GETCAPSFUNC (realpad) (GST_PAD_CAST (realpad), NULL);
+ if(caps)g_return_val_if_fail(caps->refcount > 0, NULL);
+
+ return caps;
}
else if (GST_PAD_PAD_TEMPLATE (realpad)) {
GstPadTemplate *templ = GST_PAD_PAD_TEMPLATE (realpad);
- GST_DEBUG (GST_CAT_CAPS, "using pad template %p with caps %p",
+ GST_CAT_DEBUG (GST_CAT_CAPS, "using pad template %p with caps %p",
templ, GST_PAD_TEMPLATE_CAPS (templ));
return GST_PAD_TEMPLATE_CAPS (templ);
}
- GST_DEBUG (GST_CAT_CAPS, "pad has no caps");
+ GST_CAT_DEBUG (GST_CAT_CAPS, "pad has no caps");
return NULL;
}
*
* Gets the template capabilities of this pad.
*
- * Returns: the template #GstCaps of this pad.
+ * Returns: the template #GstCaps of this pad, unref the caps
+ * if you no longer need it.
*/
GstCaps*
gst_pad_get_pad_template_caps (GstPad *pad)
g_return_val_if_fail (GST_IS_PAD (pad), NULL);
if (GST_PAD_PAD_TEMPLATE (pad))
- return GST_PAD_TEMPLATE_CAPS (GST_PAD_PAD_TEMPLATE (pad));
+ return gst_caps_ref (GST_PAD_TEMPLATE_CAPS (GST_PAD_PAD_TEMPLATE (pad)));
return NULL;
}
*
* Gets the capability with the given name from this pad template.
*
- * Returns: the #GstCaps, or NULL if not found or in case of an error.
+ * Returns: the #GstCaps, or NULL if not found or in case of an error. unref
+ * the caps if you no longer need it.
*/
GstCaps*
gst_pad_template_get_caps_by_name (GstPadTemplate *templ, const gchar *name)
if (!caps)
return NULL;
- return gst_caps_get_by_name (caps, name);
+ return gst_caps_ref (gst_caps_get_by_name (caps, name));
}
/**
}
}
else {
- GST_DEBUG (GST_CAT_PADS,
+ GST_CAT_DEBUG (GST_CAT_PADS,
"could not check capabilities of pads (%s:%s) and (%s:%s) %p %p",
GST_DEBUG_PAD_NAME (srcpad), GST_DEBUG_PAD_NAME (sinkpad),
GST_PAD_CAPS (srcpad), GST_PAD_CAPS (sinkpad));
* Gets the capabilities of the allowed media types that can
* flow through this pad. The caller must free the resulting caps.
*
- * Returns: a newly allocated copy of the allowed #GstCaps.
+ * Returns: the allowed #GstCaps of the pad link. unref the caps if
+ * you no longer need it.
*/
GstCaps*
gst_pad_get_allowed_caps (GstPad *pad)
{
+ GstCaps *caps;
+ GstRealPad *realpad;
+
g_return_val_if_fail (pad != NULL, NULL);
g_return_val_if_fail (GST_IS_PAD (pad), NULL);
- GST_DEBUG (GST_CAT_PROPERTIES, "get allowed caps of %s:%s",
+ realpad = GST_PAD_REALIZE (pad);
+
+ GST_CAT_DEBUG (GST_CAT_PROPERTIES, "get allowed caps of %s:%s",
GST_DEBUG_PAD_NAME (pad));
- return gst_caps_copy (GST_RPAD_FILTER (pad));
+ caps = gst_caps_ref (GST_RPAD_FILTER (realpad));
+
+ return caps;
}
/**
g_return_val_if_fail (pad != NULL, FALSE);
g_return_val_if_fail (GST_IS_PAD (pad), FALSE);
- GST_DEBUG (GST_CAT_PROPERTIES, "set allowed caps of %s:%s",
+ GST_CAT_DEBUG (GST_CAT_PROPERTIES, "set allowed caps of %s:%s",
GST_DEBUG_PAD_NAME (pad));
+
peer = GST_RPAD_PEER (pad);
if (peer)
return gst_pad_try_relink_filtered (pad, GST_PAD (peer),
- GST_RPAD_APPFILTER (pad));
+ GST_RPAD_APPFILTER (pad));
return TRUE;
}
/**
+ * gst_pad_recover_caps_error:
+ * @pad: a #GstPad that had a failed capsnego
+ * @allowed: possible caps for the link
+ *
+ * Attempt to recover from a failed caps negotiation. This function
+ * is typically called by a plugin that exhausted its list of caps
+ * and wants the application to resolve the issue. The application
+ * should connect to the pad's caps_nego_failed signal and should
+ * resolve the issue by connecting another element for example.
+ *
+ * Returns: TRUE when the issue was resolved, dumps detailed information
+ * on the console and returns FALSE otherwise.
+ */
+gboolean
+gst_pad_recover_caps_error (GstPad *pad, GstCaps *allowed)
+{
+ GstElement *parent;
+
+ g_return_val_if_fail (GST_IS_PAD (pad), FALSE);
+
+ /* see if someone can resolve this */
+ if (g_signal_has_handler_pending (G_OBJECT (pad),
+ gst_real_pad_signals[REAL_CAPS_NEGO_FAILED], 0, FALSE))
+ {
+ /* clear pad caps first */
+ gst_caps_replace (&GST_PAD_CAPS (pad), NULL);
+
+ /* lets hope some signal manages to set the caps again */
+ g_signal_emit (G_OBJECT (pad), gst_real_pad_signals[REAL_CAPS_NEGO_FAILED], 0, allowed);
+
+ /* if the pad has caps now or is disabled, it's ok */
+ if (GST_PAD_CAPS (pad) != NULL || !GST_PAD_IS_ACTIVE (pad))
+ return TRUE;
+ }
+
+ /* report error */
+ parent = gst_pad_get_parent (pad);
+ gst_element_error (parent, "negotiation failed on pad %s:%s",
+ GST_DEBUG_PAD_NAME (pad));
+
+ return FALSE;
+}
+
+/**
* gst_pad_get_bufferpool:
* @pad: a #GstPad to get the bufferpool from.
*
if (!peer)
return NULL;
- GST_DEBUG_ENTER ("(%s:%s)", GST_DEBUG_PAD_NAME (pad));
+ GST_CAT_DEBUG (GST_CAT_BUFFER, "(%s:%s): getting bufferpool", GST_DEBUG_PAD_NAME (pad));
if (peer->bufferpoolfunc) {
- GST_DEBUG (GST_CAT_PADS,
+ GST_CAT_DEBUG (GST_CAT_PADS,
"calling bufferpoolfunc &%s (@%p) of peer pad %s:%s",
GST_DEBUG_FUNCPTR_NAME (peer->bufferpoolfunc),
&peer->bufferpoolfunc, GST_DEBUG_PAD_NAME (((GstPad*) peer)));
return (peer->bufferpoolfunc) (((GstPad*) peer));
} else {
- GST_DEBUG (GST_CAT_PADS, "no bufferpoolfunc for peer pad %s:%s at %p",
+ GST_CAT_DEBUG (GST_CAT_PADS, "no bufferpoolfunc for peer pad %s:%s at %p",
GST_DEBUG_PAD_NAME (((GstPad*) peer)), &peer->bufferpoolfunc);
return NULL;
}
* and a parent would hold a reference */
g_assert (GST_PAD_PEER (pad) == NULL);
- GST_DEBUG (GST_CAT_REFCOUNTING, "dispose %s:%s", GST_DEBUG_PAD_NAME(pad));
+ GST_CAT_DEBUG (GST_CAT_REFCOUNTING, "dispose %s:%s", GST_DEBUG_PAD_NAME(pad));
- if (GST_PAD_PAD_TEMPLATE (pad)){
- GST_DEBUG (GST_CAT_REFCOUNTING, "unreffing padtemplate'%s'",
- GST_OBJECT_NAME (GST_PAD_PAD_TEMPLATE (pad)));
- gst_object_unref (GST_OBJECT (GST_PAD_PAD_TEMPLATE (pad)));
- GST_PAD_PAD_TEMPLATE (pad) = NULL;
- }
-
/* we destroy the ghostpads, because they are nothing without the real pad */
if (GST_REAL_PAD (pad)->ghostpads) {
GList *orig, *ghostpads;
GstPad *ghostpad = GST_PAD (ghostpads->data);
if (GST_IS_ELEMENT (GST_OBJECT_PARENT (ghostpad))){
- GST_DEBUG (GST_CAT_REFCOUNTING, "removing ghost pad from element '%s'",
+ GST_CAT_DEBUG (GST_CAT_REFCOUNTING, "removing ghost pad from element '%s'",
GST_OBJECT_NAME (GST_OBJECT_PARENT (ghostpad)));
gst_element_remove_ghost_pad (GST_ELEMENT (GST_OBJECT_PARENT (ghostpad)), GST_PAD (ghostpad));
g_list_free (GST_REAL_PAD(pad)->ghostpads);
}
+ gst_caps_replace (&GST_PAD_CAPS (pad), NULL);
+ gst_caps_replace (&GST_RPAD_APPFILTER (pad), NULL);
+
if (GST_IS_ELEMENT (GST_OBJECT_PARENT (pad))) {
- GST_DEBUG (GST_CAT_REFCOUNTING, "removing pad from element '%s'",
+ GST_CAT_DEBUG (GST_CAT_REFCOUNTING, "removing pad from element '%s'",
GST_OBJECT_NAME (GST_OBJECT (GST_ELEMENT (GST_OBJECT_PARENT (pad)))));
gst_element_remove_pad (GST_ELEMENT (GST_OBJECT_PARENT (pad)), pad);
split = g_strsplit (peer, ".", 2);
if (split[0] == NULL || split[1] == NULL) {
- GST_DEBUG (GST_CAT_XML,
+ GST_CAT_DEBUG (GST_CAT_XML,
"Could not parse peer '%s' for pad %s:%s, leaving unlinked",
peer, GST_DEBUG_PAD_NAME (pad));
return;
gst_pad_push (GstPad *pad, GstBuffer *buf)
{
GstRealPad *peer;
+ GstData *data = GST_DATA(buf);
- GST_DEBUG_ENTER ("(%s:%s)", GST_DEBUG_PAD_NAME (pad));
+ g_assert (GST_IS_PAD (pad));
+ GST_CAT_LOG_OBJECT (GST_CAT_DATAFLOW, pad, "pushing");
g_return_if_fail (GST_PAD_DIRECTION (pad) == GST_PAD_SRC);
- if (!gst_probe_dispatcher_dispatch (&(GST_REAL_PAD (pad)->probedisp), GST_DATA (buf)))
+ if (!gst_probe_dispatcher_dispatch (&(GST_REAL_PAD (pad)->probedisp), &data))
return;
peer = GST_RPAD_PEER (pad);
GST_DEBUG_PAD_NAME (pad));
}
else {
- if (!GST_IS_EVENT (buf) && !GST_PAD_IS_ACTIVE (pad)) {
- g_warning ("push on pad %s:%s but it is not active",
+ if (!GST_IS_EVENT (data) && !GST_PAD_IS_ACTIVE (peer)) {
+ g_warning ("push on peer of pad %s:%s but peer is not active",
GST_DEBUG_PAD_NAME (pad));
return;
}
if (peer->chainhandler) {
- if (buf) {
- GST_DEBUG (GST_CAT_DATAFLOW,
+ if (data) {
+ GST_CAT_LOG_OBJECT (GST_CAT_DATAFLOW, pad,
"calling chainhandler &%s of peer pad %s:%s",
GST_DEBUG_FUNCPTR_NAME (peer->chainhandler),
GST_DEBUG_PAD_NAME (GST_PAD (peer)));
- if (!gst_probe_dispatcher_dispatch (&peer->probedisp, GST_DATA (buf)))
+ if (!gst_probe_dispatcher_dispatch (&peer->probedisp, &data))
return;
- (peer->chainhandler) (GST_PAD_CAST (peer), buf);
+ (peer->chainhandler) (GST_PAD_CAST (peer), (GstBuffer *)data);
return;
}
else {
}
}
/* clean up the mess here */
- if (buf != NULL) gst_data_unref (GST_DATA (buf));
+ if (data != NULL) gst_data_unref (data);
}
/**
{
GstRealPad *peer;
- GST_DEBUG_ENTER("(%s:%s)",GST_DEBUG_PAD_NAME(pad));
+ GST_CAT_LOG_OBJECT (GST_CAT_DATAFLOW, pad, "pulling");
g_return_val_if_fail (GST_PAD_DIRECTION (pad) == GST_PAD_SINK,
GST_BUFFER (gst_event_new (GST_EVENT_INTERRUPT)));
else {
restart:
if (peer->gethandler) {
- GstBuffer *buf;
- gboolean active = GST_PAD_IS_ACTIVE (peer);
+ GstData *data;
- GST_DEBUG (GST_CAT_DATAFLOW, "calling gethandler %s of peer pad %s:%s",
- GST_DEBUG_FUNCPTR_NAME (peer->gethandler),
- GST_DEBUG_PAD_NAME (peer));
+ GST_CAT_LOG_OBJECT (GST_CAT_DATAFLOW, pad,
+ "calling gethandler %s of peer pad %s:%s",
+ GST_DEBUG_FUNCPTR_NAME (peer->gethandler),
+ GST_DEBUG_PAD_NAME (peer));
- buf = (peer->gethandler) (GST_PAD_CAST (peer));
+ data = GST_DATA((peer->gethandler) (GST_PAD_CAST (peer)));
- if (buf) {
- if (!gst_probe_dispatcher_dispatch (&peer->probedisp, GST_DATA (buf)))
+ if (data) {
+ if (!gst_probe_dispatcher_dispatch (&peer->probedisp, &data))
goto restart;
-
- if (!GST_IS_EVENT (buf) && !active) {
- g_warning ("pull on pad %s:%s but it is not active",
- GST_DEBUG_PAD_NAME (peer));
- return GST_BUFFER (gst_event_new (GST_EVENT_INTERRUPT));
- }
- return buf;
+ return GST_BUFFER(data);
}
/* no null buffers allowed */
gst_element_error (GST_PAD_PARENT (pad),
"NULL buffer during pull on %s:%s",
- GST_DEBUG_PAD_NAME (pad), NULL);
+ GST_DEBUG_PAD_NAME (pad));
} else {
gst_element_error (GST_PAD_PARENT (pad),
"internal error: pull on pad %s:%s "
"but the peer pad %s:%s has no gethandler",
- GST_DEBUG_PAD_NAME (pad), GST_DEBUG_PAD_NAME (peer),
- NULL);
+ GST_DEBUG_PAD_NAME (pad), GST_DEBUG_PAD_NAME (peer));
}
}
return GST_BUFFER (gst_event_new (GST_EVENT_INTERRUPT));
*/
static void gst_pad_template_class_init (GstPadTemplateClass *klass);
static void gst_pad_template_init (GstPadTemplate *templ);
+static void gst_pad_template_dispose (GObject *object);
GType
gst_pad_template_get_type (void)
NULL, NULL, gst_marshal_VOID__POINTER, G_TYPE_NONE, 1,
G_TYPE_POINTER);
+ gobject_class->dispose = gst_pad_template_dispose;
+
gstobject_class->path_string_separator = "*";
}
{
}
+static void
+gst_pad_template_dispose (GObject *object)
+{
+ GstPadTemplate *templ = GST_PAD_TEMPLATE (object);
+
+ g_free (GST_PAD_TEMPLATE_NAME_TEMPLATE (templ));
+ gst_caps_unref (GST_PAD_TEMPLATE_CAPS (templ));
+
+ G_OBJECT_CLASS (padtemplate_parent_class)->dispose (object);
+}
+
/* ALWAYS padtemplates cannot have conversion specifications, it doesn't make
* sense.
* SOMETIMES padtemplates can do whatever they want, they are provided by the
}
/**
- * gst_pad_template_new:
+ * gst_pad_template_newv:
* @name_template: the name template.
* @direction: the #GstPadDirection of the template.
* @presence: the #GstPadPresence of the pad.
* @caps: a #GstCaps set for the template.
- * @...: a NULL-terminated list of #GstCaps.
+ * @var_args: a NULL-terminated list of #GstCaps.
*
* Creates a new pad template with a name according to the given template
* and with the given arguments.
* Returns: a new #GstPadTemplate.
*/
GstPadTemplate*
-gst_pad_template_new (const gchar *name_template,
- GstPadDirection direction, GstPadPresence presence,
- GstCaps *caps, ...)
+gst_pad_template_newv (const gchar *name_template,
+ GstPadDirection direction, GstPadPresence presence,
+ GstCaps *caps, va_list var_args)
{
GstPadTemplate *new;
- va_list var_args;
GstCaps *thecaps = NULL;
g_return_val_if_fail (name_template != NULL, NULL);
GST_PAD_TEMPLATE_DIRECTION (new) = direction;
GST_PAD_TEMPLATE_PRESENCE (new) = presence;
- va_start (var_args, caps);
-
+ GST_FLAG_SET (GST_OBJECT (new), GST_PAD_TEMPLATE_FIXED);
while (caps) {
- new->fixed &= caps->fixed;
- thecaps = gst_caps_append (thecaps, gst_caps_ref (caps));
+ if (!GST_CAPS_IS_FIXED (caps)) {
+ GST_FLAG_UNSET (GST_OBJECT (new), GST_PAD_TEMPLATE_FIXED);
+ }
+ thecaps = gst_caps_append (thecaps, caps);
caps = va_arg (var_args, GstCaps*);
}
- va_end (var_args);
GST_PAD_TEMPLATE_CAPS (new) = thecaps;
+ gst_caps_ref (thecaps);
+ gst_caps_sink (thecaps);
+
+ return new;
+}
+
+/**
+ * gst_pad_template_new:
+ * @name_template: the name template.
+ * @direction: the #GstPadDirection of the template.
+ * @presence: the #GstPadPresence of the pad.
+ * @caps: a #GstCaps set for the template.
+ * @...: a NULL-terminated list of #GstCaps.
+ *
+ * Creates a new pad template with a name according to the given template
+ * and with the given arguments.
+ *
+ * Returns: a new #GstPadTemplate.
+ */
+GstPadTemplate*
+gst_pad_template_new (const gchar *name_template,
+ GstPadDirection direction, GstPadPresence presence,
+ GstCaps *caps, ...)
+{
+ GstPadTemplate *new;
+ va_list var_args;
+
+ va_start (var_args, caps);
+
+ new = gst_pad_template_newv (name_template, direction, presence,
+ caps, var_args);
+
+ va_end (var_args);
return new;
}
*
* Gets the capabilities of the pad template.
*
- * Returns: the #GstCaps of the pad template.
+ * Returns: the #GstCaps of the pad template. unref the caps
+ * after use.
*/
GstCaps*
gst_pad_template_get_caps (GstPadTemplate *templ)
{
g_return_val_if_fail (templ != NULL, NULL);
- return GST_PAD_TEMPLATE_CAPS (templ);
+ return gst_caps_ref (GST_PAD_TEMPLATE_CAPS (templ));
}
/**
/***** ghost pads *****/
GType _gst_ghost_pad_type = 0;
-static void gst_ghost_pad_class_init (GstGhostPadClass *klass);
-static void gst_ghost_pad_init (GstGhostPad *pad);
+static void gst_ghost_pad_class_init (GstGhostPadClass *klass);
+static void gst_ghost_pad_init (GstGhostPad *pad);
+static void gst_ghost_pad_dispose (GObject *object);
static GstPad *ghost_pad_parent_class = NULL;
/* static guint gst_ghost_pad_signals[LAST_SIGNAL] = { 0 }; */
gobject_class = (GObjectClass*) klass;
ghost_pad_parent_class = g_type_class_ref (GST_TYPE_PAD);
+
+ gobject_class->dispose = GST_DEBUG_FUNCPTR (gst_ghost_pad_dispose);
}
static void
gst_ghost_pad_init (GstGhostPad *pad)
{
- pad->realpad = NULL;
+ /* zeroed by glib */
+}
+static void
+gst_ghost_pad_dispose (GObject *object)
+{
+ GstGhostPad *pad = GST_GHOST_PAD (object);
+
+ if (pad->realpad)
+ gst_pad_remove_ghost_pad((GstPad *) pad->realpad, (GstPad *) pad);
+
+ G_OBJECT_CLASS (ghost_pad_parent_class)->dispose (object);
}
/**
*
* Creates a new ghost pad associated with the given pad, and names it with
* the given name. If name is NULL, a guaranteed unique name (across all
- * ghost pads) will be assigned (most likely of the form ghostpad&perc;d).
+ * ghost pads) will be assigned (most likely of the form ghostpad%d).
*
* Returns: a new ghost #GstPad, or NULL in case of an error.
*/
realpad = GST_PAD_REALIZE (realpad);
}
GST_GPAD_REALPAD (ghostpad) = realpad;
- GST_PAD_PAD_TEMPLATE (ghostpad) = GST_PAD_PAD_TEMPLATE (pad);
+ gst_pad_set_pad_template (GST_PAD (ghostpad), GST_PAD_PAD_TEMPLATE (pad));
/* add ourselves to the real pad's list of ghostpads */
gst_pad_add_ghost_pad (pad, GST_PAD (ghostpad));
/* FIXME need to ref the real pad here... ? */
- GST_DEBUG (GST_CAT_PADS, "created ghost pad \"%s\"",
+ GST_CAT_DEBUG (GST_CAT_PADS, "created ghost pad \"%s\"",
gst_pad_get_name (GST_PAD (ghostpad)));
return GST_PAD (ghostpad);
gst_pad_event_default_dispatch (GstPad *pad, GstElement *element,
GstEvent *event)
{
- GList *pads = element->pads;
+ GList *orig, *pads;
+
+ orig = pads = gst_pad_get_internal_links (pad);
while (pads) {
GstPad *eventpad = GST_PAD (pads->data);
pads = g_list_next (pads);
/* for all pads in the opposite direction that are linked */
- if (GST_PAD_DIRECTION (eventpad) != GST_PAD_DIRECTION (pad)
- && GST_PAD_IS_LINKED (eventpad)) {
+ if (GST_PAD_IS_LINKED (eventpad)) {
if (GST_PAD_DIRECTION (eventpad) == GST_PAD_SRC) {
/* increase the refcount */
gst_event_ref (event);
/* we only send the event on one pad, multi-sinkpad elements
* should implement a handler */
+ g_list_free (orig);
return gst_pad_send_event (peerpad, event);
}
}
}
gst_event_unref (event);
+ g_list_free (orig);
return TRUE;
}
if (GST_EVENT_SRC (event) == NULL)
GST_EVENT_SRC (event) = gst_object_ref (GST_OBJECT (rpad));
- GST_DEBUG (GST_CAT_EVENT, "have event %d on pad %s:%s",
+ GST_CAT_DEBUG (GST_CAT_EVENT, "have event %d on pad %s:%s",
GST_EVENT_TYPE (event), GST_DEBUG_PAD_NAME (rpad));
- if (GST_RPAD_EVENTFUNC (rpad))
- success = GST_RPAD_EVENTFUNC (rpad) (GST_PAD_CAST (rpad), event);
+ if (GST_RPAD_EVENTHANDLER (rpad))
+ success = GST_RPAD_EVENTHANDLER (rpad) (GST_PAD_CAST (rpad), event);
else {
- GST_DEBUG (GST_CAT_EVENT, "there's no event function for pad %s:%s",
- GST_DEBUG_PAD_NAME (rpad));
+ g_warning ("pad %s:%s has no event handler", GST_DEBUG_PAD_NAME (rpad));
gst_event_unref (event);
}
* gst_pad_query:
* @pad: a #GstPad to invoke the default query on.
* @type: the #GstQueryType of the query to perform.
- * @format: a pointer to the #GstFormat of the result.
+ * @format: a pointer to the #GstFormat asked for.
+ * On return contains the #GstFormat used.
* @value: a pointer to the result.
*
- * Queries a pad for one of the available properties.
+ * Queries a pad for one of the available properties. The format will be
+ * adjusted to the actual format used when specifying formats such as
+ * GST_FORMAT_DEFAULT.
+ * FIXME: Tell if the format can be adjusted when specifying a definite format.
*
* Returns: TRUE if the query could be performed.
*/
return NULL;
}
-