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

Jochen Nitschke (via logerrit) logerrit at kemper.freedesktop.org
Sat May 1 17:03:24 UTC 2021


 xmloff/source/style/xmlexppr.cxx |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit ce7729ab473b971164822338661d1a4828d860d2
Author:     Jochen Nitschke <j.nitschke+logerrit at ok.de>
AuthorDate: Sat May 1 16:02:00 2021 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Sat May 1 19:02:42 2021 +0200

    remove superfluous loop variable
    
    Change-Id: I418ea4b1770b82541bae8901d8d1b78d36cfb5d2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114920
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/xmloff/source/style/xmlexppr.cxx b/xmloff/source/style/xmlexppr.cxx
index bd3ee809ec1a..129a568065c1 100644
--- a/xmloff/source/style/xmlexppr.cxx
+++ b/xmloff/source/style/xmlexppr.cxx
@@ -267,8 +267,7 @@ void FilterPropertiesInfo_Impl::FillPropertyStateArray(
                 const beans::GetDirectPropertyTolerantResult *pResults = aResults.getConstArray();
                 FilterPropertyInfoList_Impl::iterator aPropIter(aPropInfos.begin());
                 XMLPropertyState aNewProperty( -1 );
-                size_t i = 0;
-                while (nResultCount > 0 && i < aPropInfos.size())
+                while (nResultCount > 0 && aPropIter != aPropInfos.end())
                 {
                     if (pResults->Name == aPropIter->GetApiName())
                     {
@@ -284,7 +283,6 @@ void FilterPropertiesInfo_Impl::FillPropertyStateArray(
                         --nResultCount;
                     }
                     ++aPropIter;
-                    ++i;
                 }
             }
         }


More information about the Libreoffice-commits mailing list