framework/source/fwe/classes/addonsoptions.cxx |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit c3ff4d1bc34c31a8d3f2fbb434984d39a6cea8c3
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Mon Jan 27 09:18:31 2020 +0100
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Mon Jan 27 11:46:27 2020 +0100

    Use properly typed variables for iteration
    
    Change-Id: I3c8a8adc20fe404befa360e49ab42f2ffd93d27f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87489
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/framework/source/fwe/classes/addonsoptions.cxx 
b/framework/source/fwe/classes/addonsoptions.cxx
index 6a5ef90cbe19..2fa0307e37d4 100644
--- a/framework/source/fwe/classes/addonsoptions.cxx
+++ b/framework/source/fwe/classes/addonsoptions.cxx
@@ -996,7 +996,7 @@ void AddonsOptions_Impl::ReadMergeMenuData( const OUString& 
aMergeAddonInstructi
     aMergeMenuBaseNode += m_aPathDelimiter;
 
     // extend the node names to have full path strings
-    for ( sal_uInt32 i = 0; i < 
static_cast<sal_uInt32>(aSubMenuNodeNames.getLength()); i++ )
+    for ( sal_Int32 i = 0; i < aSubMenuNodeNames.getLength(); i++ )
         aSubMenuNodeNames[i] = aMergeMenuBaseNode + aSubMenuNodeNames[i];
 
     ReadSubMenuEntries( aSubMenuNodeNames, rMergeMenu );
@@ -1323,7 +1323,7 @@ bool AddonsOptions_Impl::ReadMenuItem( const OUString& 
aMenuNodeName, Sequence<
             // Continue to read the sub menu nodes
             Sequence< Sequence< PropertyValue > > aSubMenuSeq;
             OUString aSubMenuRootNodeName( aRootSubMenuName + m_aPathDelimiter 
);
-            for ( sal_uInt32 n = 0; n < 
static_cast<sal_uInt32>(aRootSubMenuNodeNames.getLength()); n++ )
+            for ( sal_Int32 n = 0; n < aRootSubMenuNodeNames.getLength(); n++ )
                 aRootSubMenuNodeNames[n] = aSubMenuRootNodeName + 
aRootSubMenuNodeNames[n];
             ReadSubMenuEntries( aRootSubMenuNodeNames, aSubMenuSeq );
             aMenuItem[ OFFSET_MENUITEM_SUBMENU ].Value <<= aSubMenuSeq;
@@ -1387,7 +1387,7 @@ bool AddonsOptions_Impl::ReadPopupMenu( const OUString& 
aPopupMenuNodeName, Sequ
             // Continue to read the sub menu nodes
             Sequence< Sequence< PropertyValue > > aSubMenuSeq;
             OUString aSubMenuRootNodeName( aRootSubMenuName + m_aPathDelimiter 
);
-            for ( sal_uInt32 n = 0; n < 
static_cast<sal_uInt32>(aRootSubMenuNodeNames.getLength()); n++ )
+            for ( sal_Int32 n = 0; n < aRootSubMenuNodeNames.getLength(); n++ )
                 aRootSubMenuNodeNames[n] = aSubMenuRootNodeName + 
aRootSubMenuNodeNames[n];
             ReadSubMenuEntries( aRootSubMenuNodeNames, aSubMenuSeq );
             aPopupMenu[ OFFSET_POPUPMENU_SUBMENU ].Value <<= aSubMenuSeq;
@@ -1408,7 +1408,7 @@ void AddonsOptions_Impl::AppendPopupMenu( Sequence< 
PropertyValue >& rTargetPopu
     {
         sal_uInt32 nIndex = aTargetSubMenuSeq.getLength();
         aTargetSubMenuSeq.realloc( nIndex + aSourceSubMenuSeq.getLength() );
-        for ( sal_uInt32 i = 0; i < sal_uInt32( aSourceSubMenuSeq.getLength() 
); i++ )
+        for ( sal_Int32 i = 0; i < aSourceSubMenuSeq.getLength(); i++ )
             aTargetSubMenuSeq[nIndex++] = aSourceSubMenuSeq[i];
         rTargetPopupMenu[ OFFSET_POPUPMENU_SUBMENU ].Value <<= 
aTargetSubMenuSeq;
     }
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to