diff options
author | Tor Lillqvist <tml@collabora.com> | 2013-09-04 01:57:11 +0300 |
---|---|---|
committer | Tor Lillqvist <tml@collabora.com> | 2013-09-04 01:57:54 +0300 |
commit | f1dd6b8d6221ef4da982cc9e10a761df52ab2310 (patch) | |
tree | 0e5ac5a656edfe082566ec4c9e5bd4a7b06ae34c /xmlsecurity/source/framework/buffernode.cxx | |
parent | aa4868de78b3a3893722f7a079f4d05e18abd8db (diff) |
WaE: 'BufferNode::notifyAncestor': recursive call has no side effects
Interesting function that just traverses a chain of objects using
recursion, but does nothing at all otherwise. Kill it. But at least it
had an Enterprise Ready block comment in front.
Change-Id: If2296d2534cb9451d4ceadaf966b8c70f0a17557
Diffstat (limited to 'xmlsecurity/source/framework/buffernode.cxx')
-rw-r--r-- | xmlsecurity/source/framework/buffernode.cxx | 32 |
1 files changed, 0 insertions, 32 deletions
diff --git a/xmlsecurity/source/framework/buffernode.cxx b/xmlsecurity/source/framework/buffernode.cxx index 64a5a516a5bf..bea9a958f740 100644 --- a/xmlsecurity/source/framework/buffernode.cxx +++ b/xmlsecurity/source/framework/buffernode.cxx @@ -788,38 +788,6 @@ void BufferNode::notifyBranch() } } -void BufferNode::notifyAncestor() -/****** BufferNode/notifyAncestor ******************************************** - * - * NAME - * notifyAncestor -- notifies each ancestor BufferNode through the parent - * link. - * - * SYNOPSIS - * notifyAncestor(); - * - * FUNCTION - * see NAME - * - * INPUTS - * empty - * - * RESULT - * empty - * - * AUTHOR - * Michael Mi - * Email: michael.mi@sun.com - ******************************************************************************/ -{ - BufferNode* pParent = m_pParent; - while (pParent != NULL) - { - pParent->notifyAncestor(); - pParent = (BufferNode*)pParent->getParent(); - } -} - void BufferNode::elementCollectorNotify() /****** BufferNode/elementCollectorNotify ************************************ * |