isa/mutter
1
0
Fork 0
mirror of https://github.com/brl/mutter.git synced 2024-09-20 14:35:48 -04:00
Code Issues Packages Projects Releases Wiki Activity

material-node: track if node owns reference on parent

Browse Source 
MaterialNodes are used for the sparse graph of material state and layer
state. In the case of materials there is the idea of weak materials that
don't take a reference on their parent and in that case we need to be
careful not to unref our parent during
_cogl_material_node_unparent_real. This adds a has_parent_reference
member to the CoglMaterialNode struct so we now know when to skip the
unref.
This commit is contained in:
Robert Bragg 2010-10-26 17:43:16 +01:00
parent e5202d6a0d
commit bbf27e6b28
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
clutter/cogl/cogl/cogl-material-private.h
View File

@ -138,6 +138,10 @@ struct _CoglMaterialNode
/* The parent material/layer */
CoglMaterialNode *parent;
/* TRUE if the node took a strong reference on its parent. Weak
* materials for instance don't take a reference on their parent. */
gboolean has_parent_reference;
/* As an optimization for creating leaf node materials/layers (the
* most common) we don't require any list node allocations to link
* to a single descendant. */

4
clutter/cogl/cogl/cogl-material.c
View File

@ -127,6 +127,7 @@ _cogl_material_node_set_parent_real (CoglMaterialNode *node,
}
node->parent = parent;
node->has_parent_reference = take_strong_reference;
/* Now that there is a consistent parent->child link we can remove
* the parent reference if no reference was requested. If it turns
@ -160,7 +161,8 @@ _cogl_material_node_unparent_real (CoglMaterialNode *node)
else
parent->children = g_list_remove (parent->children, node);
cogl_object_unref (parent);
if (node->has_parent_reference)
cogl_object_unref (parent);
node->parent = NULL;
}