#define TRY_SETUP_LINK(l) G_STMT_START { \
if( (!(l)->src.element) && (!(l)->src.name) ){ \
SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link has no source [sink=%s@%p]"), \
- (l)->sink.name ? (l)->sink.name : _(""), \
+ (l)->sink.name ? (l)->sink.name : "", \
(l)->sink.element); \
gst_parse_free_link (l); \
}else if( (!(l)->sink.element) && (!(l)->sink.name) ){ \
SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link has no sink [source=%s@%p]"), \
- (l)->src.name ? (l)->src.name :_(""), \
+ (l)->src.name ? (l)->src.name : "", \
(l)->src.element); \
gst_parse_free_link (l); \
}else{ \
- graph->links = g_slist_prepend (graph->links, l ); \
+ graph->links = g_slist_append (graph->links, l ); \
} \
} G_STMT_END
gchar *sink_pad;
GstElement *sink;
GstCaps *caps;
- gulong signal_id;
+ gulong pad_added_signal_id, no_more_pads_signal_id;
+ gboolean all_pads;
} DelayedLink;
typedef struct {
bin = GST_BIN (pipeline);
rr->element = gst_bin_get_by_name_recurse_up (bin, rr->name);
} else {
- rr->element = strcmp (GST_ELEMENT_NAME (pipeline), rr->name) == 0 ?
+ rr->element = strcmp (GST_ELEMENT_NAME (pipeline), rr->name) == 0 ?
gst_object_ref(pipeline) : NULL;
}
if(rr->element) return 0; /* resolved */
gst_parse_add_delayed_set(parent, sub_name, value_str);
g_free (sub_name);
}
+ gst_object_unref (parent);
parent = child;
current++;
}
+ if (parent)
+ gst_object_unref (parent);
g_strfreev (names);
}
}
GstElement *bin;
bin = gst_parse_bin_from_description_full (set->value_str, TRUE, NULL,
- GST_PARSE_FLAG_NO_SINGLE_ELEMENT_BINS, NULL);
+ GST_PARSE_FLAG_NO_SINGLE_ELEMENT_BINS | GST_PARSE_FLAG_PLACE_IN_BIN, NULL);
if (bin) {
g_value_set_object (&v, bin);
got_value = TRUE;
}
pos++;
while (g_ascii_isspace (*pos)) pos++;
- if (*pos == '"') {
+ /* truncate a string if it is delimited with double quotes */
+ if (*pos == '"' && pos[strlen (pos) - 1] == '"') {
pos++;
pos[strlen (pos) - 1] = '\0';
}
} else {
pspec = g_object_class_find_property (G_OBJECT_GET_CLASS (element), value);
if (pspec != NULL) {
- target = g_object_ref (element);
+ target = G_OBJECT (g_object_ref (element));
GST_CAT_LOG_OBJECT (GST_CAT_PIPELINE, target, "found %s property", value);
} else {
SET_ERROR (graph->error, GST_PARSE_ERROR_NO_SUCH_PROPERTY, \
GstElement *bin;
bin = gst_parse_bin_from_description_full (pos, TRUE, NULL,
- GST_PARSE_FLAG_NO_SINGLE_ELEMENT_BINS, NULL);
+ GST_PARSE_FLAG_NO_SINGLE_ELEMENT_BINS | GST_PARSE_FLAG_PLACE_IN_BIN, NULL);
if (bin) {
g_value_set_object (&v, bin);
got_value = TRUE;
g_slice_free (DelayedLink, link);
}
+#define PRETTY_PAD_NAME_FMT "%s %s of %s named %s"
+#define PRETTY_PAD_NAME_ARGS(elem, pad_name) \
+ (pad_name ? "pad " : "some"), (pad_name ? pad_name : "pad"), \
+ G_OBJECT_TYPE_NAME(elem), GST_STR_NULL (GST_ELEMENT_NAME (elem))
+
+static void gst_parse_no_more_pads (GstElement *src, gpointer data)
+{
+ DelayedLink *link = data;
+
+ /* Don't warn for all-pads links, as we expect those to
+ * still be active at no-more-pads */
+ if (!link->all_pads) {
+ GST_ELEMENT_WARNING(src, PARSE, DELAYED_LINK,
+ (_("Delayed linking failed.")),
+ ("failed delayed linking " PRETTY_PAD_NAME_FMT " to " PRETTY_PAD_NAME_FMT,
+ PRETTY_PAD_NAME_ARGS (src, link->src_pad),
+ PRETTY_PAD_NAME_ARGS (link->sink, link->sink_pad)));
+ }
+ /* we keep the handlers connected, so that in case an element still adds a pad
+ * despite no-more-pads, we will consider it for pending delayed links */
+}
+
static void gst_parse_found_pad (GstElement *src, GstPad *pad, gpointer data)
{
DelayedLink *link = data;
- GST_CAT_INFO (GST_CAT_PIPELINE, "trying delayed linking %s:%s to %s:%s",
- GST_STR_NULL (GST_ELEMENT_NAME (src)), GST_STR_NULL (link->src_pad),
- GST_STR_NULL (GST_ELEMENT_NAME (link->sink)), GST_STR_NULL (link->sink_pad));
+ GST_CAT_INFO (GST_CAT_PIPELINE,
+ "trying delayed linking %s " PRETTY_PAD_NAME_FMT " to " PRETTY_PAD_NAME_FMT,
+ link->all_pads ? "all pads" : "one pad",
+ PRETTY_PAD_NAME_ARGS (src, link->src_pad),
+ PRETTY_PAD_NAME_ARGS (link->sink, link->sink_pad));
if (gst_element_link_pads_filtered (src, link->src_pad, link->sink,
link->sink_pad, link->caps)) {
/* do this here, we don't want to get any problems later on when
* unlocking states */
- GST_CAT_DEBUG (GST_CAT_PIPELINE, "delayed linking %s:%s to %s:%s worked",
- GST_STR_NULL (GST_ELEMENT_NAME (src)), GST_STR_NULL (link->src_pad),
- GST_STR_NULL (GST_ELEMENT_NAME (link->sink)), GST_STR_NULL (link->sink_pad));
- g_signal_handler_disconnect (src, link->signal_id);
+ GST_CAT_DEBUG (GST_CAT_PIPELINE,
+ "delayed linking %s " PRETTY_PAD_NAME_FMT " to " PRETTY_PAD_NAME_FMT " worked",
+ link->all_pads ? "all pads" : "one pad",
+ PRETTY_PAD_NAME_ARGS (src, link->src_pad),
+ PRETTY_PAD_NAME_ARGS (link->sink, link->sink_pad));
+ g_signal_handler_disconnect (src, link->no_more_pads_signal_id);
+ /* releases 'link' */
+ if (!link->all_pads)
+ g_signal_handler_disconnect (src, link->pad_added_signal_id);
}
}
static gboolean
gst_parse_perform_delayed_link (GstElement *src, const gchar *src_pad,
GstElement *sink, const gchar *sink_pad,
- GstCaps *caps)
+ GstCaps *caps, gboolean all_pads)
{
GList *templs = gst_element_class_get_pad_template_list (
GST_ELEMENT_GET_CLASS (src));
{
DelayedLink *data = g_slice_new (DelayedLink);
+ data->all_pads = all_pads;
+
/* TODO: maybe we should check if src_pad matches this template's names */
- GST_CAT_DEBUG (GST_CAT_PIPELINE, "trying delayed link %s:%s to %s:%s",
- GST_STR_NULL (GST_ELEMENT_NAME (src)), GST_STR_NULL (src_pad),
- GST_STR_NULL (GST_ELEMENT_NAME (sink)), GST_STR_NULL (sink_pad));
+ GST_CAT_DEBUG (GST_CAT_PIPELINE,
+ "trying delayed link " PRETTY_PAD_NAME_FMT " to " PRETTY_PAD_NAME_FMT,
+ PRETTY_PAD_NAME_ARGS (src, src_pad),
+ PRETTY_PAD_NAME_ARGS (sink, sink_pad));
data->src_pad = g_strdup (src_pad);
data->sink = sink;
} else {
data->caps = NULL;
}
- data->signal_id = g_signal_connect_data (src, "pad-added",
+ data->pad_added_signal_id = g_signal_connect_data (src, "pad-added",
G_CALLBACK (gst_parse_found_pad), data,
(GClosureNotify) gst_parse_free_delayed_link, (GConnectFlags) 0);
+ data->no_more_pads_signal_id = g_signal_connect (src, "no-more-pads",
+ G_CALLBACK (gst_parse_no_more_pads), data);
return TRUE;
}
}
return FALSE;
}
+static gboolean
+gst_parse_element_can_do_caps (GstElement * e, GstPadDirection dir,
+ GstCaps * link_caps)
+{
+ gboolean can_do = FALSE, done = FALSE;
+ GstIterator *it;
+
+ it = (dir == GST_PAD_SRC) ? gst_element_iterate_src_pads (e) : gst_element_iterate_sink_pads (e);
+
+ while (!done && !can_do) {
+ GValue v = G_VALUE_INIT;
+ GstPad *pad;
+ GstCaps *caps;
+
+ switch (gst_iterator_next (it, &v)) {
+ case GST_ITERATOR_OK:
+ pad = g_value_get_object (&v);
+
+ caps = gst_pad_get_current_caps (pad);
+ if (caps == NULL)
+ caps = gst_pad_query_caps (pad, NULL);
+
+ can_do = gst_caps_can_intersect (caps, link_caps);
+
+ GST_TRACE ("can_do: %d for %" GST_PTR_FORMAT " and %" GST_PTR_FORMAT,
+ can_do, caps, link_caps);
+
+ gst_caps_unref (caps);
+
+ g_value_unset (&v);
+ break;
+ case GST_ITERATOR_DONE:
+ case GST_ITERATOR_ERROR:
+ done = TRUE;
+ break;
+ case GST_ITERATOR_RESYNC:
+ gst_iterator_resync (it);
+ break;
+ }
+ }
+
+ gst_iterator_free (it);
+
+ return can_do;
+}
+
/*
* performs a link and frees the struct. src and sink elements must be given
* return values 0 - link performed
g_assert (GST_IS_ELEMENT (sink));
GST_CAT_INFO (GST_CAT_PIPELINE,
- "linking %s:%s to %s:%s (%u/%u) with caps \"%" GST_PTR_FORMAT "\"",
- GST_ELEMENT_NAME (src), link->src.name ? link->src.name : "(any)",
- GST_ELEMENT_NAME (sink), link->sink.name ? link->sink.name : "(any)",
+ "linking " PRETTY_PAD_NAME_FMT " to " PRETTY_PAD_NAME_FMT " (%u/%u) with caps \"%" GST_PTR_FORMAT "\"",
+ PRETTY_PAD_NAME_ARGS (src, link->src.name),
+ PRETTY_PAD_NAME_ARGS (sink, link->sink.name),
g_slist_length (srcs), g_slist_length (sinks), link->caps);
if (!srcs || !sinks) {
- if (gst_element_link_pads_filtered (src,
+ gboolean found_one = gst_element_link_pads_filtered (src,
+ srcs ? (const gchar *) srcs->data : NULL, sink,
+ sinks ? (const gchar *) sinks->data : NULL, link->caps);
+
+ if (found_one) {
+ if (!link->all_pads)
+ goto success; /* Linked one, and not an all-pads link = we're done */
+
+ /* Try and link more available pads */
+ while (gst_element_link_pads_filtered (src,
srcs ? (const gchar *) srcs->data : NULL, sink,
- sinks ? (const gchar *) sinks->data : NULL, link->caps)) {
+ sinks ? (const gchar *) sinks->data : NULL, link->caps));
+ }
+
+ /* We either didn't find any static pads, or this is a all-pads link,
+ * in which case watch for future pads and link those. Not a failure
+ * in the all-pads case if there's no sometimes pads to watch */
+ if (gst_parse_perform_delayed_link (src,
+ srcs ? (const gchar *) srcs->data : NULL,
+ sink, sinks ? (const gchar *) sinks->data : NULL, link->caps,
+ link->all_pads) || link->all_pads) {
goto success;
} else {
- if (gst_parse_perform_delayed_link (src,
- srcs ? (const gchar *) srcs->data : NULL,
- sink, sinks ? (const gchar *) sinks->data : NULL, link->caps)) {
- goto success;
- } else {
- goto error;
- }
+ goto error;
}
}
if (g_slist_length (link->src.pads) != g_slist_length (link->sink.pads)) {
} else {
if (gst_parse_perform_delayed_link (src, src_pad,
sink, sink_pad,
- link->caps)) {
+ link->caps, link->all_pads)) {
continue;
} else {
goto error;
return 0;
error:
- SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
- _("could not link %s to %s"), GST_ELEMENT_NAME (src),
- GST_ELEMENT_NAME (sink));
+ if (link->caps != NULL) {
+ gboolean src_can_do_caps, sink_can_do_caps;
+ gchar *caps_str = gst_caps_to_string (link->caps);
+
+ src_can_do_caps =
+ gst_parse_element_can_do_caps (src, GST_PAD_SRC, link->caps);
+ sink_can_do_caps =
+ gst_parse_element_can_do_caps (sink, GST_PAD_SINK, link->caps);
+
+ if (!src_can_do_caps && sink_can_do_caps) {
+ SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
+ _("could not link %s to %s, %s can't handle caps %s"),
+ GST_ELEMENT_NAME (src), GST_ELEMENT_NAME (sink),
+ GST_ELEMENT_NAME (src), caps_str);
+ } else if (src_can_do_caps && !sink_can_do_caps) {
+ SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
+ _("could not link %s to %s, %s can't handle caps %s"),
+ GST_ELEMENT_NAME (src), GST_ELEMENT_NAME (sink),
+ GST_ELEMENT_NAME (sink), caps_str);
+ } else if (!src_can_do_caps && !sink_can_do_caps) {
+ SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
+ _("could not link %s to %s, neither element can handle caps %s"),
+ GST_ELEMENT_NAME (src), GST_ELEMENT_NAME (sink), caps_str);
+ } else {
+ SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
+ _("could not link %s to %s with caps %s"),
+ GST_ELEMENT_NAME (src), GST_ELEMENT_NAME (sink), caps_str);
+ }
+ g_free (caps_str);
+ } else {
+ SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
+ _("could not link %s to %s"), GST_ELEMENT_NAME (src),
+ GST_ELEMENT_NAME (sink));
+ }
gst_parse_free_link (link);
return -1;
}
graph_t *gg;
}
+/* No grammar ambiguities expected, FAIL otherwise */
+%expect 0
+
%token <ss> PARSE_URL
%token <ss> IDENTIFIER
-%left <ss> REF PADREF BINREF
+%left <ss> REF PADREF BINREF
%token <ss> ASSIGNMENT
%token <ss> LINK
+%token <ss> LINK_ALL
%type <ss> binopener
%type <gg> graph
%type <pp> morepads pads assignments
%destructor { gst_parse_strfree ($$); } <ss>
-%destructor { gst_parse_free_chain($$); } <cc>
+%destructor { if($$)
+ gst_parse_free_chain($$); } <cc>
%destructor { gst_parse_free_link ($$); } <ll>
%destructor { gst_parse_free_reference(&($$));} <rr>
%destructor { gst_object_unref ($$); } <ee>
%left '(' ')'
%left ','
%right '.'
-%left '!' '='
+%left '!' '=' ':'
%lex-param { void *scanner }
%parse-param { void *scanner }
/*************************************************************
* Grammar explanation: (cont'd)
-* a _chain_ is a list of _elementary_s that have _link_s inbetween
+* a _chain_ is a list of _elementary_s that have _link_s in between
* which are represented through infix-notation.
*
* fakesrc ! sometransformation ! fakesink
* that are syntactically closed (handled later in this file).
* (e.g. fakesrc ! sinkreferencename.padname)
**************************************************************/
-chain: openchain { $$=$1;
+chain: openchain { $$=$1;
if($$->last.name){
SET_ERROR (graph->error, GST_PARSE_ERROR_SYNTAX,
_("unexpected reference \"%s\" - ignoring"), $$->last.name);
$$ = $4;
$$->last.pads = g_slist_concat ($$->last.pads, $5);
}
-
;
link: LINK { $$ = gst_parse_link_new ();
- $$->src.element = NULL;
- $$->sink.element = NULL;
- $$->src.name = NULL;
- $$->sink.name = NULL;
- $$->src.pads = NULL;
- $$->sink.pads = NULL;
- $$->caps = NULL;
+ $$->all_pads = FALSE;
+ if ($1) {
+ $$->caps = gst_caps_from_string ($1);
+ if ($$->caps == NULL)
+ SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("could not parse caps \"%s\""), $1);
+ gst_parse_strfree ($1);
+ }
+ }
+ | LINK_ALL { $$ = gst_parse_link_new ();
+ $$->all_pads = TRUE;
if ($1) {
$$->caps = gst_caps_from_string ($1);
if ($$->caps == NULL)
* _reference_ on each side. That
* works already after the explanations above.
* someSourceName.Pad ! someSinkName.
-* someSourceName.Pad,anotherPad ! someSinkName.Apad,Bpad
+* someSourceName.Pad,anotherPad ! someSinkName.Apad,Bpad
*
* If a syntax error occurs, the already finished _chain_s
* and _links_ are kept intact.
assignments: /* NOP */ { $$ = NULL; }
- | assignments ASSIGNMENT { $$ = g_slist_prepend ($1, $2); }
+ | ASSIGNMENT assignments { $$ = g_slist_prepend ($2, $1); }
;
-binopener: '(' { $$ = gst_parse_strdup(_("bin")); }
+binopener: '(' { $$ = gst_parse_strdup("bin"); }
| BINREF { $$ = $1; }
;
-bin: binopener assignments chainlist ')' {
+bin: binopener assignments chainlist ')' {
chain_t *chain = $3;
GSList *walk;
GstBin *bin = (GstBin *) gst_element_factory_make ($1, NULL);
graph_t g;
gchar *dstr;
GSList *walk;
- GstBin *bin = NULL;
GstElement *ret;
yyscan_t scanner;
g.chain->last.name=NULL;
g.chain->last.pads=NULL;
};
-
+
/* ensure elements is not empty */
if(!g.chain->elements){
g.chain->elements= g_slist_prepend (NULL, NULL);
};
-
+
/* put all elements in our bin if necessary */
if(g.chain->elements->next){
- bin = GST_BIN (gst_element_factory_make ("pipeline", NULL));
+ GstBin *bin;
+ if (flags & GST_PARSE_FLAG_PLACE_IN_BIN)
+ bin = GST_BIN (gst_element_factory_make ("bin", NULL));
+ else
+ bin = GST_BIN (gst_element_factory_make ("pipeline", NULL));
g_assert (bin);
for (walk = g.chain->elements; walk; walk = walk->next) {
}
g_slist_free (g.chain->elements);
g.chain->elements = g_slist_prepend (NULL, bin);
- }
+ }
ret = (GstElement *) g.chain->elements->data;
g_slist_free (g.chain->elements);
g.chain->elements=NULL;
- if (GST_IS_BIN (ret))
- bin = GST_BIN (ret);
gst_parse_free_chain (g.chain);
g.chain = NULL;
error1:
if (g.chain) {
- g_slist_foreach (g.chain->elements, (GFunc)gst_object_unref, NULL);
- g_slist_free (g.chain->elements);
gst_parse_free_chain (g.chain);
+ g.chain=NULL;
}
g_slist_foreach (g.links, (GFunc)gst_parse_free_link, NULL);