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/saxeventkeeperimpl.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/saxeventkeeperimpl.cxx')
| -rw-r--r-- | xmlsecurity/source/framework/saxeventkeeperimpl.cxx | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/xmlsecurity/source/framework/saxeventkeeperimpl.cxx b/xmlsecurity/source/framework/saxeventkeeperimpl.cxx index afe98c50715f..c79ee65af7b3 100644 --- a/xmlsecurity/source/framework/saxeventkeeperimpl.cxx +++ b/xmlsecurity/source/framework/saxeventkeeperimpl.cxx @@ -722,8 +722,6 @@ void SAXEventKeeperImpl::releaseElementMarkBuffer() ElementCollector* pElementCollector = (ElementCollector*)pElementMark; cssxc::sax::ElementMarkPriority nPriority = pElementCollector->getPriority(); - bool bToModify = pElementCollector->getModify(); - /* * Delete the EC from the buffer node. */ @@ -735,11 +733,6 @@ void SAXEventKeeperImpl::releaseElementMarkBuffer() pBufferNode->notifyBranch(); } - if (bToModify) - { - pBufferNode->notifyAncestor(); - } - /* * delete the ElementMark */ |