summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSunjin Yang <fan4326@gmail.com>2011-10-07 15:21:34 +0900
committerEmmanuele Bassi <ebassi@linux.intel.com>2011-10-07 12:17:34 +0100
commitba1964e16e0da1eb69f9eb67dc4d335d3f1a5ce0 (patch)
treeb446dffbc1fb03b37ba403bee444a42a8ba120f7
parentaf272cab68ca56f3b1947af1952ddd9540fd20b5 (diff)
downloadclutter-ba1964e16e0da1eb69f9eb67dc4d335d3f1a5ce0.tar.gz
box-layout: Plug memory leaks
https://bugzilla.gnome.org/show_bug.cgi?id=660985 (cherry picked from commit 5dbc00e625c515a6732eaec06ddd6185ad8b5916) Signed-off-by: Emmanuele Bassi <ebassi@linux.intel.com>
-rw-r--r--clutter/clutter-box-layout.c26
1 files changed, 12 insertions, 14 deletions
diff --git a/clutter/clutter-box-layout.c b/clutter/clutter-box-layout.c
index ba71599bb..8a7c8ad26 100644
--- a/clutter/clutter-box-layout.c
+++ b/clutter/clutter-box-layout.c
@@ -749,16 +749,15 @@ count_expand_children (ClutterLayoutManager *layout,
gint *visible_children,
gint *expand_children)
{
- GList *children;
+ GList *children, *l;
ClutterActor *child;
*visible_children = *expand_children = 0;
- for (children = clutter_container_get_children (container);
- children;
- children = children->next)
+ children = clutter_container_get_children (container);
+ for (l = children; l != NULL; l = l->next)
{
- child = children->data;
+ child = l->data;
if (CLUTTER_ACTOR_IS_VISIBLE (child))
{
@@ -901,7 +900,7 @@ clutter_box_layout_allocate (ClutterLayoutManager *layout,
{
ClutterBoxLayoutPrivate *priv = CLUTTER_BOX_LAYOUT (layout)->priv;
ClutterActor *child;
- GList *children;
+ GList *children, *l;
gint nvis_children;
gint nexpand_children;
gboolean is_rtl;
@@ -929,11 +928,10 @@ clutter_box_layout_allocate (ClutterLayoutManager *layout,
size = box->x2 - box->x1 - (nvis_children - 1) * priv->spacing;
/* Retrieve desired size for visible children. */
- for (i = 0, children = clutter_container_get_children (container);
- children;
- children = children->next)
+ children = clutter_container_get_children (container);
+ for (i = 0, l = children; l != NULL; l = l->next)
{
- child = children->data;
+ child = l->data;
if (!CLUTTER_ACTOR_IS_VISIBLE (child))
continue;
@@ -1037,14 +1035,14 @@ clutter_box_layout_allocate (ClutterLayoutManager *layout,
}
children = clutter_container_get_children (container);
- for (i = g_list_length (children) - 1, children = g_list_last (children);
- children;
- children = children->prev, i--)
+ for (i = g_list_length (children) - 1, l = g_list_last (children);
+ l != NULL;
+ l = l->prev, i--)
{
ClutterLayoutMeta *meta;
ClutterBoxChild *box_child;
- child = children->data;
+ child = l->data;
meta = clutter_layout_manager_get_child_meta (layout,
container,
child);