[Libreoffice-commits] .: 2 commits - binfilter/bf_sc binfilter/bf_sd binfilter/bf_xmloff lotuswordpro/source

Caolán McNamara caolan at kemper.freedesktop.org
Fri Jan 7 03:08:23 PST 2011


 binfilter/bf_sc/source/filter/xml/sc_XMLChangeTrackingImportHelper.cxx |    6 +++---
 binfilter/bf_sc/source/ui/unoobj/sc_cellsuno.cxx                       |    2 +-
 binfilter/bf_sd/source/ui/view/sd_frmview.cxx                          |    2 +-
 binfilter/bf_xmloff/source/draw/xmloff_animexp.cxx                     |    2 +-
 binfilter/bf_xmloff/source/style/xmloff_prhdlfac.cxx                   |    2 +-
 binfilter/bf_xmloff/source/style/xmloff_xmlprmap.cxx                   |    4 ++--
 lotuswordpro/source/filter/lwpbulletstylemgr.cxx                       |    2 +-
 lotuswordpro/source/filter/xfilter/xffilestream.cxx                    |    2 +-
 lotuswordpro/source/filter/xfilter/xftextspan.cxx                      |    8 ++++----
 9 files changed, 15 insertions(+), 15 deletions(-)

New commits:
commit 3a75c8a25a2be9a986ae13ced05d9345b6fd5b9c
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Jan 7 11:08:16 2011 +0000

    cppcheck: prefer prefix variant

diff --git a/binfilter/bf_sc/source/filter/xml/sc_XMLChangeTrackingImportHelper.cxx b/binfilter/bf_sc/source/filter/xml/sc_XMLChangeTrackingImportHelper.cxx
index 7a320c2..8684ddf 100644
--- a/binfilter/bf_sc/source/filter/xml/sc_XMLChangeTrackingImportHelper.cxx
+++ b/binfilter/bf_sc/source/filter/xml/sc_XMLChangeTrackingImportHelper.cxx
@@ -528,7 +528,7 @@ void ScXMLChangeTrackingImportHelper::CreateGeneratedActions(ScMyGeneratedList&
                     DBG_ASSERT((*aItr)->nID, "could not insert generated action");
                 }
             }
-            aItr++;
+            ++aItr;
         }
     }
 }
@@ -799,7 +799,7 @@ void ScXMLChangeTrackingImportHelper::CreateChangeTrack(ScDocument* pTempDoc)
             else
                 DBG_ERROR("no action");
 
-            aItr++;
+            ++aItr;
         }
         if (pTrack->GetLast())
             pTrack->SetActionMax(pTrack->GetLast()->GetActionNumber());
@@ -810,7 +810,7 @@ void ScXMLChangeTrackingImportHelper::CreateChangeTrack(ScDocument* pTempDoc)
             SetDependences(*aItr);
 
             if ((*aItr)->nActionType == SC_CAT_CONTENT)
-                aItr++;
+                ++aItr;
             else
             {
                 if (*aItr)
diff --git a/binfilter/bf_sc/source/ui/unoobj/sc_cellsuno.cxx b/binfilter/bf_sc/source/ui/unoobj/sc_cellsuno.cxx
index e605ed8..b37d977 100644
--- a/binfilter/bf_sc/source/ui/unoobj/sc_cellsuno.cxx
+++ b/binfilter/bf_sc/source/ui/unoobj/sc_cellsuno.cxx
@@ -8269,7 +8269,7 @@ void ScUniqueCellFormatsObj::GetObjects_Impl()
                     aItr = aList.erase(aItr);
                 }
                 else
-                    aItr++;
+                    ++aItr;
                 if (aItr == aList.end() && !aList.empty())
                 {
                     aItr = aList.begin();
diff --git a/binfilter/bf_xmloff/source/draw/xmloff_animexp.cxx b/binfilter/bf_xmloff/source/draw/xmloff_animexp.cxx
index 0bc0880..6af3611 100644
--- a/binfilter/bf_xmloff/source/draw/xmloff_animexp.cxx
+++ b/binfilter/bf_xmloff/source/draw/xmloff_animexp.cxx
@@ -540,7 +540,7 @@ void XMLAnimationsExporter::exportAnimations( SvXMLExport& rExport )
                 }
             }
 
