parser: update for childproxy api changes
[platform/upstream/gstreamer.git] / gst / parse / grammar.y
index a7cb76d..b48072e 100644 (file)
@@ -1,26 +1,47 @@
 %{
+#include "../gst_private.h"
+
 #include <glib-object.h>
 #include <glib.h>
 #include <stdio.h>
 #include <string.h>
 #include <stdlib.h>
 
-#include "../gst_private.h"
 #include "../gst-i18n-lib.h"
 
 #include "../gstconfig.h"
 #include "../gstparse.h"
 #include "../gstinfo.h"
+#include "../gsterror.h"
+#include "../gststructure.h"
 #include "../gsturi.h"
+#include "../gstutils.h"
+#include "../gstvalue.h"
+#include "../gstchildproxy.h"
 #include "types.h"
 
 /* All error messages in this file are user-visible and need to be translated.
  * Don't start the message with a capital, and don't end them with a period,
  * as they will be presented inside a sentence/error.
  */
-  
+
 #define YYERROR_VERBOSE 1
-#define YYPARSE_PARAM graph
+#define YYLEX_PARAM scanner
+
+#define YYENABLE_NLS 0
+
+#ifndef YYLTYPE_IS_TRIVIAL
+#define YYLTYPE_IS_TRIVIAL 0
+#endif
+
+typedef void* yyscan_t;
+
+int priv_gst_parse_yylex (void * yylval_param , yyscan_t yyscanner);
+int priv_gst_parse_yylex_init (yyscan_t scanner);
+int priv_gst_parse_yylex_destroy (yyscan_t scanner);
+struct yy_buffer_state * priv_gst_parse_yy_scan_string (char* , yyscan_t);
+void _gst_parse_yypush_buffer_state (void * new_buffer ,yyscan_t yyscanner );
+void _gst_parse_yypop_buffer_state (yyscan_t yyscanner );
 
 #ifdef __GST_PARSE_TRACE
 static guint __strings;
@@ -29,7 +50,7 @@ static guint __chains;
 gchar *
 __gst_parse_strdup (gchar *org)
 {
-  gchar *ret; 
+  gchar *ret;
   __strings++;
   ret = g_strdup (org);
   /* g_print ("ALLOCATED STR   (%3u): %p %s\n", __strings, ret, ret); */
@@ -49,7 +70,7 @@ link_t *__gst_parse_link_new ()
 {
   link_t *ret;
   __links++;
-  ret = g_new0 (link_t, 1);
+  ret = g_slice_new0 (link_t);
   /* g_print ("ALLOCATED LINK  (%3u): %p\n", __links, ret); */
   return ret;
 }
@@ -58,7 +79,7 @@ __gst_parse_link_free (link_t *data)
 {
   if (data) {
     /* g_print ("FREEING LINK    (%3u): %p\n", __links - 1, data); */
-    g_free (data);
+    g_slice_free (link_t, data);
     g_return_if_fail (__links > 0);
     __links--;
   }
@@ -68,19 +89,17 @@ __gst_parse_chain_new ()
 {
   chain_t *ret;
   __chains++;
-  ret = g_new0 (chain_t, 1);
+  ret = g_slice_new0 (chain_t);
   /* g_print ("ALLOCATED CHAIN (%3u): %p\n", __chains, ret); */
   return ret;
 }
 void
 __gst_parse_chain_free (chain_t *data)
 {
-  if (data) {
-    /* g_print ("FREEING CHAIN   (%3u): %p\n", __chains - 1, data); */
-    g_free (data);
-    g_return_if_fail (__chains > 0);
-    __chains--;
-  }
+  /* g_print ("FREEING CHAIN   (%3u): %p\n", __chains - 1, data); */
+  g_slice_free (chain_t, data);
+  g_return_if_fail (__chains > 0);
+  __chains--;
 }
 
 #endif /* __GST_PARSE_TRACE */
@@ -91,110 +110,150 @@ typedef struct {
   GstElement *sink;
   GstCaps *caps;
   gulong signal_id;
-  /* FIXME: need to connect to "disposed" signal to clean up, but there is no such signal */
 } DelayedLink;
 
+typedef struct {
+  gchar *name;
+  gchar *value_str;
+  gulong signal_id;
+} DelayedSet;
+
+/*** define SET_ERROR macro/function */
+
 #ifdef G_HAVE_ISO_VARARGS
-#define SET_ERROR(error, type, ...) G_STMT_START{ \
-  if (error) { \
-    if (*(error)) { \
-      g_warning (__VA_ARGS__); \
-    } else { \
-      g_set_error ((error), GST_PARSE_ERROR, (type), __VA_ARGS__); \
-    }\
+
+#  define SET_ERROR(error, type, ...) \
+G_STMT_START { \
+  GST_CAT_ERROR (GST_CAT_PIPELINE, __VA_ARGS__); \
+  if ((error) && !*(error)) { \
+    g_set_error ((error), GST_PARSE_ERROR, (type), __VA_ARGS__); \
   } \
-}G_STMT_END
-#define ERROR(type, ...) SET_ERROR (((graph_t *) graph)->error, (type), __VA_ARGS__ )
-#ifndef GST_DISABLE_GST_DEBUG
-#  define YYDEBUG 1
-   /* bison 1.35 calls this macro with side effects, we need to make sure the
-      side effects work - crappy bison
-#  define YYFPRINTF(a, ...) GST_CAT_DEBUG (GST_CAT_PIPELINE, __VA_ARGS__)
- */
-#  define YYFPRINTF(a, ...) G_STMT_START{ \
-     gchar *temp = g_strdup_printf (__VA_ARGS__); \
-     GST_CAT_DEBUG (GST_CAT_PIPELINE, temp); \
-     g_free (temp); \
-   }G_STMT_END
-#endif
+} G_STMT_END
 
 #elif defined(G_HAVE_GNUC_VARARGS)
 
-#define SET_ERROR(error, type, args...) G_STMT_START{ \
-  if (error) { \
-    if (*(error)) { \
-      g_warning ( ## args ); \
-    } else { \
-      g_set_error ((error), GST_PARSE_ERROR, (type), ## args ); \
-    }\
+#  define SET_ERROR(error, type, args...) \
+G_STMT_START { \
+  GST_CAT_ERROR (GST_CAT_PIPELINE, args ); \
+  if ((error) && !*(error)) { \
+    g_set_error ((error), GST_PARSE_ERROR, (type), args ); \
   } \
-}G_STMT_END
-#define ERROR(type, args...) SET_ERROR (((graph_t *) graph)->error, (type), ## args )
-#ifndef GST_DISABLE_GST_DEBUG
-#  define YYDEBUG 1
-   /* bison 1.35 calls this macro with side effects, we need to make sure the
-      side effects work - crappy bison
-#  define YYFPRINTF(a, args...) GST_CAT_DEBUG (GST_CAT_PIPELINE, ## args )
- */
-#  define YYFPRINTF(a, args...) G_STMT_START{ \
-     gchar *temp = g_strdup_printf ( ## args ); \
-     GST_CAT_DEBUG (GST_CAT_PIPELINE, temp); \
-     g_free (temp); \
-   }G_STMT_END
-#endif
+} G_STMT_END
 
 #else
 
-#define SET_ERROR(error, type, ...) G_STMT_START{ \
-  if (error) { \
-    if (*(error)) { \
-      g_warning ("error while parsing"); \
-    } else { \
-      g_set_error ((error), GST_PARSE_ERROR, (type), "error while parsing"); \
-    }\
-  } \
-}G_STMT_END
-#define ERROR(type, ...) SET_ERROR (((graph_t *) graph)->error, (type), "error while parsing")
+static inline void
+SET_ERROR (GError **error, gint type, const char *format, ...)
+{
+  if (error) {
+    if (*error) {
+      g_warning ("error while parsing");
+    } else {
+      va_list varargs;
+      char *string;
+
+      va_start (varargs, format);
+      string = g_strdup_vprintf (format, varargs);
+      va_end (varargs);
+
+      g_set_error (error, GST_PARSE_ERROR, type, string);
+
+      g_free (string);
+    }
+  }
+}
+
+#endif /* G_HAVE_ISO_VARARGS */
+
+/*** define YYPRINTF macro/function if we're debugging */
+
+/* bison 1.35 calls this macro with side effects, we need to make sure the
+   side effects work - crappy bison */
+
 #ifndef GST_DISABLE_GST_DEBUG
 #  define YYDEBUG 1
-#endif
 
-#endif /* G_HAVE_ISO_VARARGS */
+#  ifdef G_HAVE_ISO_VARARGS
+
+/* #  define YYFPRINTF(a, ...) GST_CAT_DEBUG (GST_CAT_PIPELINE, __VA_ARGS__) */
+#    define YYFPRINTF(a, ...) \
+G_STMT_START { \
+     GST_CAT_LOG (GST_CAT_PIPELINE, __VA_ARGS__); \
+} G_STMT_END
+
+#  elif defined(G_HAVE_GNUC_VARARGS)
 
-#define GST_BIN_MAKE(res, type, chainval, assign) G_STMT_START{ \
+#    define YYFPRINTF(a, args...) \
+G_STMT_START { \
+     GST_CAT_LOG (GST_CAT_PIPELINE, args); \
+} G_STMT_END
+
+#  else
+
+static inline void
+YYPRINTF(const char *format, ...)
+{
+  va_list varargs;
+  gchar *temp;
+
+  va_start (varargs, format);
+  temp = g_strdup_vprintf (format, varargs);
+  GST_CAT_LOG (GST_CAT_PIPELINE, "%s", temp);
+  g_free (temp);
+  va_end (varargs);
+}
+
+#  endif /* G_HAVE_ISO_VARARGS */
+
+#endif /* GST_DISABLE_GST_DEBUG */
+
+#define ADD_MISSING_ELEMENT(graph,name) G_STMT_START {                      \
+    if ((graph)->ctx) {                                                     \
+      (graph)->ctx->missing_elements =                                      \
+          g_list_append ((graph)->ctx->missing_elements, g_strdup (name));  \
+    } } G_STMT_END
+
+static void
+no_free (gconstpointer foo)
+{
+  /* do nothing */
+}
+
+#define GST_BIN_MAKE(res, type, chainval, assign, type_string_free_func) \
+G_STMT_START { \
   chain_t *chain = chainval; \
   GSList *walk; \
   GstBin *bin = (GstBin *) gst_element_factory_make (type, NULL); \
   if (!chain) { \
-    ERROR (GST_PARSE_ERROR_EMPTY_BIN, _("specified empty bin \"%s\", not allowed"), type); \
+    SET_ERROR (graph->error, GST_PARSE_ERROR_EMPTY_BIN, \
+        _("specified empty bin \"%s\", not allowed"), type); \
     g_slist_foreach (assign, (GFunc) gst_parse_strfree, NULL); \
     g_slist_free (assign); \
+    gst_object_unref (bin); \
+    type_string_free_func (type); /* Need to clean up the string */ \
     YYERROR; \
   } else if (!bin) { \
-    ERROR (GST_PARSE_ERROR_NO_SUCH_ELEMENT, _("no bin \"%s\", skipping"), type); \
+    ADD_MISSING_ELEMENT(graph, type); \
+    SET_ERROR (graph->error, GST_PARSE_ERROR_NO_SUCH_ELEMENT, \
+        _("no bin \"%s\", skipping"), type); \
     g_slist_foreach (assign, (GFunc) gst_parse_strfree, NULL); \
     g_slist_free (assign); \
     res = chain; \
   } else { \
-    walk = chain->elements; \
-    while (walk) { \
+    for (walk = chain->elements; walk; walk = walk->next ) \
       gst_bin_add (bin, GST_ELEMENT (walk->data)); \
-      walk = walk->next; \
-    } \
     g_slist_free (chain->elements); \
     chain->elements = g_slist_prepend (NULL, bin); \
     res = chain; \
     /* set the properties now */ \
-    walk = assign; \
-    while (walk) { \
+    for (walk = assign; walk; walk = walk->next) \
       gst_parse_element_set ((gchar *) walk->data, GST_ELEMENT (bin), graph); \
-      walk = g_slist_next (walk); \
-    } \
     g_slist_free (assign); \
   } \
-}G_STMT_END
+} G_STMT_END
 
-#define MAKE_LINK(link, _src, _src_name, _src_pads, _sink, _sink_name, _sink_pads) G_STMT_START{ \
+#define MAKE_LINK(link, _src, _src_name, _src_pads, _sink, _sink_name, _sink_pads) \
+G_STMT_START { \
   link = gst_parse_link_new (); \
   link->src = _src; \
   link->sink = _sink; \
@@ -203,9 +262,10 @@ typedef struct {
   link->src_pads = _src_pads; \
   link->sink_pads = _sink_pads; \
   link->caps = NULL; \
-}G_STMT_END
+} G_STMT_END
 
-#define MAKE_REF(link, _src, _pads) G_STMT_START{ \
+#define MAKE_REF(link, _src, _pads) \
+G_STMT_START { \
   gchar *padname = _src; \
   GSList *pads = _pads; \
   if (padname) { \
@@ -216,123 +276,194 @@ typedef struct {
       pads = g_slist_prepend (pads, gst_parse_strdup (padname)); \
   } \
   MAKE_LINK (link, NULL, _src, pads, NULL, NULL, NULL); \
-}G_STMT_END
+} G_STMT_END
+
+static void
+gst_parse_free_delayed_set (DelayedSet *set)
+{
+  g_free(set->name);
+  g_free(set->value_str);
+  g_slice_free(DelayedSet, set);
+}
+
+static void gst_parse_new_child(GstChildProxy *child_proxy, GObject *object,
+                                gpointer data);
+
+static void
+gst_parse_add_delayed_set (GstElement *element, gchar *name, gchar *value_str)
+{
+  DelayedSet *data = g_slice_new0 (DelayedSet);
+
+  GST_CAT_LOG_OBJECT (GST_CAT_PIPELINE, element, "delaying property set %s to %s",
+    name, value_str);
+
+  data->name = g_strdup(name);
+  data->value_str = g_strdup(value_str);
+  data->signal_id = g_signal_connect_data(element, "child-added",
+      G_CALLBACK (gst_parse_new_child), data, (GClosureNotify)
+      gst_parse_free_delayed_set, (GConnectFlags) 0);
+
+  /* FIXME: we would need to listen on all intermediate bins too */
+  if (GST_IS_BIN (element)) {
+    gchar **names, **current;
+    GstElement *parent, *child;
+
+    current = names = g_strsplit (name, "::", -1);
+    parent = gst_bin_get_by_name (GST_BIN_CAST (element), current[0]);
+    current++;
+    while (parent && current[0]) {
+      child = gst_bin_get_by_name (GST_BIN (parent), current[0]);
+      if (!child && current[1]) {
+        char *sub_name = g_strjoinv ("::", &current[0]);
+
+        gst_parse_add_delayed_set(parent, sub_name, value_str);
+        g_free (sub_name);
+      }
+      parent = child;
+      current++;
+    }
+    g_strfreev (names);
+  }
+}
+
+static void gst_parse_new_child(GstChildProxy *child_proxy, GObject *object,
+                                gpointer data)
+{
+  DelayedSet *set = (DelayedSet *) data;
+  GParamSpec *pspec;
+  GValue v = { 0, };
+  GstObject *target = NULL;
+  GType value_type;
+
+  GST_CAT_LOG_OBJECT (GST_CAT_PIPELINE, child_proxy, "new child %s, checking property %s",
+      GST_OBJECT_NAME(object), set->name);
+
+  if (gst_child_proxy_lookup (G_OBJECT (child_proxy), set->name, &target, &pspec)) {
+    gboolean got_value = FALSE;
+
+    value_type = pspec->value_type;
+
+    GST_CAT_LOG_OBJECT (GST_CAT_PIPELINE, child_proxy, "parsing delayed property %s as a %s from %s",
+      pspec->name, g_type_name (value_type), set->value_str);
+    g_value_init (&v, value_type);
+    if (gst_value_deserialize (&v, set->value_str))
+      got_value = TRUE;
+    else if (g_type_is_a (value_type, GST_TYPE_ELEMENT)) {
+       GstElement *bin;
+
+       bin = gst_parse_bin_from_description (set->value_str, TRUE, NULL);
+       if (bin) {
+         g_value_set_object (&v, bin);
+         got_value = TRUE;
+       }
+    }
+    g_signal_handler_disconnect (child_proxy, set->signal_id);
+    if (!got_value)
+      goto error;
+    g_object_set_property (G_OBJECT (target), pspec->name, &v);
+  } else {
+    const gchar *obj_name = GST_OBJECT_NAME(object);
+    gint len = strlen (obj_name);
+
+    /* do a delayed set */
+    if ((strlen (set->name) > (len + 2)) && !strncmp (set->name, obj_name, len) && !strncmp (&set->name[len], "::", 2)) {
+      gst_parse_add_delayed_set (GST_ELEMENT(child_proxy), set->name, set->value_str);
+    }
+  }
+
+out:
+  if (G_IS_VALUE (&v))
+    g_value_unset (&v);
+  if (target)
+    gst_object_unref (target);
+  return;
+
+error:
+  GST_CAT_ERROR (GST_CAT_PIPELINE, "could not set property \"%s\" in element \"%s\"",
+        pspec->name, GST_ELEMENT_NAME (target));
+  goto out;
+}
 
 static void
 gst_parse_element_set (gchar *value, GstElement *element, graph_t *graph)
 {
   GParamSpec *pspec;
   gchar *pos = value;
+  GValue v = { 0, };
+  GstObject *target = NULL;
+  GType value_type;
+
+  /* do nothing if assignment is for missing element */
+  if (element == NULL)
+    goto out;
+
   /* parse the string, so the property name is null-terminated an pos points
      to the beginning of the value */
-  while (!g_ascii_isspace (*pos) && (*pos != '=')) pos++; 
-  if (*pos == '=') { 
-    *pos = '\0'; 
-  } else { 
-    *pos = '\0'; 
+  while (!g_ascii_isspace (*pos) && (*pos != '=')) pos++;
+  if (*pos == '=') {
+    *pos = '\0';
+  } else {
+    *pos = '\0';
     pos++;
-    while (g_ascii_isspace (*pos)) pos++; 
-  } 
-  pos++; 
-  while (g_ascii_isspace (*pos)) pos++; 
+    while (g_ascii_isspace (*pos)) pos++;
+  }
+  pos++;
+  while (g_ascii_isspace (*pos)) pos++;
   if (*pos == '"') {
     pos++;
     pos[strlen (pos) - 1] = '\0';
   }
-  gst_parse_unescape (pos); 
-  if ((pspec = g_object_class_find_property (G_OBJECT_GET_CLASS (element), value))) { 
-    GValue v = { 0, }; 
-    GValue v2 = { 0, };
-    g_value_init (&v, G_PARAM_SPEC_VALUE_TYPE(pspec)); 
-    switch (G_TYPE_FUNDAMENTAL (G_PARAM_SPEC_VALUE_TYPE (pspec))) {
-    case G_TYPE_STRING:
-      g_value_set_string (&v, pos);
-      break;      
-    case G_TYPE_BOOLEAN:
-      if (g_ascii_strcasecmp (pos, "true") && g_ascii_strcasecmp (pos, "yes") && g_ascii_strcasecmp (pos, "1")) {
-        g_value_set_boolean (&v, FALSE);
-      } else {
-        g_value_set_boolean (&v, TRUE);
-      }
-      break;
-    case G_TYPE_ENUM: {
-      GEnumValue *en;
-      gchar *endptr = NULL;
-      GEnumClass *klass = (GEnumClass *) g_type_class_peek (G_PARAM_SPEC_VALUE_TYPE (pspec));
-      if (klass == NULL) goto error;
-      if (!(en = g_enum_get_value_by_name (klass, pos))) {
-        if (!(en = g_enum_get_value_by_nick (klass, pos))) {
-          gint i = strtol (pos, &endptr, 0);
-         if (endptr && *endptr == '\0') {
-           en = g_enum_get_value (klass, i);
-         }
-        }
-      }
-      if (!en)
-       goto error;
-      g_value_set_enum (&v, en->value);
-      break;
-    }
-    case G_TYPE_INT:
-    case G_TYPE_LONG:
-    case G_TYPE_INT64: {
-      gchar *endptr;
-      glong l;
-      g_value_init (&v2, G_TYPE_LONG); 
-      l = strtol (pos, &endptr, 0);
-      if (*endptr != '\0') goto error_conversion;
-      g_value_set_long (&v2, l);
-      if (!g_value_transform (&v2, &v)) goto error_conversion;
-      break;      
-    }
-    case G_TYPE_UINT:
-    case G_TYPE_ULONG:
-    case G_TYPE_UINT64: {
-      gchar *endptr;
-      gulong ul;
-      g_value_init (&v2, G_TYPE_ULONG); 
-      ul = strtoul (pos, &endptr, 0);
-      if (*endptr != '\0') goto error_conversion;
-      g_value_set_ulong (&v2, ul);
-      if (!g_value_transform (&v2, &v)) goto error_conversion;
-      break;      
-    }
-    case G_TYPE_FLOAT:
-    case G_TYPE_DOUBLE: {
-      gchar *endptr;
-      gdouble d;
-      g_value_init (&v2, G_TYPE_DOUBLE); 
-      d = g_ascii_strtod (pos, &endptr);
-      if (*endptr != '\0') goto error_conversion;
-      g_value_set_double (&v2, d);
-      if (!g_value_transform (&v2, &v)) goto error_conversion;
-      break;      
+  gst_parse_unescape (pos);
+
+  if (gst_child_proxy_lookup (G_OBJECT (element), value, &target, &pspec)) {
+    gboolean got_value = FALSE;
+
+    value_type = pspec->value_type;
+
+    GST_CAT_LOG_OBJECT (GST_CAT_PIPELINE, element, "parsing property %s as a %s", pspec->name,
+      g_type_name (value_type));
+    g_value_init (&v, value_type);
+    if (gst_value_deserialize (&v, pos))
+      got_value = TRUE;
+    else if (g_type_is_a (value_type, GST_TYPE_ELEMENT)) {
+       GstElement *bin;
+
+       bin = gst_parse_bin_from_description (pos, TRUE, NULL);
+       if (bin) {
+         g_value_set_object (&v, bin);
+         got_value = TRUE;
+       }
     }
-    default:
-      /* add more */
-      g_warning ("property \"%s\" in element %s cannot be set", value, GST_ELEMENT_NAME (element)); 
+    if (!got_value)
       goto error;
+    g_object_set_property (G_OBJECT (target), pspec->name, &v);
+  } else {
+    /* do a delayed set */
+    if (GST_IS_CHILD_PROXY (element)) {
+      gst_parse_add_delayed_set (element, value, pos);
+    }
+    else {
+      SET_ERROR (graph->error, GST_PARSE_ERROR_NO_SUCH_PROPERTY, \
+          _("no property \"%s\" in element \"%s\""), value, \
+          GST_ELEMENT_NAME (element));
     }
-    g_object_set_property (G_OBJECT (element), value, &v); 
-  } else { 
-    ERROR (GST_PARSE_ERROR_NO_SUCH_PROPERTY, _("no property \"%s\" in element \"%s\""), value, GST_ELEMENT_NAME (element)); 
   }
 
 out:
   gst_parse_strfree (value);
+  if (G_IS_VALUE (&v))
+    g_value_unset (&v);
+  if (target)
+    gst_object_unref (target);
   return;
-  
+
 error:
-  ERROR (GST_PARSE_ERROR_COULD_NOT_SET_PROPERTY,
-         _("could not set property \"%s\" in element \"%s\" to \"%s\""), 
-        value, GST_ELEMENT_NAME (element), pos); 
-  goto out;
-error_conversion:
-  ERROR (GST_PARSE_ERROR_COULD_NOT_SET_PROPERTY,
-         _("could not convert \"%s\" so that it fits property \"%s\" in element \"%s\""),
-         pos, value, GST_ELEMENT_NAME (element)); 
+  SET_ERROR (graph->error, GST_PARSE_ERROR_COULD_NOT_SET_PROPERTY,
+         _("could not set property \"%s\" in element \"%s\" to \"%s\""),
+        value, GST_ELEMENT_NAME (element), pos);
   goto out;
 }
+
 static inline void
 gst_parse_free_link (link_t *link)
 {
@@ -342,89 +473,60 @@ gst_parse_free_link (link_t *link)
   g_slist_foreach (link->sink_pads, (GFunc) gst_parse_strfree, NULL);
   g_slist_free (link->src_pads);
   g_slist_free (link->sink_pads);
-  if (link->caps) gst_caps_free (link->caps);
-  gst_parse_link_free (link);  
+  if (link->caps) gst_caps_unref (link->caps);
+  gst_parse_link_free (link);
 }
+
 static void
-gst_parse_element_lock (GstElement *element, gboolean lock)
+gst_parse_free_delayed_link (DelayedLink *link)
 {
-  GstPad *pad;
-  GList *walk = (GList *) gst_element_get_pad_list (element);
-  gboolean unlocked_peer = FALSE;
-  
-  if (gst_element_is_locked_state (element) == lock)
-    return;
-  /* check if we have an unlocked peer */
-  while (walk) {
-    pad = (GstPad *) GST_PAD_REALIZE (walk->data);
-    walk = walk->next;
-    if (GST_PAD_IS_SINK (pad) && GST_PAD_PEER (pad) &&
-        !gst_element_is_locked_state (GST_PAD_PARENT (GST_PAD_PEER (pad)))) {
-      unlocked_peer = TRUE;
-      break;
-    }
-  }  
-  
-  if (!(lock && unlocked_peer)) {
-    gst_element_set_locked_state (element, lock);
-    if (!lock)
-      gst_element_sync_state_with_parent (element);
-  } else {
-    return;
-  }
-  
-  /* check if there are other pads to (un)lock */
-  walk = (GList *) gst_element_get_pad_list (element);
-  while (walk) {
-    pad = (GstPad *) GST_PAD_REALIZE (walk->data);
-    walk = walk->next;
-    if (GST_PAD_IS_SRC (pad) && GST_PAD_PEER (pad)) {
-      GstElement *next = GST_ELEMENT (GST_OBJECT_PARENT (GST_PAD_PEER (pad)));
-      if (gst_element_is_locked_state (next) != lock)
-        gst_parse_element_lock (next, lock);
-    }
-  }
+  g_free (link->src_pad);
+  g_free (link->sink_pad);
+  if (link->caps) gst_caps_unref (link->caps);
+  g_slice_free (DelayedLink, link);
 }
+
 static void
 gst_parse_found_pad (GstElement *src, GstPad *pad, gpointer data)
 {
-  DelayedLink *link = (DelayedLink *) data;
-  
-  GST_CAT_INFO (GST_CAT_PIPELINE, "trying delayed linking %s:%s to %s:%s", 
-                GST_ELEMENT_NAME (src), link->src_pad,
-                GST_ELEMENT_NAME (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_ELEMENT_NAME (src), link->src_pad,
-                          GST_ELEMENT_NAME (link->sink), link->sink_pad);
+  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));
+
+  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);
-    g_free (link->src_pad);
-    g_free (link->sink_pad);
-    if (link->caps) gst_caps_free (link->caps);
-    if (!gst_element_is_locked_state (src))
-      gst_parse_element_lock (link->sink, FALSE);
-    g_free (link);
   }
 }
+
 /* both padnames and the caps may be NULL */
 static gboolean
-gst_parse_perform_delayed_link (GstElement *src, const gchar *src_pad, 
-                                GstElement *sink, const gchar *sink_pad, GstCaps *caps)
+gst_parse_perform_delayed_link (GstElement *src, const gchar *src_pad,
+                                GstElement *sink, const gchar *sink_pad,
+                                GstCaps *caps)
 {
-  GList *templs = gst_element_get_pad_template_list (src);
-        
-  while (templs) {
+  GList *templs = gst_element_class_get_pad_template_list (
+      GST_ELEMENT_GET_CLASS (src));
+
+  for (; templs; templs = templs->next) {
     GstPadTemplate *templ = (GstPadTemplate *) templs->data;
-    if ((GST_PAD_TEMPLATE_DIRECTION (templ) == GST_PAD_SRC) && (GST_PAD_TEMPLATE_PRESENCE(templ) == GST_PAD_SOMETIMES))
+    if ((GST_PAD_TEMPLATE_DIRECTION (templ) == GST_PAD_SRC) &&
+        (GST_PAD_TEMPLATE_PRESENCE(templ) == GST_PAD_SOMETIMES))
     {
-      DelayedLink *data = g_new (DelayedLink, 1); 
-      
+      DelayedLink *data = g_slice_new (DelayedLink);
+
       /* 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_ELEMENT_NAME (src), src_pad, GST_ELEMENT_NAME (sink), sink_pad);
+      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));
 
       data->src_pad = g_strdup (src_pad);
       data->sink = sink;
@@ -434,14 +536,15 @@ gst_parse_perform_delayed_link (GstElement *src, const gchar *src_pad,
       } else {
        data->caps = NULL;
       }
-      data->signal_id = g_signal_connect (G_OBJECT (src), "new_pad", 
-                                         G_CALLBACK (gst_parse_found_pad), data);
+      data->signal_id = g_signal_connect_data (src, "pad-added",
+          G_CALLBACK (gst_parse_found_pad), data,
+          (GClosureNotify) gst_parse_free_delayed_link, (GConnectFlags) 0);
       return TRUE;
     }
-    templs = g_list_next (templs);
   }
   return FALSE;
 }
+
 /*
  * performs a link and frees the struct. src and sink elements must be given
  * return values   0 - link performed
@@ -457,30 +560,29 @@ gst_parse_perform_link (link_t *link, graph_t *graph)
   GSList *sinks = link->sink_pads;
   g_assert (GST_IS_ELEMENT (src));
   g_assert (GST_IS_ELEMENT (sink));
-  
-  GST_CAT_INFO (GST_CAT_PIPELINE, "linking %s(%s):%u to %s(%s):%u with caps \"%s\"", 
-                GST_ELEMENT_NAME (src), link->src_name ? link->src_name : "---", g_slist_length (srcs),
-                GST_ELEMENT_NAME (sink), link->sink_name ? link->sink_name : "---", g_slist_length (sinks),
-               link->caps ? gst_caps_to_string (link->caps) : "-");
+
+  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)",
+      g_slist_length (srcs), g_slist_length (sinks), link->caps);
 
   if (!srcs || !sinks) {
-    if (gst_element_link_pads_filtered (src, srcs ? (const gchar *) srcs->data : NULL,
-                                        sink, sinks ? (const gchar *) sinks->data : NULL,
-                                       link->caps)) {
-      gst_parse_element_lock (sink, gst_element_is_locked_state (src));
+    if (gst_element_link_pads_filtered (src,
+        srcs ? (const gchar *) srcs->data : NULL, sink,
+        sinks ? (const gchar *) sinks->data : NULL, link->caps)) {
       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)) {
-        gst_parse_element_lock (sink, TRUE);
+      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;
       }
     }
   }
-  if (g_slist_length (link->src_pads) != g_slist_length (link->src_pads)) {
+  if (g_slist_length (link->src_pads) != g_slist_length (link->sink_pads)) {
     goto error;
   }
   while (srcs && sinks) {
@@ -488,34 +590,34 @@ gst_parse_perform_link (link_t *link, graph_t *graph)
     const gchar *sink_pad = (const gchar *) sinks->data;
     srcs = g_slist_next (srcs);
     sinks = g_slist_next (sinks);
-    if (gst_element_link_pads_filtered (src, src_pad, sink, sink_pad, link->caps)) {
-      gst_parse_element_lock (sink, gst_element_is_locked_state (src));
+    if (gst_element_link_pads_filtered (src, src_pad, sink, sink_pad,
+        link->caps)) {
       continue;
     } else {
       if (gst_parse_perform_delayed_link (src, src_pad,
                                           sink, sink_pad,
                                          link->caps)) {
-        gst_parse_element_lock (sink, TRUE);
        continue;
       } else {
         goto error;
       }
     }
   }
-  
+
 success:
   gst_parse_free_link (link);
   return 0;
-  
+
 error:
-  ERROR (GST_PARSE_ERROR_LINK, _("could not link %s to %s"), GST_ELEMENT_NAME (src), GST_ELEMENT_NAME (sink));
+  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;
 }
 
 
-static int yylex (void *lvalp);
-static int yyerror (const char *s);
+static int yyerror (void *scanner, graph_t *graph, const char *s);
 %}
 
 %union {
@@ -538,25 +640,34 @@ static int yyerror (const char *s);
 %type <l> reference
 %type <l> linkpart link
 %type <p> linklist
-%type <e> element 
+%type <e> element
 %type <p> padlist pads assignments
 
-%left '{' '}' '(' ')'
+%left '(' ')'
 %left ','
 %right '.'
 %left '!' '='
 
-%pure_parser
+%parse-param { void *scanner }
+%parse-param { graph_t *graph }
+%pure-parser
 
 %start graph
 %%
 
-element:       IDENTIFIER                    { $$ = gst_element_factory_make ($1, NULL); 
-                                               if (!$$)
-                                                 ERROR (GST_PARSE_ERROR_NO_SUCH_ELEMENT, _("no element \"%s\""), $1);
+element:       IDENTIFIER                    { $$ = gst_element_factory_make ($1, NULL);
+                                               if ($$ == NULL) {
+                                                 ADD_MISSING_ELEMENT (graph, $1);
+                                                 SET_ERROR (graph->error, GST_PARSE_ERROR_NO_SUCH_ELEMENT, _("no element \"%s\""), $1);
+                                                 /* if FATAL_ERRORS flag is set, we don't have to worry about backwards
+                                                  * compatibility and can continue parsing and check for other missing
+                                                  * elements */
+                                                 if ((graph->flags & GST_PARSE_FLAG_FATAL_ERRORS) == 0) {
+                                                   gst_parse_strfree ($1);
+                                                   YYERROR;
+                                                 }
+                                               }
                                                gst_parse_strfree ($1);
-                                               if (!$$)
-                                                 YYERROR;
                                               }
        |       element ASSIGNMENT            { gst_parse_element_set ($2, $1, graph);
                                                $$ = $1;
@@ -564,33 +675,28 @@ element:  IDENTIFIER                    { $$ = gst_element_factory_make ($1, NULL);
        ;
 assignments:   /* NOP */                     { $$ = NULL; }
        |       assignments ASSIGNMENT        { $$ = g_slist_prepend ($1, $2); }
-       ;               
-bin:           '{' assignments chain '}' { GST_BIN_MAKE ($$, "thread", $3, $2); }
-        |       '(' assignments chain ')' { GST_BIN_MAKE ($$, "bin", $3, $2); }
-        |       BINREF assignments chain ')'  { GST_BIN_MAKE ($$, $1, $3, $2); 
+       ;
+bin:           '(' assignments chain ')' { GST_BIN_MAKE ($$, "bin", $3, $2, no_free); }
+        |       BINREF assignments chain ')'  { GST_BIN_MAKE ($$, $1, $3, $2, gst_parse_strfree);
                                                gst_parse_strfree ($1);
                                              }
-       |       '{' assignments '}'           { GST_BIN_MAKE ($$, "thread", NULL, $2); }
-       |       '(' assignments '}'           { GST_BIN_MAKE ($$, "thread", NULL, $2); }
-        |       BINREF assignments ')'       { GST_BIN_MAKE ($$, $1, NULL, $2); 
+        |       BINREF assignments ')'       { GST_BIN_MAKE ($$, $1, NULL, $2, gst_parse_strfree);
                                                gst_parse_strfree ($1);
                                              }
-       |       '{' assignments error '}'     { GST_BIN_MAKE ($$, "thread", NULL, $2); }
-       |       '(' assignments error '}'     { GST_BIN_MAKE ($$, "thread", NULL, $2); }
-        |       BINREF assignments error ')'  { GST_BIN_MAKE ($$, $1, NULL, $2); 
+        |       BINREF assignments error ')'  { GST_BIN_MAKE ($$, $1, NULL, $2, gst_parse_strfree);
                                                gst_parse_strfree ($1);
                                              }
        ;
-       
+
 pads:          PADREF                        { $$ = g_slist_prepend (NULL, $1); }
        |       PADREF padlist                { $$ = $2;
                                                $$ = g_slist_prepend ($$, $1);
-                                             }                              
+                                             }
        ;
 padlist:       ',' IDENTIFIER                { $$ = g_slist_prepend (NULL, $2); }
        |       ',' IDENTIFIER padlist        { $$ = g_slist_prepend ($3, $2); }
        ;
-       
+
 reference:     REF                           { MAKE_REF ($$, $1, NULL); }
        |       REF padlist                   { MAKE_REF ($$, $1, $2); }
        ;
@@ -599,12 +705,12 @@ linkpart: reference                     { $$ = $1; }
        |       pads                          { MAKE_REF ($$, NULL, $1); }
        |       /* NOP */                     { MAKE_REF ($$, NULL, NULL); }
        ;
-       
+
 link:          linkpart LINK linkpart        { $$ = $1;
                                                if ($2) {
                                                  $$->caps = gst_caps_from_string ($2);
-                                                 if (!$$->caps)
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("could not parse caps \"%s\""), $2);
+                                                 if ($$->caps == NULL)
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("could not parse caps \"%s\""), $2);
                                                  gst_parse_strfree ($2);
                                                }
                                                $$->sink_name = $3->src_name;
@@ -612,11 +718,11 @@ link:             linkpart LINK linkpart        { $$ = $1;
                                                gst_parse_link_free ($3);
                                              }
        ;
-       
+
 linklist:      link                          { $$ = g_slist_prepend (NULL, $1); }
        |       link linklist                 { $$ = g_slist_prepend ($2, $1); }
        |       linklist error                { $$ = $1; }
-       ;       
+       ;
 
 chain:         element                       { $$ = gst_parse_chain_new ();
                                                $$->first = $$->last = $1;
@@ -626,16 +732,16 @@ chain:    element                       { $$ = gst_parse_chain_new ();
        |       bin                           { $$ = $1; }
        |       chain chain                   { if ($1->back && $2->front) {
                                                  if (!$1->back->sink_name) {
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("link without source element"));
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link without source element"));
                                                    gst_parse_free_link ($1->back);
                                                  } else {
-                                                   ((graph_t *) graph)->links = g_slist_prepend (((graph_t *) graph)->links, $1->back);
+                                                   graph->links = g_slist_prepend (graph->links, $1->back);
                                                  }
                                                  if (!$2->front->src_name) {
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("link without sink element"));
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link without sink element"));
                                                    gst_parse_free_link ($2->front);
                                                  } else {
-                                                   ((graph_t *) graph)->links = g_slist_prepend (((graph_t *) graph)->links, $2->front);
+                                                   graph->links = g_slist_prepend (graph->links, $2->front);
                                                  }
                                                  $1->back = NULL;
                                                } else if ($1->back) {
@@ -648,19 +754,15 @@ chain:    element                       { $$ = gst_parse_chain_new ();
                                                  }
                                                  $1->back = $2->front;
                                                }
-                                               
+
                                                if ($1->back) {
-                                                 if (!$1->back->sink || !$1->back->src) {
-                                                   ((graph_t *) graph)->links = g_slist_prepend (((graph_t *) graph)->links, $1->back);
-                                                   $1->back = NULL;
-                                                 } else {
-                                                   gst_parse_perform_link ($1->back, (graph_t *) graph);
-                                                 }
+                                                 graph->links = g_slist_prepend (graph->links, $1->back);
                                                }
                                                $1->last = $2->last;
                                                $1->back = $2->back;
                                                $1->elements = g_slist_concat ($1->elements, $2->elements);
-                                               gst_parse_chain_free ($2);
+                                               if ($2)
+                                                 gst_parse_chain_free ($2);
                                                $$ = $1;
                                              }
        |       chain linklist                { GSList *walk;
@@ -670,21 +772,19 @@ chain:    element                       { $$ = gst_parse_chain_new ();
                                                } else {
                                                  if (!((link_t *) $2->data)->src_name) {
                                                    ((link_t *) $2->data)->src = $1->last;
-                                                 }                                               
+                                                 }
                                                }
-                                               walk = $2;
-                                               while (walk) {
+                                               for (walk = $2; walk; walk = walk->next) {
                                                  link_t *link = (link_t *) walk->data;
-                                                 walk = walk->next;
-                                                 if (!link->sink_name && walk) {
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("link without sink element"));
+                                                 if (!link->sink_name && walk->next) {
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link without sink element"));
                                                    gst_parse_free_link (link);
                                                  } else if (!link->src_name && !link->src) {
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("link without source element"));
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link without source element"));
                                                    gst_parse_free_link (link);
                                                  } else {
-                                                   if (walk) {
-                                                     ((graph_t *) graph)->links = g_slist_prepend (((graph_t *) graph)->links, link);
+                                                   if (walk->next) {
+                                                     graph->links = g_slist_prepend (graph->links, link);
                                                    } else {
                                                      $1->back = link;
                                                    }
@@ -696,10 +796,10 @@ chain:    element                       { $$ = gst_parse_chain_new ();
        |       chain error                   { $$ = $1; }
        |       link chain                    { if ($2->front) {
                                                  if (!$2->front->src_name) {
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("link without source element"));
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link without source element"));
                                                    gst_parse_free_link ($2->front);
                                                  } else {
-                                                   ((graph_t *) graph)->links = g_slist_prepend (((graph_t *) graph)->links, $2->front);
+                                                   graph->links = g_slist_prepend (graph->links, $2->front);
                                                  }
                                                }
                                                if (!$1->sink_name) {
@@ -710,20 +810,20 @@ chain:    element                       { $$ = gst_parse_chain_new ();
                                              }
        |       PARSE_URL chain               { $$ = $2;
                                                if ($$->front) {
-                                                 GstElement *element = 
+                                                 GstElement *element =
                                                          gst_element_make_from_uri (GST_URI_SRC, $1, NULL);
                                                  if (!element) {
-                                                   ERROR (GST_PARSE_ERROR_NO_SUCH_ELEMENT, 
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_NO_SUCH_ELEMENT,
                                                            _("no source element for URI \"%s\""), $1);
                                                  } else {
                                                    $$->front->src = element;
-                                                   ((graph_t *) graph)->links = g_slist_prepend (
-                                                           ((graph_t *) graph)->links, $$->front);
+                                                   graph->links = g_slist_prepend (
+                                                           graph->links, $$->front);
                                                    $$->front = NULL;
                                                    $$->elements = g_slist_prepend ($$->elements, element);
                                                  }
                                                } else {
-                                                 ERROR (GST_PARSE_ERROR_LINK, 
+                                                 SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
                                                          _("no element to link URI \"%s\" to"), $1);
                                                }
                                                g_free ($1);
@@ -731,12 +831,17 @@ chain:    element                       { $$ = gst_parse_chain_new ();
        |       link PARSE_URL                { GstElement *element =
                                                          gst_element_make_from_uri (GST_URI_SINK, $2, NULL);
                                                if (!element) {
-                                                 ERROR (GST_PARSE_ERROR_NO_SUCH_ELEMENT, 
+                                                 SET_ERROR (graph->error, GST_PARSE_ERROR_NO_SUCH_ELEMENT,
                                                          _("no sink element for URI \"%s\""), $2);
+                                                 gst_parse_link_free ($1);
+                                                 g_free ($2);
                                                  YYERROR;
                                                } else if ($1->sink_name || $1->sink_pads) {
-                                                 ERROR (GST_PARSE_ERROR_LINK, 
+                                                  gst_object_unref (element);
+                                                 SET_ERROR (graph->error, GST_PARSE_ERROR_LINK,
                                                          _("could not link sink element for URI \"%s\""), $2);
+                                                 gst_parse_link_free ($1);
+                                                 g_free ($2);
                                                  YYERROR;
                                                } else {
                                                  $$ = gst_parse_chain_new ();
@@ -748,13 +853,13 @@ chain:    element                       { $$ = gst_parse_chain_new ();
                                                g_free ($2);
                                              }
        ;
-graph:         /* NOP */                     { ERROR (GST_PARSE_ERROR_EMPTY, _("empty pipeline not allowed"));
-                                               $$ = (graph_t *) graph;
+graph:         /* NOP */                     { SET_ERROR (graph->error, GST_PARSE_ERROR_EMPTY, _("empty pipeline not allowed"));
+                                               $$ = graph;
                                              }
-       |       chain                         { $$ = (graph_t *) graph;
+       |       chain                         { $$ = graph;
                                                if ($1->front) {
                                                  if (!$1->front->src_name) {
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("link without source element"));
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link without source element"));
                                                    gst_parse_free_link ($1->front);
                                                  } else {
                                                    $$->links = g_slist_prepend ($$->links, $1->front);
@@ -763,7 +868,7 @@ graph:              /* NOP */                     { ERROR (GST_PARSE_ERROR_EMPTY, _("empty pipeline not a
                                                }
                                                if ($1->back) {
                                                  if (!$1->back->sink_name) {
-                                                   ERROR (GST_PARSE_ERROR_LINK, _("link without sink element"));
+                                                   SET_ERROR (graph->error, GST_PARSE_ERROR_LINK, _("link without sink element"));
                                                    gst_parse_free_link ($1->back);
                                                  } else {
                                                    $$->links = g_slist_prepend ($$->links, $1->back);
@@ -776,96 +881,109 @@ graph:           /* NOP */                     { ERROR (GST_PARSE_ERROR_EMPTY, _("empty pipeline not a
 
 %%
 
-extern FILE *_gst_parse_yyin;
-int _gst_parse_yylex (YYSTYPE *lvalp);
-
-static int yylex (void *lvalp) {
-    return _gst_parse_yylex ((YYSTYPE*) lvalp);
-}
 
 static int
-yyerror (const char *s)
+yyerror (void *scanner, graph_t *graph, const char *s)
 {
   /* FIXME: This should go into the GError somehow, but how? */
-  g_warning ("error: %s\n", s);
+  GST_WARNING ("Error during parsing: %s", s);
   return -1;
 }
 
-int _gst_parse_yy_scan_string (char*);
+
 GstElement *
-_gst_parse_launch (const gchar *str, GError **error)
+priv_gst_parse_launch (const gchar *str, GError **error, GstParseContext *ctx,
+    GstParseFlags flags)
 {
   graph_t g;
   gchar *dstr;
   GSList *walk;
   GstBin *bin = NULL;
   GstElement *ret;
-  
+  yyscan_t scanner;
+
   g_return_val_if_fail (str != NULL, NULL);
+  g_return_val_if_fail (error == NULL || *error == NULL, NULL);
 
   g.chain = NULL;
   g.links = NULL;
   g.error = error;
-  
+  g.ctx = ctx;
+  g.flags = flags;
+
 #ifdef __GST_PARSE_TRACE
   GST_CAT_DEBUG (GST_CAT_PIPELINE, "TRACE: tracing enabled");
   __strings = __chains = __links = 0;
 #endif /* __GST_PARSE_TRACE */
 
   dstr = g_strdup (str);
-  _gst_parse_yy_scan_string (dstr);
+  priv_gst_parse_yylex_init (&scanner);
+  priv_gst_parse_yy_scan_string (dstr, scanner);
 
-#ifndef GST_DISABLE_GST_DEBUG
+#ifndef YYDEBUG
   yydebug = 1;
 #endif
 
-  if (yyparse (&g) != 0) {
-    SET_ERROR (error, GST_PARSE_ERROR_SYNTAX, "Unrecoverable syntax error while parsing pipeline");
-    
+  if (yyparse (scanner, &g) != 0) {
+    SET_ERROR (error, GST_PARSE_ERROR_SYNTAX,
+        "Unrecoverable syntax error while parsing pipeline %s", str);
+
+    priv_gst_parse_yylex_destroy (scanner);
+    g_free (dstr);
+
     goto error1;
   }
+  priv_gst_parse_yylex_destroy (scanner);
   g_free (dstr);
-  
-  GST_CAT_INFO (GST_CAT_PIPELINE, "got %u elements and %u links", g.chain ? g_slist_length (g.chain->elements) : 0, g_slist_length (g.links));
-  
+
+  GST_CAT_DEBUG (GST_CAT_PIPELINE, "got %u elements and %u links",
+      g.chain ? g_slist_length (g.chain->elements) : 0,
+      g_slist_length (g.links));
+
   if (!g.chain) {
     ret = NULL;
-  } else if (!(((chain_t *) g.chain)->elements->next)) {
-    /* only one toplevel element */  
-    ret = (GstElement *) ((chain_t *) g.chain)->elements->data;
-    g_slist_free (((chain_t *) g.chain)->elements);
+  } else if (!g.chain->elements->next) {
+    /* only one toplevel element */
+    ret = (GstElement *) g.chain->elements->data;
+    g_slist_free (g.chain->elements);
     if (GST_IS_BIN (ret))
       bin = GST_BIN (ret);
-  } else {  
+    gst_parse_chain_free (g.chain);
+  } else {
     /* put all elements in our bin */
     bin = GST_BIN (gst_element_factory_make ("pipeline", NULL));
     g_assert (bin);
-    walk = g.chain->elements;
-    while (walk) {
-      gst_bin_add (bin, GST_ELEMENT (walk->data));
-      walk = g_slist_next (walk);  
+
+    for (walk = g.chain->elements; walk; walk = walk->next) {
+      if (walk->data != NULL)
+        gst_bin_add (bin, GST_ELEMENT (walk->data));
     }
+
     g_slist_free (g.chain->elements);
     ret = GST_ELEMENT (bin);
+    gst_parse_chain_free (g.chain);
   }
-  gst_parse_chain_free (g.chain);
-  
+
   /* remove links */
-  walk = g.links;
-  while (walk) {
+  for (walk = g.links; walk; walk = walk->next) {
     link_t *l = (link_t *) walk->data;
-    GstElement *sink;
-    walk = g_slist_next (walk);
     if (!l->src) {
       if (l->src_name) {
         if (bin) {
           l->src = gst_bin_get_by_name_recurse_up (bin, l->src_name);
+          if (l->src)
+            gst_object_unref (l->src);
         } else {
           l->src = strcmp (GST_ELEMENT_NAME (ret), l->src_name) == 0 ? ret : NULL;
         }
       }
       if (!l->src) {
-        SET_ERROR (error, GST_PARSE_ERROR_NO_SUCH_ELEMENT, "No element named \"%s\" - omitting link", l->src_name);
+        if (l->src_name) {
+          SET_ERROR (error, GST_PARSE_ERROR_NO_SUCH_ELEMENT,
+              "No element named \"%s\" - omitting link", l->src_name);
+        } else {
+          /* probably a missing element which we've handled already */
+        }
         gst_parse_free_link (l);
         continue;
       }
@@ -874,53 +992,53 @@ _gst_parse_launch (const gchar *str, GError **error)
       if (l->sink_name) {
         if (bin) {
           l->sink = gst_bin_get_by_name_recurse_up (bin, l->sink_name);
+          if (l->sink)
+            gst_object_unref (l->sink);
         } else {
           l->sink = strcmp (GST_ELEMENT_NAME (ret), l->sink_name) == 0 ? ret : NULL;
         }
       }
       if (!l->sink) {
-        SET_ERROR (error, GST_PARSE_ERROR_NO_SUCH_ELEMENT, "No element named \"%s\" - omitting link", l->sink_name);
+        if (l->sink_name) {
+          SET_ERROR (error, GST_PARSE_ERROR_NO_SUCH_ELEMENT,
+              "No element named \"%s\" - omitting link", l->sink_name);
+        } else {
+          /* probably a missing element which we've handled already */
+        }
         gst_parse_free_link (l);
         continue;
       }
     }
-    sink = l->sink;
     gst_parse_perform_link (l, &g);
   }
   g_slist_free (g.links);
 
 out:
 #ifdef __GST_PARSE_TRACE
-  GST_CAT_DEBUG (GST_CAT_PIPELINE, "TRACE: %u strings, %u chains and %u links left", __strings, __chains, __links);
+  GST_CAT_DEBUG (GST_CAT_PIPELINE,
+      "TRACE: %u strings, %u chains and %u links left", __strings, __chains,
+      __links);
   if (__strings || __chains || __links) {
-    g_warning ("TRACE: %u strings, %u chains and %u links left", __strings, __chains, __links);
+    g_warning ("TRACE: %u strings, %u chains and %u links left", __strings,
+        __chains, __links);
   }
 #endif /* __GST_PARSE_TRACE */
 
   return ret;
-  
+
 error1:
-  g_free (dstr);
-  
   if (g.chain) {
-    walk = g.chain->elements;
-    while (walk) {
-      gst_object_unref (GST_OBJECT (walk->data));
-      walk = walk->next;
-    }
+    g_slist_foreach (g.chain->elements, (GFunc)gst_object_unref, NULL);
     g_slist_free (g.chain->elements);
+    gst_parse_chain_free (g.chain);
   }
-  gst_parse_chain_free (g.chain);
-  
-  walk = g.links;
-  while (walk) {
-    gst_parse_free_link ((link_t *) walk->data);
-    walk = walk->next;
-  }
+
+  g_slist_foreach (g.links, (GFunc)gst_parse_free_link, NULL);
   g_slist_free (g.links);
-  
-  g_assert (*error);
+
+  if (error)
+    g_assert (*error);
   ret = NULL;
-  
+
   goto out;
 }