diff options
author | Sunjin Yang <fan4326@gmail.com> | 2011-10-07 15:21:34 +0900 |
---|---|---|
committer | Emmanuele Bassi <ebassi@linux.intel.com> | 2011-10-07 12:17:34 +0100 |
commit | ba1964e16e0da1eb69f9eb67dc4d335d3f1a5ce0 (patch) | |
tree | b446dffbc1fb03b37ba403bee444a42a8ba120f7 | |
parent | af272cab68ca56f3b1947af1952ddd9540fd20b5 (diff) | |
download | clutter-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.c | 26 |
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); |