isa/mutter
1
0
Fork 0
mirror of https://github.com/brl/mutter.git synced 2024-11-23 00:20:42 -05:00
Code Issues Packages Projects Releases Wiki Activity

clutter-group: Use g_list_foreach in clutter_group_real_foreach

Browse Source 
g_list_foreach has better protection against the current node being
removed. This will happen for example if someone calls
clutter_container_foreach(container, clutter_actor_destroy). This was
causing valgrind errors for the conformance tests which do just that.
This commit is contained in:
Neil Roberts 2010-01-18 12:35:05 +00:00
parent 3d373c7278
commit ce030a3fce
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
clutter/clutter-group.c
View File

@ -277,10 +277,12 @@ clutter_group_real_foreach (ClutterContainer *container,
{ {
ClutterGroup *group = CLUTTER_GROUP (container); ClutterGroup *group = CLUTTER_GROUP (container);
ClutterGroupPrivate *priv = group->priv; ClutterGroupPrivate *priv = group->priv;
GList *l;
for (l = priv->children; l; l = l->next) /* Using g_list_foreach instead of iterating the list manually
(* callback) (CLUTTER_ACTOR (l->data), user_data); because it has better protection against the current node being
removed. This will happen for example if someone calls
clutter_container_foreach(container, clutter_actor_destroy) */
g_list_foreach (priv->children, (GFunc) callback, user_data);
} }
static void static void