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>

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;
             }