[Libreoffice-commits] .: l10ntools/source oox/source package/source svx/source sw/source

Julien Nabet serval2412 at kemper.freedesktop.org
Tue Aug 30 14:13:24 PDT 2011


 l10ntools/source/help/HelpLinker.cxx     |    2 +-
 oox/source/drawingml/fillproperties.cxx  |    2 +-
 package/source/zippackage/ZipPackage.cxx |    2 +-
 svx/source/svdraw/svdpage.cxx            |    2 +-
 sw/source/filter/ww8/ww8scan.cxx         |    2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 01a87c102fe6bdaf3aa48a0ba17219e6e0983c4c
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Tue Aug 30 23:11:44 2011 +0200

    Some cppcheck cleaning

diff --git a/l10ntools/source/help/HelpLinker.cxx b/l10ntools/source/help/HelpLinker.cxx
index b1d7c1c..4889642 100644
--- a/l10ntools/source/help/HelpLinker.cxx
+++ b/l10ntools/source/help/HelpLinker.cxx
@@ -735,7 +735,7 @@ void HelpLinker::main( std::vector<std::string> &args,
     bExtensionMode = false;
     helpFiles.clear();
 
-    if (args.size() > 0 && args[0][0] == '@')
+    if ((!args.empty()) && args[0][0] == '@')
     {
         std::vector<std::string> stringList;
         std::string strBuf;
diff --git a/oox/source/drawingml/fillproperties.cxx b/oox/source/drawingml/fillproperties.cxx
index 88e2949..bf93612 100644
--- a/oox/source/drawingml/fillproperties.cxx
+++ b/oox/source/drawingml/fillproperties.cxx
@@ -270,7 +270,7 @@ void FillProperties::pushToPropMap( ShapePropertyMap& rPropMap,
                             GradientFillProperties::GradientStopMap::const_iterator aIt = maGradientProps.maGradientStops.begin();
                             // Try to find the axial median
                             for(size_t i=0;i<nColorCount;i+=3)
-                                aIt++;
+                                ++aIt;
                             // API StartColor is inner color in axial gradient
                             // aIt->second.hasColor() kind would be better than Color != API_RGB_WHITE
                             if( aGradient.StartColor == aGradient.EndColor &&
diff --git a/package/source/zippackage/ZipPackage.cxx b/package/source/zippackage/ZipPackage.cxx
index ed5f81b..72b3ab3 100644
--- a/package/source/zippackage/ZipPackage.cxx
+++ b/package/source/zippackage/ZipPackage.cxx
@@ -1058,7 +1058,7 @@ void ZipPackage::WriteManifest( ZipOutputStream& aZipOut, const vector< uno::Seq
         sal_Int32 nInd = 0;
         for ( vector < uno::Sequence < PropertyValue > >::const_iterator aIter = aManList.begin(), aEnd = aManList.end();
              aIter != aEnd;
-             aIter++, nInd++ )
+             ++aIter, ++nInd )
         {
             aManifestSequence[nInd] = ( *aIter );
         }
diff --git a/svx/source/svdraw/svdpage.cxx b/svx/source/svdraw/svdpage.cxx
index 70deda8..dbd8a39 100644
--- a/svx/source/svdraw/svdpage.cxx
+++ b/svx/source/svdraw/svdpage.cxx
@@ -1351,7 +1351,7 @@ SdrPage::~SdrPage()
     // of page users.  Therefore we have to use a copy of the list for the
     // iteration.
     ::sdr::PageUserVector aListCopy (maPageUsers.begin(), maPageUsers.end());
-    for(::sdr::PageUserVector::iterator aIterator = aListCopy.begin(); aIterator != aListCopy.end(); aIterator++)
+    for(::sdr::PageUserVector::iterator aIterator = aListCopy.begin(); aIterator != aListCopy.end(); ++aIterator)
     {
         sdr::PageUser* pPageUser = *aIterator;
         DBG_ASSERT(pPageUser, "SdrPage::~SdrPage: corrupt PageUser list (!)");
diff --git a/sw/source/filter/ww8/ww8scan.cxx b/sw/source/filter/ww8/ww8scan.cxx
index ac92dd2..964929b 100644
--- a/sw/source/filter/ww8/ww8scan.cxx
+++ b/sw/source/filter/ww8/ww8scan.cxx
@@ -6617,7 +6617,7 @@ bool WW8PLCF_HdFt::GetTextPos(sal_uInt8 grpfIhdt, sal_uInt8 nWhich, WW8_CP& rSta
     aPLCF.SetIdx( nIdx );               // Lookup suitable CP
     aPLCF.Get( rStart, nEnd, pData );
     rLen = nEnd - rStart;
-    aPLCF++;
+    ++aPLCF;
 
     return true;
 }


More information about the Libreoffice-commits mailing list