[Libreoffice-commits] core.git: sax/source

[Hossein (via logerrit)]

 sax/source/fastparser/fastparser.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d1cfe4228316322d0ae9f3b8020036e5140fc32b
Author:     Hossein <hoss...@libreoffice.org>
AuthorDate: Tue Mar 1 14:33:33 2022 +0100
Commit:     Hossein <hoss...@libreoffice.org>
CommitDate: Wed Mar 2 02:11:23 2022 +0100

    Remove 'const' from for loop variable
    
    In the commit f63a6f2e396fa41ed1338dcec874e06159cafa9f, a for loop is
    converted into a range-based for loop. The pointer is used to change
    the nodes using xmlUnlinkNode() and xmlFreeNode(), thus the 'const'
    should have not been used.
    
    The reinterpret_cast does not change the constness, thus I have
    removed the const from the loop variable, and it will no longer be a
    const reference.
    
    Change-Id: If4c61017ea77f464230bb5802f6fc928acc7a7e3
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/130792
    Tested-by: Jenkins
    Reviewed-by: Hossein <hoss...@libreoffice.org>

diff --git a/sax/source/fastparser/fastparser.cxx 
b/sax/source/fastparser/fastparser.cxx
index beb30443ba02..71b316378e01 100644
--- a/sax/source/fastparser/fastparser.cxx
+++ b/sax/source/fastparser/fastparser.cxx
@@ -674,7 +674,7 @@ FastSaxParserImpl::~FastSaxParserImpl()
 {
     if( mxDocumentLocator.is() )
         mxDocumentLocator->dispose();
-    for (const auto& entity : m_TemporalEntities)
+    for (auto& entity : m_TemporalEntities)
     {
         if (!entity)
             continue;