diff options
author | Noel Grandin <noel@peralex.com> | 2016-04-12 16:39:03 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2016-04-18 07:37:31 +0000 |
commit | 789055bc2acb4c71483fd60ea258d158bd5aec10 (patch) | |
tree | 7849de841a71f667a30b2a971ad0c3d406110396 /xmlsecurity | |
parent | 150ac9cf05ed9da6a2af5bc3f820280fd853e519 (diff) |
clang-tidy performance-unnecessary-copy-initialization
probably not much performance benefit, but it sure is good at
identifying leftover intermediate variables from previous
refactorings.
Change-Id: I3ce16fe496ac2733c1cb0a35f74c0fc9193cc657
Reviewed-on: https://gerrit.libreoffice.org/24026
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'xmlsecurity')
-rw-r--r-- | xmlsecurity/source/xmlsec/xmldocumentwrapper_xmlsecimpl.cxx | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/xmlsecurity/source/xmlsec/xmldocumentwrapper_xmlsecimpl.cxx b/xmlsecurity/source/xmlsec/xmldocumentwrapper_xmlsecimpl.cxx index b05f7c89a50f..5f525caf5b5f 100644 --- a/xmlsecurity/source/xmlsec/xmldocumentwrapper_xmlsecimpl.cxx +++ b/xmlsecurity/source/xmlsec/xmldocumentwrapper_xmlsecimpl.cxx @@ -748,20 +748,19 @@ void SAL_CALL XMLDocumentWrapper_XmlSecImpl::getTree( const cssu::Reference< css m_pCurrentElement = m_pRootElement; m_nCurrentPosition = NODEPOSITION_STARTELEMENT; - cssu::Reference< cssxs::XDocumentHandler > xHandler = handler; while(true) { switch (m_nCurrentPosition) { case NODEPOSITION_STARTELEMENT: - sendStartElement(nullptr, xHandler, m_pCurrentElement); + sendStartElement(nullptr, handler, m_pCurrentElement); break; case NODEPOSITION_ENDELEMENT: - sendEndElement(nullptr, xHandler, m_pCurrentElement); + sendEndElement(nullptr, handler, m_pCurrentElement); break; case NODEPOSITION_NORMAL: - sendNode(nullptr, xHandler, m_pCurrentElement); + sendNode(nullptr, handler, m_pCurrentElement); break; } |