diff options
author | Lukas Tönne <lukas.toenne@gmail.com> | 2014-04-15 18:06:12 +0400 |
---|---|---|
committer | Lukas Tönne <lukas.toenne@gmail.com> | 2014-04-15 18:28:10 +0400 |
commit | 09874df135888b89f51d7becaa369ebb1d1623c6 (patch) | |
tree | bccf5950d42963992adff95dbc3f00ed7db1cf16 /source/blender/compositor/intern/COM_NodeBase.h | |
parent | 28a829893c702918afc5ac1945a06eaefa611594 (diff) |
Structural cleanup and improvements for the compositor.
Many parts of the compositor are unnecessarily complicated. This patch
aims at reducing the complexity of writing nodes and making the code
more transparent.
== Separating Nodes and Operations ==
Currently these are both mixed in the same graph, even though they have
very different purposes and are used at distinct stages in the
compositing process. The patch introduces dedicated graph classes for
nodes and for operations.
This removes the need for a lot of special case checks (isOperation etc.)
and explicit type casts. It simplifies the code since it becomes clear
at every stage what type of node we are dealing with. The compiler can
use static typing to avoid common bugs from mixing up these types and
fewer runtime sanity checks are needed.
== Simplified Node Conversion ==
Converting nodes to operations was previously based on "relinking", i.e.
nodes would start with by mirroring links in the Blender DNA node trees,
then add operations and redirect these links to them. This was very hard
to follow in many cases and required a lot of attention to avoid invalid
states.
Now there is a helper class called the NodeConverter, which is passed to
nodes and implements a much simpler API for this process. Nodes can add
operations and explicit connections as before, but defining "external"
links to the inputs/outputs of the original node now uses mapping
instead of directly modifying link data. Input data (node graph) and
result (operations graph) are cleanly separated.
== Removed Redundant Data Structures ==
A few redundant data structures have been removed, notably the
SocketConnection. These are only needed temporarily during graph
construction. For executing the compositor operations it is perfectly
sufficient to store only the direct input link pointers. A common
pointer indirection is avoided this way (which might also give a little
performance improvement).
== Avoid virtual recursive functions ==
Recursive virtual functions are evil. They are very hard to follow
during debugging. At least in the parts this patch is concerned with
these functions have been replaced by a non-virtual recursive core
function (which might then call virtual non-recursive functions if
needed). See for example NodeOperationBuilder::group_operations.
Diffstat (limited to 'source/blender/compositor/intern/COM_NodeBase.h')
-rw-r--r-- | source/blender/compositor/intern/COM_NodeBase.h | 185 |
1 files changed, 0 insertions, 185 deletions
diff --git a/source/blender/compositor/intern/COM_NodeBase.h b/source/blender/compositor/intern/COM_NodeBase.h deleted file mode 100644 index e2072575509..00000000000 --- a/source/blender/compositor/intern/COM_NodeBase.h +++ /dev/null @@ -1,185 +0,0 @@ -/* - * Copyright 2011, Blender Foundation. - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License - * as published by the Free Software Foundation; either version 2 - * of the License, or (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software Foundation, - * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. - * - * Contributor: - * Jeroen Bakker - * Monique Dewanchand - */ - -#ifndef __COM_NODEBASE_H__ -#define __COM_NODEBASE_H__ - -#include "COM_InputSocket.h" -#include "COM_OutputSocket.h" -#include "DNA_node_types.h" -#include "BKE_text.h" -#include <vector> -#include <string> - -using namespace std; - - -class NodeOperation; -class ExecutionSystem; - -/** - * @brief The NodeBase class is the super-class of all node related objects like @see Node @see NodeOperation - * the reason for the existence of this class is to support graph-nodes when using ExecutionSystem - * the NodeBase also contains the reference to InputSocket and OutputSocket. - * @ingroup Model - */ -class NodeBase { -private: - /** - * @brief the list of actual inputsockets @see InputSocket - */ - vector<InputSocket *> m_inputsockets; - - /** - * @brief the list of actual outputsockets @see OutputSocket - */ - vector<OutputSocket *> m_outputsockets; - - /** - * @brief stores the reference to the SDNA bNode struct - */ - bNode *m_editorNode; - - /** - * @brief stores the reference to the SDNA bNode struct - */ - bNodeTree *m_editorNodeTree; - -protected: - /** - * @brief get access to the vector of input sockets - */ - inline vector<InputSocket *>& getInputSockets() { return this->m_inputsockets; } - - /** - * @brief get access to the vector of input sockets - */ - inline vector<OutputSocket *>& getOutputSockets() { return this->m_outputsockets; } - - -protected: - /** - * @brief destructor - * clean up memory related to this NodeBase. - */ - virtual ~NodeBase(); - -public: - /** - * @brief get the reference to the SDNA bNode struct - */ - bNode *getbNode() const {return m_editorNode;} - - /** - * @brief get the reference to the SDNA bNodeTree struct - */ - bNodeTree *getbNodeTree() const {return m_editorNodeTree;} - - /** - * @brief set the reference to the bNode - * @note used in Node instances to receive the storage/settings and complex node for highlight during execution - * @param bNode - */ - void setbNode(bNode *node) {this->m_editorNode = node;} - - /** - * @brief set the reference to the bNodeTree - * @param bNodeTree - */ - void setbNodeTree(bNodeTree *nodetree) {this->m_editorNodeTree = nodetree;} - - /** - * @brief is this node an operation? - * This is true when the instance is of the subclass NodeOperation. - * @return [true:false] - * @see NodeOperation - */ - virtual const bool isOperation() const { return false; } - - /** - * @brief check if this is an input node - * An input node is a node that only has output sockets and no input sockets - * @return [false..true] - */ - const bool isInputNode() const; - - /** - * @brief Return the number of input sockets of this node. - */ - const unsigned int getNumberOfInputSockets() const { return this->m_inputsockets.size(); } - - /** - * @brief Return the number of output sockets of this node. - */ - const unsigned int getNumberOfOutputSockets() const { return this->m_outputsockets.size(); } - - /** - * get the reference to a certain outputsocket - * @param index - * the index of the needed outputsocket - */ - OutputSocket *getOutputSocket(const unsigned int index); - - /** - * get the reference to the first outputsocket - * @param index - * the index of the needed outputsocket - */ - inline OutputSocket *getOutputSocket() { return getOutputSocket(0); } - - /** - * get the reference to a certain inputsocket - * @param index - * the index of the needed inputsocket - */ - InputSocket *getInputSocket(const unsigned int index); - - virtual bool isStatic() const { return false; } - void getStaticValues(float *result) const { } - -protected: - NodeBase(); - - /** - * @brief add an InputSocket to the collection of inputsockets - * @note may only be called in an constructor - * @param socket the InputSocket to add - */ - void addInputSocket(DataType datatype); - void addInputSocket(DataType datatype, InputSocketResizeMode resizeMode); - void addInputSocket(DataType datatype, InputSocketResizeMode resizeMode, bNodeSocket *socket); - - /** - * @brief add an OutputSocket to the collection of outputsockets - * @note may only be called in an constructor - * @param socket the OutputSocket to add - */ - void addOutputSocket(DataType datatype); - void addOutputSocket(DataType datatype, bNodeSocket *socket); - - -#ifdef WITH_CXX_GUARDEDALLOC - MEM_CXX_CLASS_ALLOC_FUNCS("COM:NodeBase") -#endif -}; - -#endif /* __COM_NODEBASE_H__ */ |