-            aIter++;
+            ++aIter;
         }
         while( aIter != aEnd );
     }
diff --git a/binfilter/bf_xmloff/source/style/xmloff_prhdlfac.cxx b/binfilter/bf_xmloff/source/style/xmloff_prhdlfac.cxx
index 59210b9..5dd74b8 100644
--- a/binfilter/bf_xmloff/source/style/xmloff_prhdlfac.cxx
+++ b/binfilter/bf_xmloff/source/style/xmloff_prhdlfac.cxx
@@ -116,7 +116,7 @@ SvXMLEnumMapEntry const aXML_WritingDirection_Enum[] =
 //
 XMLPropertyHandlerFactory::~XMLPropertyHandlerFactory()
 {
-    for( CacheMap::iterator pPos = maHandlerCache.begin(); pPos != maHandlerCache.end(); pPos++ )
+    for( CacheMap::iterator pPos = maHandlerCache.begin(); pPos != maHandlerCache.end(); ++pPos )
         delete pPos->second;
 }
 
diff --git a/binfilter/bf_xmloff/source/style/xmloff_xmlprmap.cxx b/binfilter/bf_xmloff/source/style/xmloff_xmlprmap.cxx
index 7f2f12b..5e02173 100644
--- a/binfilter/bf_xmloff/source/style/xmloff_xmlprmap.cxx
+++ b/binfilter/bf_xmloff/source/style/xmloff_xmlprmap.cxx
@@ -103,7 +103,7 @@ void XMLPropertySetMapper::AddMapperEntry(
     for( vector < UniReference < XMLPropertyHandlerFactory > >::iterator
             aFIter = rMapper->aHdlFactories.begin();
          aFIter != rMapper->aHdlFactories.end();
-         aFIter++ )
+         ++aFIter )
     {
         aHdlFactories.push_back( *aFIter );
     }
@@ -111,7 +111,7 @@ void XMLPropertySetMapper::AddMapperEntry(
     for( vector < XMLPropertySetMapperEntry_Impl >::iterator
             aEIter = rMapper->aMapEntries.begin();
          aEIter != rMapper->aMapEntries.end();
-         aEIter++ )
+         ++aEIter )
     {
         aMapEntries.push_back( *aEIter );
     }
