From af3fd19637d0a4d4d24e5fde964e17271efc9dcd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tim-Philipp=20M=C3=BCller?= <tim@centricular.com>
Date: Fri, 27 Feb 2015 01:26:24 +0000
Subject: [PATCH] ges-base-xml-formatter: fix setting of child properties

Make sure all child properties get set. GstStructureForeachFunc
takes a gboolean return value that decides whether to
continue or not.
---
 ges/ges-base-xml-formatter.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/ges/ges-base-xml-formatter.c b/ges/ges-base-xml-formatter.c
index 6f81f94858..7b78d012d6 100644
--- a/ges/ges-base-xml-formatter.c
+++ b/ges/ges-base-xml-formatter.c
@@ -479,20 +479,22 @@ _loading_done_cb (GESFormatter * self)
   return FALSE;
 }
 
-static void
+static gboolean
 _set_child_property (GQuark field_id, const GValue * value,
     GESTrackElement * effect)
 {
   GParamSpec *pspec;
   GstElement *element;
 
+  /* FIXME: error handling? */
   if (!ges_track_element_lookup_child (effect,
           g_quark_to_string (field_id), &element, &pspec))
-    return;
+    return TRUE;
 
   g_object_set_property (G_OBJECT (element), pspec->name, value);
   g_param_spec_unref (pspec);
   gst_object_unref (element);
+  return TRUE;
 }
 
 gboolean