[Bf-blender-cvs] [4673528] depsgraph_refactor: Removed unused method declarations for component/operation finding directly from the graph.

Lukas Tönne noreply at git.blender.org
Sat Apr 12 12:56:36 CEST 2014


Commit: 467352810aa564563aa190c8ee2b609c196b17c6
Author: Lukas Tönne
Date:   Sat Apr 12 12:38:31 2014 +0200
https://developer.blender.org/rB467352810aa564563aa190c8ee2b609c196b17c6

Removed unused method declarations for component/operation finding
directly from the graph.

The graph should rather not require such deep lookup wrappers. The use
cases for such functionality are all in the build procedure, so it makes
sense to keep the graph as a simple container with raw access only to
the root node types and id nodes. Builder classes can then provide
wrappers for detailed lookup in the respective subnodes.

===================================================================

M	source/blender/depsgraph/intern/depsgraph.h

===================================================================

diff --git a/source/blender/depsgraph/intern/depsgraph.h b/source/blender/depsgraph/intern/depsgraph.h
index 2f5fabf..67ef285 100644
--- a/source/blender/depsgraph/intern/depsgraph.h
+++ b/source/blender/depsgraph/intern/depsgraph.h
@@ -159,13 +159,6 @@ struct Depsgraph {
 	void remove_id_node(const ID *id);
 	void clear_id_nodes();
 	
-	ComponentDepsNode *find_component_node(const ID *id, eDepsNode_Type type, const string &subdata = "");
-	OperationDepsNode *find_operation_node(const ID *id, const string &subdata, eDepsNode_Type type);
-	OperationDepsNode *find_operation_node(const ID *id, eDepsNode_Type type)
-	{
-		return find_operation_node(id, "", type);
-	}
-	
 	/* Remove node from graph, but don't free any of its data */
 	void remove_node(DepsNode *node);




More information about the Bf-blender-cvs mailing list