loplugin:loopvartoosmall

(cherry picked from commit 55bc128636596032c23ee855904822ad813986fc)

Change-Id: I86ff38a90018a2ddfb2db3babf67168b0e6257a5

26 files changed, 40 insertions, 40 deletions

diff --git a/compilerplugins/clang/loopvartoosmall.cxx b/compilerplugins/clang/loopvartoosmall.cxx
index e52b11b65aa3..5e07328990b0 100644
--- a/compilerplugins/clang/loopvartoosmall.cxx
+++ b/compilerplugins/clang/loopvartoosmall.cxx
@@ -103,7 +103,7 @@ bool LoopVarTooSmall::VisitForStmt( const ForStmt* stmt )
     {
         report(
             DiagnosticsEngine::Warning,
-            "loop index variable is shorter than length type. " + qt.getAsString() + " < " + qt2.getAsString(),
+            "loop index type is narrower than length type. " + qt.getAsString() + " < " + qt2.getAsString(),
             stmt->getInit()->getLocStart())
           << stmt->getInit()->getSourceRange();
 //        stmt->getCond()->dump();
diff --git a/filter/source/flash/swfexporter.cxx b/filter/source/flash/swfexporter.cxx
index b9bd58dd39c0..8021b087c5a1 100644
--- a/filter/source/flash/swfexporter.cxx
+++ b/filter/source/flash/swfexporter.cxx
@@ -774,7 +774,7 @@ BitmapChecksum FlashExporter::ActionSummer(Reference< XShapes >& xShapes)
 
     Reference< XShape > xShape2;
 
-    for( sal_uInt16 nShape = 0; nShape < nShapeCount; nShape++ )
+    for( sal_uInt32 nShape = 0; nShape < nShapeCount; nShape++ )
     {
         xShapes->getByIndex( nShape ) >>= xShape2;
 
diff --git a/filter/source/graphicfilter/eps/eps.cxx b/filter/source/graphicfilter/eps/eps.cxx
index 99a09f97bcaf..7a7ff6d56476 100644
--- a/filter/source/graphicfilter/eps/eps.cxx
+++ b/filter/source/graphicfilter/eps/eps.cxx
@@ -2601,7 +2601,7 @@ void PSWriter::ImplWriteF( sal_Int32 nNumber, sal_uLong nCount, sal_uLong nMode
         }
     }
     mnCursorPos += nLen;
-    for( sal_uInt16 n = 0UL; n < nLen; n++  )
+    for( sal_uLong n = 0; n < nLen; n++  )
     {
         if ( n == nLen - nCount )
         {
diff --git a/filter/source/graphicfilter/icgm/actimpr.cxx b/filter/source/graphicfilter/icgm/actimpr.cxx
index 1e73c13c5538..206a54607b19 100644
--- a/filter/source/graphicfilter/icgm/actimpr.cxx
+++ b/filter/source/graphicfilter/icgm/actimpr.cxx
@@ -701,7 +701,7 @@ void CGMImpressOutAct::DrawPolyPolygon( tools::PolyPolygon& rPolyPolygon )
         drawing::PointSequence* pOuterSequence = aRetval.Coordinates.getArray();
         drawing::FlagSequence* pOuterFlags = aRetval.Flags.getArray();
 
-        for( sal_uInt16 a = 0; a < nNumPolys; a++ )
+        for( sal_uInt32 a = 0; a < nNumPolys; a++ )
         {
             Polygon aPolygon( rPolyPolygon.GetObject( a ) );
             sal_uInt32 nNumPoints = aPolygon.GetSize();
@@ -714,7 +714,7 @@ void CGMImpressOutAct::DrawPolyPolygon( tools::PolyPolygon& rPolyPolygon )
             awt::Point* pInnerSequence = pOuterSequence->getArray();
             drawing::PolygonFlags* pInnerFlags = pOuterFlags->getArray();
 
-            for( sal_uInt16 b = 0; b < nNumPoints; b++ )
+            for( sal_uInt32 b = 0; b < nNumPoints; b++ )
             {
                 *pInnerSequence++ = awt::Point( aPolygon.GetPoint( b ).X(), aPolygon.GetPoint( b ).Y() ) ;
                 *pInnerFlags++ = (drawing::PolygonFlags)aPolygon.GetFlags( b );
diff --git a/filter/source/graphicfilter/icgm/class4.cxx b/filter/source/graphicfilter/icgm/class4.cxx
index b5b860f63b22..e484e52d739f 100644
--- a/filter/source/graphicfilter/icgm/class4.cxx
+++ b/filter/source/graphicfilter/icgm/class4.cxx
@@ -127,7 +127,7 @@ void CGM::ImplDoClass4()
             {
                 sal_uInt32 nPoints = mnElementSize / ImplGetPointSize();
                 Polygon aPolygon( (sal_uInt16)nPoints );
-                for ( sal_uInt16 i = 0; i < nPoints; i++)
+                for ( sal_uInt32 i = 0; i < nPoints; i++)
                 {
                     FloatPoint  aFloatPoint;
                     ImplGetPoint( aFloatPoint, true );
diff --git a/filter/source/pdf/impdialog.cxx b/filter/source/pdf/impdialog.cxx
index 63cec8d78622..8902b4c8636b 100644
--- a/filter/source/pdf/impdialog.cxx
+++ b/filter/source/pdf/impdialog.cxx
@@ -1630,7 +1630,7 @@ ImplErrorDialog::~ImplErrorDialog()
 void ImplErrorDialog::dispose()
 {
     // free strings again
-    for( sal_uInt16 n = 0; n < m_pErrors->GetEntryCount(); n++ )
+    for( sal_Int32 n = 0; n < m_pErrors->GetEntryCount(); n++ )
         delete static_cast<OUString*>(m_pErrors->GetEntryData( n ));
     m_pErrors.clear();
     m_pExplanation.clear();
diff --git a/filter/source/svg/svgexport.cxx b/filter/source/svg/svgexport.cxx
index 53f681ba6ea8..927ebabf12b1 100644
--- a/filter/source/svg/svgexport.cxx
+++ b/filter/source/svg/svgexport.cxx
@@ -1176,7 +1176,7 @@ bool SVGFilter::implGenerateMetaData()
                 }
             }
             // text fields are used only for generating meta info so we don't need them anymore
-            for( sal_uInt32 i = 0; i < aFieldSet.size(); ++i )
+            for( size_t i = 0; i < aFieldSet.size(); ++i )
             {
                 if( aFieldSet[i] != NULL )
                 {
diff --git a/filter/source/svg/svgreader.cxx b/filter/source/svg/svgreader.cxx
index 446cf1aba7fc..088b623856aa 100644
--- a/filter/source/svg/svgreader.cxx
+++ b/filter/source/svg/svgreader.cxx
@@ -1650,7 +1650,7 @@ struct ShapeWritingVisitor
                       boost::bind(&basegfx::B2DPolyPolygon::transform,
                                   _1,boost::cref(aState.maCTM)));
 
-        for( sal_uInt32 i=0; i<aPolys.size(); ++i )
+        for( size_t i=0; i<aPolys.size(); ++i )
         {
             const basegfx::B2DRange aBounds(
                 aPolys[i].areControlPointsUsed() ?
diff --git a/filter/source/svg/svgwriter.cxx b/filter/source/svg/svgwriter.cxx
index 7220734d110c..b83d9fe548dd 100644
--- a/filter/source/svg/svgwriter.cxx
+++ b/filter/source/svg/svgwriter.cxx
@@ -2041,7 +2041,7 @@ void SVGActionWriter::ImplWriteShape( const SVGShapeDescriptor& rShape, bool bAp
         const OUString   aComma( "," );
         OUString         aDashArrayStr;
 
-        for( unsigned int k = 0; k < rShape.maDashArray.size(); ++k )
+        for( size_t k = 0; k < rShape.maDashArray.size(); ++k )
         {
             const sal_Int32 nDash = ( bApplyMapping ?
                                         ImplMap( FRound( rShape.maDashArray[ k ] ) ) :
diff --git a/forms/source/component/ListBox.cxx b/forms/source/component/ListBox.cxx
index c6243902ccc5..890e901a587a 100644
--- a/forms/source/component/ListBox.cxx
+++ b/forms/source/component/ListBox.cxx
@@ -610,7 +610,7 @@ namespace frm
                 pStr++;
             }
             aListSourceSeq.realloc( nTokens );
-            for (sal_uInt16 i=0; i<nTokens; ++i)
+            for (sal_Int32 i=0; i<nTokens; ++i)
             {
                 sal_Int32 nTmp = 0;
                 aListSourceSeq.getArray()[i] = sListSource.getToken(i,';',nTmp);
@@ -941,9 +941,9 @@ namespace frm
                         if(*aBoundColumn == -1)
                         {
                             // the type of i matters! It will be the type of the ORowSetValue pushed to aValueList!
-                            for(sal_Int16 i=0; static_cast<ValueList::size_type>(i) < aDisplayList.size(); ++i)
+                            for(size_t i=0; i < aDisplayList.size(); ++i)
                             {
-                                aValueList.push_back(i);
+                                aValueList.push_back(static_cast<connectivity::ORowSetValue >(i));
                             }
                         }
                         else
diff --git a/formula/source/ui/dlg/formula.cxx b/formula/source/ui/dlg/formula.cxx
index 425593d1507b..7fcb6bd207b1 100644
--- a/formula/source/ui/dlg/formula.cxx
+++ b/formula/source/ui/dlg/formula.cxx
@@ -883,7 +883,7 @@ void FormulaDlg_Impl::FillControls(bool &rbNext, bool &rbPrev)
         sal_Int32   nEditPos = pMEdit->GetSelection().Min();
         bool    bFlag    = false;
 
-        for(sal_uInt16 i=0;i<nArgs;i++)
+        for(sal_Int32 i=0;i<nArgs;i++)
         {
             sal_Int32 nLength = m_aArguments[i].getLength()+1;
             pParaWin->SetArgument(i,m_aArguments[i]);
@@ -1180,7 +1180,7 @@ void FormulaDlg_Impl::SaveArg( sal_uInt16 nEd )
             m_aArguments[nEd] = pParaWin->GetArgument(nEd);
 
         sal_uInt16 nClearPos=nEd+1;
-        for(sal_uInt16 i=nEd+1; i<nArgs; i++)
+        for(sal_Int32 i=nEd+1; i<nArgs; i++)
         {
             if( !pParaWin->GetArgument(i).isEmpty() )
             {
@@ -1188,7 +1188,7 @@ void FormulaDlg_Impl::SaveArg( sal_uInt16 nEd )
             }
         }
 
-        for(sal_uInt16 i=nClearPos; i<nArgs; i++)
+        for(sal_Int32 i=nClearPos; i<nArgs; i++)
         {
             m_aArguments[i].clear();
         }
diff --git a/framework/source/uielement/fontmenucontroller.cxx b/framework/source/uielement/fontmenucontroller.cxx
index 91b262f8bc34..e47321f803a2 100644
--- a/framework/source/uielement/fontmenucontroller.cxx
+++ b/framework/source/uielement/fontmenucontroller.cxx
@@ -88,7 +88,7 @@ void FontMenuController::fillPopupMenu( const Sequence< OUString >& rFontNameSeq
     {
         vector<OUString> aVector;
         aVector.reserve(rFontNameSeq.getLength());
-        for ( sal_uInt16 i = 0; i < rFontNameSeq.getLength(); i++ )
+        for ( sal_Int32 i = 0; i < rFontNameSeq.getLength(); i++ )
         {
             aVector.push_back(MnemonicGenerator::EraseAllMnemonicChars(pFontNameArray[i]));
         }
diff --git a/i18npool/source/localedata/LocaleNode.cxx b/i18npool/source/localedata/LocaleNode.cxx
index 89325aeaf0ba..29adfabcea6d 100644
--- a/i18npool/source/localedata/LocaleNode.cxx
+++ b/i18npool/source/localedata/LocaleNode.cxx
@@ -661,7 +661,7 @@ void LCFormatNode::generateCode (const OFileWriter &of) const
     bool bCtypeIsRef = false;
     bool bHaveEngineering = false;
 
-    for (sal_Int16 i = 0; i< getNumberOfChildren() ; i++, formatCount++)
+    for (sal_Int32 i = 0; i< getNumberOfChildren() ; i++, formatCount++)
     {
         LocaleNode * currNode = getChildAt (i);
         if ( currNode->getName() == "DateAcceptancePattern" )
diff --git a/i18npool/source/localedata/localedata.cxx b/i18npool/source/localedata/localedata.cxx
index 320d171aec62..c7d4e2e6374e 100644
--- a/i18npool/source/localedata/localedata.cxx
+++ b/i18npool/source/localedata/localedata.cxx
@@ -671,7 +671,7 @@ Sequence< CalendarItem2 > LocaleDataImpl::getCalendarItems(
             case REF_MONTHS:
             case REF_GMONTHS:
             case REF_PMONTHS:
-                for (sal_Int16 j = 0; j < nSize; ++j, ++pItem)
+                for (sal_Int32 j = 0; j < nSize; ++j, ++pItem)
                 {
                     CalendarItem2 item( allCalendars[rnOffset], allCalendars[rnOffset+1],
                             allCalendars[rnOffset+2], allCalendars[rnOffset+3]);
@@ -681,7 +681,7 @@ Sequence< CalendarItem2 > LocaleDataImpl::getCalendarItems(
                 break;
             case REF_ERAS:
                 // Absent narrow name.
-                for (sal_Int16 j = 0; j < nSize; ++j, ++pItem)
+                for (sal_Int32 j = 0; j < nSize; ++j, ++pItem)
                 {
                     CalendarItem2 item( allCalendars[rnOffset], allCalendars[rnOffset+1],
                             allCalendars[rnOffset+2], OUString());
diff --git a/i18npool/source/transliteration/transliterationImpl.cxx b/i18npool/source/transliteration/transliterationImpl.cxx
index 4ff2f3e6dc1e..782c633f2626 100644
--- a/i18npool/source/transliteration/transliterationImpl.cxx
+++ b/i18npool/source/transliteration/transliterationImpl.cxx
@@ -222,7 +222,7 @@ TransliterationImpl::loadModuleNew( const Sequence < TransliterationModulesNew >
     sal_Int32 mask = 0, count = modType.getLength();
     if (count > maxCascade)
         throw ERROR; // could not handle more than maxCascade
-    for (sal_Int16 i = 0; i < count; i++) {
+    for (sal_Int32 i = 0; i < count; i++) {
         for (sal_Int16 j = 0; TMlist[j].tmn; j++) {
             if (TMlist[j].tmn == modType[i]) {
                 if (mask == 0)
diff --git a/jvmfwk/plugins/sunmajor/javaenvsetup/javaldx.cxx b/jvmfwk/plugins/sunmajor/javaenvsetup/javaldx.cxx
index 7c249d31844a..f1ef2cd38aff 100644
--- a/jvmfwk/plugins/sunmajor/javaenvsetup/javaldx.cxx
+++ b/jvmfwk/plugins/sunmajor/javaenvsetup/javaldx.cxx
@@ -125,7 +125,7 @@ OString getLD_LIBRARY_PATH(const rtl::ByteSequence & vendorData)
 static bool hasOption(char const * szOption, int argc, char** argv)
 {
     bool retVal= false;
-    for(sal_Int16 i= 1; i < argc; i++)
+    for(int i= 1; i < argc; i++)
     {
         if( ! strcmp(argv[i], szOption))
         {
diff --git a/lotuswordpro/source/filter/lwpdrawobj.cxx b/lotuswordpro/source/filter/lwpdrawobj.cxx
index 5fc78170fd5b..9a605b65b0f4 100644
--- a/lotuswordpro/source/filter/lwpdrawobj.cxx
+++ b/lotuswordpro/source/filter/lwpdrawobj.cxx
@@ -537,7 +537,7 @@ XFFrame* LwpDrawPolyLine::CreateDrawObj(const OUString& rStyleName )
     XFDrawPath* pPolyline = new XFDrawPath();
     pPolyline->MoveTo(XFPoint((double)m_pVector[0].x/TWIPS_PER_CM * m_pTransData->fScaleX,
             (double)m_pVector[0].y/TWIPS_PER_CM * m_pTransData->fScaleY));
-    for (sal_uInt8 nC = 1; nC < m_aPolyLineRec.nNumPoints; nC++)
+    for (sal_uInt16 nC = 1; nC < m_aPolyLineRec.nNumPoints; nC++)
     {
         pPolyline->LineTo(XFPoint((double)m_pVector[nC].x/TWIPS_PER_CM * m_pTransData->fScaleX,
             (double)m_pVector[nC].y/TWIPS_PER_CM * m_pTransData->fScaleY));
@@ -552,7 +552,7 @@ XFFrame* LwpDrawPolyLine::CreateDrawObj(const OUString& rStyleName )
 XFFrame* LwpDrawPolyLine::CreateStandardDrawObj(const  OUString& rStyleName)
 {
     XFDrawPolyline* pPolyline = new XFDrawPolyline();
-    for (sal_uInt8 nC = 0; nC < m_aPolyLineRec.nNumPoints; nC++)
+    for (sal_uInt16 nC = 0; nC < m_aPolyLineRec.nNumPoints; nC++)
     {
         pPolyline->AddPoint((double)m_pVector[nC].x/TWIPS_PER_CM,
             (double)m_pVector[nC].y/TWIPS_PER_CM);
@@ -1169,7 +1169,7 @@ void LwpDrawTextArt::CreateFWPath(XFDrawPath* pPath)
     pPath->MoveTo(aStart);
 
     sal_uInt8 nPtIndex = 1;
-    for (sal_uInt8 nC = 1; nC <= m_aTextArtRec.aPath[0].n; nC++)
+    for (sal_uInt16 nC = 1; nC <= m_aTextArtRec.aPath[0].n; nC++)
     {
         nX = (m_aTextArtRec.aPath[0].pPts[nPtIndex].x + m_aTextArtRec.aPath[1].pPts[nPtIndex].x) / 2;
         nY = (m_aTextArtRec.aPath[0].pPts[nPtIndex].y + m_aTextArtRec.aPath[1].pPts[nPtIndex].y) / 2;
diff --git a/lotuswordpro/source/filter/lwpidxmgr.cxx b/lotuswordpro/source/filter/lwpidxmgr.cxx
index 43208ab7a92c..21be76bd3a18 100644
--- a/lotuswordpro/source/filter/lwpidxmgr.cxx
+++ b/lotuswordpro/source/filter/lwpidxmgr.cxx
@@ -291,14 +291,14 @@ void LwpIndexManager::ReadLeafData( LwpObjectStream *pObjStrm )
         akey->id.Read(pObjStrm);
         m_ObjectKeys.push_back(akey);
 
-        for (sal_uInt8 k = 1; k < KeyCount; k++)
+        for (sal_uInt16 k = 1; k < KeyCount; k++)
         {
             akey = new LwpKey();
             akey->id.ReadCompressed(pObjStrm, m_ObjectKeys.at(m_nKeyCount+k-1)->id);
             m_ObjectKeys.push_back(akey);
         }
 
-        for (sal_uInt8 j = 0; j < KeyCount; j++)
+        for (sal_uInt16 j = 0; j < KeyCount; j++)
             m_ObjectKeys.at(m_nKeyCount+j)->offset = pObjStrm->QuickReaduInt32();
     }
     m_nKeyCount += KeyCount;
diff --git a/lotuswordpro/source/filter/lwprowlayout.cxx b/lotuswordpro/source/filter/lwprowlayout.cxx
index 1f8265e370c5..0ad389a7e36e 100644
--- a/lotuswordpro/source/filter/lwprowlayout.cxx
+++ b/lotuswordpro/source/filter/lwprowlayout.cxx
@@ -309,7 +309,7 @@ sal_Int32 LwpRowLayout::FindMarkConnCell(sal_uInt8 nStartCol,sal_uInt8 nEndCol)
     sal_uInt16 nSpannRows = 1;
     sal_Int32 nMarkConnCell = -1;
 
-    for (sal_uInt16 i=0;i<m_ConnCellList.size();i++)
+    for (size_t i=0;i<m_ConnCellList.size();i++)
     {
         if (m_ConnCellList[i]->GetColID()>=nEndCol)
             break;
@@ -334,7 +334,7 @@ sal_Int32 LwpRowLayout::FindNextMarkConnCell(sal_uInt16 nMarkConnCell,sal_uInt8
 {
     sal_uInt16 nMaxRows = m_ConnCellList[nMarkConnCell]->GetNumrows();
 
-    for (sal_uInt16 i=nMarkConnCell+1;i<m_ConnCellList.size();i++)
+    for (size_t i=nMarkConnCell+1;i<m_ConnCellList.size();i++)
     {
         if (m_ConnCellList[i]->GetColID()>=nEndCol)
             break;
@@ -449,7 +449,7 @@ void LwpRowLayout::CollectMergeInfo()
 void LwpRowLayout::SetCellSplit(sal_uInt16 nEffectRows)
 {
     LwpConnectedCellLayout* pConnCell;
-    for (sal_uInt32 i=0; i<m_ConnCellList.size(); i++)
+    for (size_t i=0; i<m_ConnCellList.size(); i++)
     {
         pConnCell = m_ConnCellList[i];
         sal_uInt16 nRowSpan = pConnCell->GetRowID()+pConnCell->GetNumrows();
diff --git a/lotuswordpro/source/filter/xfilter/xfinputlist.hxx b/lotuswordpro/source/filter/xfilter/xfinputlist.hxx
index a4e36800df04..8f19d4ada9a5 100644
--- a/lotuswordpro/source/filter/xfilter/xfinputlist.hxx
+++ b/lotuswordpro/source/filter/xfilter/xfinputlist.hxx
@@ -97,7 +97,7 @@ inline void XFInputList::ToXml(IXFStream *pStrm)
     pAttrList->AddAttribute( "text:value", " " );
     pStrm->StartElement( "text:label" );
     pStrm->EndElement( "text:label" );
-    for(sal_uInt32 i=0; i< m_list.size();i++)
+    for(size_t i=0; i< m_list.size();i++)
     {
         pAttrList->Clear();
         pAttrList->AddAttribute( "text:value", m_list[i] );
diff --git a/package/source/manifest/ManifestImport.cxx b/package/source/manifest/ManifestImport.cxx
index 9376ab060721..7c0bbde0501c 100644
--- a/package/source/manifest/ManifestImport.cxx
+++ b/package/source/manifest/ManifestImport.cxx
@@ -417,7 +417,7 @@ OUString ManifestImport::PushNameAndNamespaces( const OUString& aName, const uno
 
     aStack.push_back( ManifestScopeEntry( aConvertedName, aNamespaces ) );
 
-    for ( sal_uInt16 nInd = 0; nInd < aAttribsStrs.size(); nInd++ )
+    for ( size_t nInd = 0; nInd < aAttribsStrs.size(); nInd++ )
     {
         // convert the attribute names on filling
         o_aConvertedAttribs[ConvertName( aAttribsStrs[nInd].first )] = aAttribsStrs[nInd].second;
diff --git a/package/source/zippackage/ZipPackageStream.cxx b/package/source/zippackage/ZipPackageStream.cxx
index 2ed7f571f61a..002dfcdc3002 100644
--- a/package/source/zippackage/ZipPackageStream.cxx
+++ b/package/source/zippackage/ZipPackageStream.cxx
@@ -1239,7 +1239,7 @@ void SAL_CALL ZipPackageStream::setPropertyValue( const OUString& aPropertyName,
                 Sequence < sal_Int8 > aSequence ( nPathLength );
                 sal_Int8 *pArray = aSequence.getArray();
                 const sal_Unicode *pChar = sTempString.getStr();
-                for ( sal_Int16 i = 0; i < nPathLength; i++ )
+                for ( sal_Int32 i = 0; i < nPathLength; i++ )
                     pArray[i] = static_cast < const sal_Int8 > ( pChar[i] );
                 aNewKey = aSequence;
             }
diff --git a/svgio/source/svgreader/svgcharacternode.cxx b/svgio/source/svgreader/svgcharacternode.cxx
index 7b8e4cb252e8..5fe2ef3d7b9c 100644
--- a/svgio/source/svgreader/svgcharacternode.cxx
+++ b/svgio/source/svgreader/svgcharacternode.cxx
@@ -306,7 +306,7 @@ namespace svgio
                         ::std::vector< double > aExtendArray(aTextLayouterDevice.getTextArray(getText(), nArray, nLength - nArray));
                         aTextArray.reserve(nLength);
 
-                        for(sal_uInt32 a(0); a < aExtendArray.size(); a++)
+                        for(size_t a(0); a < aExtendArray.size(); a++)
                         {
                             aTextArray.push_back(aExtendArray[a] + fStartX);
                         }
@@ -331,7 +331,7 @@ namespace svgio
                             aTextArray = aTextLayouterDevice.getTextArray(getText(), nIndex, nLength);
                         }
 
-                        for(sal_uInt32 a(0); a < aTextArray.size(); a++)
+                        for(size_t a(0); a < aTextArray.size(); a++)
                         {
                             aTextArray[a] *= fFactor;
                         }
@@ -615,7 +615,7 @@ namespace svgio
             {
                 const double fFactor(F_PI / 180.0);
 
-                for(sal_uInt32 a(0); a < maRotate.size(); a++)
+                for(size_t a(0); a < maRotate.size(); a++)
                 {
                     maRotate[a] *= fFactor;
                 }
diff --git a/svgio/source/svgreader/svgnode.cxx b/svgio/source/svgreader/svgnode.cxx
index 7c6b336a4d41..88cc39db92db 100644
--- a/svgio/source/svgreader/svgnode.cxx
+++ b/svgio/source/svgreader/svgnode.cxx
@@ -108,7 +108,7 @@ namespace svgio
                             }
                         }
 
-                        for(sal_uInt32 a(0); a < aParts.size(); a++)
+                        for(size_t a(0); a < aParts.size(); a++)
                         {
                             const OUString aNewConcatenated(
                                 "." + aParts[a] + aConcatenated);
@@ -238,7 +238,7 @@ namespace svgio
                 // as current
                 SvgStyleAttributes* pCurrent = const_cast< SvgStyleAttributes* >(maCssStyleVector[0]);
 
-                for(sal_uInt32 a(1); a < maCssStyleVector.size(); a++)
+                for(size_t a(1); a < maCssStyleVector.size(); a++)
                 {
                     SvgStyleAttributes* pNext = const_cast< SvgStyleAttributes* >(maCssStyleVector[a]);
 
diff --git a/svgio/source/svgreader/svgstylenode.cxx b/svgio/source/svgreader/svgstylenode.cxx
index ab9fa025ec25..42cc554a654e 100644
--- a/svgio/source/svgreader/svgstylenode.cxx
+++ b/svgio/source/svgreader/svgstylenode.cxx
@@ -117,7 +117,7 @@ namespace svgio
                     OUString aConcatenatedSelector;
 
                     // re-combine without spaces, create a unique name (for now)
-                    for(sal_uInt32 a(0); a < aSelectorParts.size(); a++)
+                    for(size_t a(0); a < aSelectorParts.size(); a++)
                     {
                         aConcatenatedSelector += aSelectorParts[a];
                     }
diff --git a/ucb/source/ucp/file/shell.cxx b/ucb/source/ucp/file/shell.cxx
index 658f861de701..d128b859dcb9 100644
--- a/ucb/source/ucp/file/shell.cxx
+++ b/ucb/source/ucp/file/shell.cxx
@@ -2821,7 +2821,7 @@ void SAL_CALL
 shell::notifyContentExchanged( std::vector< std::list< ContentEventNotifier* >* >* listeners_vec )
 {
     std::list< ContentEventNotifier* >* listeners;
-    for( sal_uInt32 i = 0; i < listeners_vec->size(); ++i )
+    for( size_t i = 0; i < listeners_vec->size(); ++i )
     {
         listeners = (*listeners_vec)[i];
         std::list< ContentEventNotifier* >::iterator it = listeners->begin();