diff --git a/lotuswordpro/source/filter/lwpbulletstylemgr.cxx b/lotuswordpro/source/filter/lwpbulletstylemgr.cxx
index 34f0cd2..08304de 100644
--- a/lotuswordpro/source/filter/lwpbulletstylemgr.cxx
+++ b/lotuswordpro/source/filter/lwpbulletstylemgr.cxx
@@ -134,7 +134,7 @@ rtl::OUString LwpBulletStyleMgr::RegisterBulletStyle(LwpPara* pPara, LwpBulletOv
 
     sal_uInt16 nNameIndex = 0;
     std::vector <OverridePair>::iterator iter;
-    for(iter = m_vIDsPairList.begin(); iter != m_vIDsPairList.end(); iter++)
+    for(iter = m_vIDsPairList.begin(); iter != m_vIDsPairList.end(); ++iter)
     {
         if (iter->first->GetSilverBullet() == aBulletID && iter->second == aIndentID
             && iter->first->IsRightAligned() == pBullOver->IsRightAligned())
diff --git a/lotuswordpro/source/filter/xfilter/xffilestream.cxx b/lotuswordpro/source/filter/xfilter/xffilestream.cxx
index 7621699..fc2a4ec 100644
--- a/lotuswordpro/source/filter/xfilter/xffilestream.cxx
+++ b/lotuswordpro/source/filter/xfilter/xffilestream.cxx
@@ -101,7 +101,7 @@ void		XFFileStream::StartElement(const rtl::OUString& oustr)
 
     //write attributes:
     std::vector<std::pair<rtl::OUString,rtl::OUString> >::iterator	it;
-    for( it = m_pAttrList->m_aAttrList.begin(); it != m_pAttrList->m_aAttrList.end(); it++ )
+    for( it = m_pAttrList->m_aAttrList.begin(); it != m_pAttrList->m_aAttrList.end(); ++it )
     {
         std::pair<rtl::OUString,rtl::OUString>	pair = *it;
         rtl::OUString	name = pair.first;
diff --git a/lotuswordpro/source/filter/xfilter/xftextspan.cxx b/lotuswordpro/source/filter/xfilter/xftextspan.cxx
index 97ca48d..ae23248 100644
--- a/lotuswordpro/source/filter/xfilter/xftextspan.cxx
+++ b/lotuswordpro/source/filter/xfilter/xftextspan.cxx
@@ -82,7 +82,7 @@ XFTextSpan::XFTextSpan(rtl::OUString& text,
 XFTextSpan::~XFTextSpan()
 {
     std::vector<IXFContent*>::iterator it;
-    for( it = m_aContents.begin(); it != m_aContents.end(); it++ )
+    for( it = m_aContents.begin(); it != m_aContents.end(); ++it )
     {
         IXFContent	*pContent = *it;
         if( pContent )
@@ -149,7 +149,7 @@ void	XFTextSpan::ToXml(IXFStream *pStrm)
     pStrm->StartElement( A2OUSTR("text:span") );
 
     std::vector<IXFContent*>::iterator it;
-    for( it= m_aContents.begin(); it!= m_aContents.end(); it++ )
+    for( it= m_aContents.begin(); it!= m_aContents.end(); ++it )
     {
         IXFContent *pContent = *it;
         if( pContent )
@@ -172,7 +172,7 @@ void	XFTextSpanStart::ToXml(IXFStream *pStrm)
     pStrm->StartElement( A2OUSTR("text:span") );
 
     std::vector<IXFContent*>::iterator it;
-    for( it= m_aContents.begin(); it!= m_aContents.end(); it++ )
+    for( it= m_aContents.begin(); it!= m_aContents.end(); ++it )
     {
         IXFContent *pContent = *it;
         if( pContent )
@@ -182,7 +182,7 @@ void	XFTextSpanStart::ToXml(IXFStream *pStrm)
 void	XFTextSpanEnd::ToXml(IXFStream *pStrm)
 {
     std::vector<IXFContent*>::iterator it;
-    for( it= m_aContents.begin(); it!= m_aContents.end(); it++ )
+    for( it= m_aContents.begin(); it!= m_aContents.end(); ++it )
     {
         IXFContent *pContent = *it;
         if( pContent )
commit 55eef1af4387302aea71c3f36317d10463f89606
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Jan 7 10:56:19 2011 +0000

    cppcheck: prefer prefix variant

diff --git a/binfilter/bf_sd/source/ui/view/sd_frmview.cxx b/binfilter/bf_sd/source/ui/view/sd_frmview.cxx
index fd7fc6b..c1a4d69 100644
--- a/binfilter/bf_sd/source/ui/view/sd_frmview.cxx
+++ b/binfilter/bf_sd/source/ui/view/sd_frmview.cxx
@@ -461,7 +461,7 @@ void FrameView::WriteUserDataSequence ( ::com::sun::star::uno::Sequence < ::com:
     PropertyValue* pValue = &(rValues.getArray()[nOldLength]);
 
     std::vector< std::pair< OUString, Any > >::iterator aIter( aUserData.begin() );
-    for( ; aIter != aUserData.end(); aIter++, ++pValue )
+    for( ; aIter != aUserData.end(); ++aIter, ++pValue )
     {
         pValue->Name = (*aIter).first;
         pValue->Value = (*aIter).second;


More information about the Libreoffice-commits mailing list