From 8f15f63212389aac582db2f99696a9c7d6d2714d Mon Sep 17 00:00:00 2001 From: Robert Bragg Date: Tue, 26 Oct 2010 17:43:16 +0100 Subject: [PATCH] material-node: track if node owns reference on parent 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. --- cogl/cogl-material-private.h | 4 ++++ cogl/cogl-material.c | 4 +++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/cogl/cogl-material-private.h b/cogl/cogl-material-private.h index eb20b3860..88dbfc0e8 100644 --- a/cogl/cogl-material-private.h +++ b/cogl/cogl-material-private.h @@ -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. */ diff --git a/cogl/cogl-material.c b/cogl/cogl-material.c index 905e0e5fd..720350821 100644 --- a/cogl/cogl-material.c +++ b/cogl/cogl-material.c @@ -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; }