summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Stahl <mstahl@redhat.com>2015-03-13 14:16:51 +0100
committerMichael Stahl <mstahl@redhat.com>2015-03-13 16:23:26 +0100
commit077723111292ea615437f3bc2f1e47cf77d7ad42 (patch)
treef851f81cea82fcbf8251f8418b3f2a26f3085f64
parentCppCheck cleaning : avoid static string comparison (diff)
downloadcore-077723111292ea615437f3bc2f1e47cf77d7ad42.tar.gz
core-077723111292ea615437f3bc2f1e47cf77d7ad42.zip
V803 decreased performance postfix increment
These are pretty silly anyway, but apparently it complains even about integer variables which make this rather a waste of time. Change-Id: I15e847d33d5decd2adcab04e4f1567d3997d28a2
-rw-r--r--chart2/source/view/axes/ScaleAutomatism.cxx2
-rw-r--r--chart2/source/view/main/ChartView.cxx3
-rw-r--r--dbaccess/source/core/api/RowSetCache.cxx2
-rw-r--r--include/basegfx/range/b2dconnectedranges.hxx2
-rw-r--r--include/cppuhelper/interfacecontainer.hxx2
-rw-r--r--package/source/zippackage/ZipPackageFolderEnumeration.cxx2
-rw-r--r--sc/source/core/data/dociter.cxx4
-rw-r--r--sd/source/core/shapelist.cxx2
-rw-r--r--sd/source/ui/view/ViewShellManager.cxx2
-rw-r--r--svtools/source/control/calendar.cxx18
-rw-r--r--sw/qa/extras/uiwriter/uiwriter.cxx2
-rw-r--r--sw/source/core/access/accmap.cxx2
-rw-r--r--vcl/source/control/field2.cxx4
-rw-r--r--xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx2
14 files changed, 25 insertions, 24 deletions
diff --git a/chart2/source/view/axes/ScaleAutomatism.cxx b/chart2/source/view/axes/ScaleAutomatism.cxx
index 5747d81cab1c..f58bde445981 100644
--- a/chart2/source/view/axes/ScaleAutomatism.cxx
+++ b/chart2/source/view/axes/ScaleAutomatism.cxx
@@ -570,7 +570,7 @@ void ScaleAutomatism::calculateExplicitIncrementAndScaleForDateTimeAxis(
{
case DAY:
if( rExplicitScale.ShiftedCategoryPosition )
- aMaxDate++;//for explicit scales we need one interval more (maximum excluded)
+ ++aMaxDate; //for explicit scales we need one interval more (maximum excluded)
break;
case MONTH:
aMinDate.SetDay(1);
diff --git a/chart2/source/view/main/ChartView.cxx b/chart2/source/view/main/ChartView.cxx
index 7ae6465b1307..ad10fb558add 100644
--- a/chart2/source/view/main/ChartView.cxx
+++ b/chart2/source/view/main/ChartView.cxx
@@ -1888,7 +1888,8 @@ bool ChartView::getExplicitValuesForAxis(
switch( rExplicitScale.TimeResolution )
{
case ::com::sun::star::chart::TimeUnit::DAY:
- aMaxDate--;break;
+ --aMaxDate;
+ break;
case ::com::sun::star::chart::TimeUnit::MONTH:
aMaxDate = DateHelper::GetDateSomeMonthsAway(aMaxDate,-1);
break;
diff --git a/dbaccess/source/core/api/RowSetCache.cxx b/dbaccess/source/core/api/RowSetCache.cxx
index ba588c9f569e..4a10be4f1c2d 100644
--- a/dbaccess/source/core/api/RowSetCache.cxx
+++ b/dbaccess/source/core/api/RowSetCache.cxx
@@ -1680,7 +1680,7 @@ bool ORowSetCache::reFillMatrix(sal_Int32 _nNewStartPos, sal_Int32 _nNewEndPos)
bool ORowSetCache::fill(ORowSetMatrix::iterator& _aIter, const ORowSetMatrix::const_iterator& _aEnd, sal_Int32& _nPos, bool _bCheck)
{
const sal_Int32 nColumnCount = m_xMetaData->getColumnCount();
- for(; _bCheck && _aIter != _aEnd; _aIter++, _nPos++)
+ for (; _bCheck && _aIter != _aEnd; ++_aIter, ++_nPos)
{
if ( !_aIter->is() )
*_aIter = new ORowSetValueVector(nColumnCount);
diff --git a/include/basegfx/range/b2dconnectedranges.hxx b/include/basegfx/range/b2dconnectedranges.hxx
index 66a8a94e226d..6473b2ddf990 100644
--- a/include/basegfx/range/b2dconnectedranges.hxx
+++ b/include/basegfx/range/b2dconnectedranges.hxx
@@ -190,7 +190,7 @@ namespace basegfx
}
else
{
- aCurrAggregate++;
+ ++aCurrAggregate;
}
}
}
diff --git a/include/cppuhelper/interfacecontainer.hxx b/include/cppuhelper/interfacecontainer.hxx
index d8038eef2ec9..ac89d619773b 100644
--- a/include/cppuhelper/interfacecontainer.hxx
+++ b/include/cppuhelper/interfacecontainer.hxx
@@ -69,7 +69,7 @@ inline ::com::sun::star::uno::Sequence< key > OMultiTypeInterfaceContainerHelper
if( static_cast<OInterfaceContainerHelper*>((*iter).second)->getLength() )
// yes, put the type in the array
pArray[i++] = (*iter).first;
- iter++;
+ ++iter;
}
if( i != nSize ) {
// may be empty container, reduce the sequence to the right size
diff --git a/package/source/zippackage/ZipPackageFolderEnumeration.cxx b/package/source/zippackage/ZipPackageFolderEnumeration.cxx
index 7644cbb5576c..1699ce6d67ff 100644
--- a/package/source/zippackage/ZipPackageFolderEnumeration.cxx
+++ b/package/source/zippackage/ZipPackageFolderEnumeration.cxx
@@ -51,7 +51,7 @@ uno::Any SAL_CALL ZipPackageFolderEnumeration::nextElement( )
if (aIterator == rContents.end() )
throw container::NoSuchElementException(THROW_WHERE );
aAny <<= (*aIterator).second->xTunnel;
- aIterator++;
+ ++aIterator;
return aAny;
}
diff --git a/sc/source/core/data/dociter.cxx b/sc/source/core/data/dociter.cxx
index 6ab77bc304be..46690d7f4bf1 100644
--- a/sc/source/core/data/dociter.cxx
+++ b/sc/source/core/data/dociter.cxx
@@ -2029,7 +2029,7 @@ bool ScHorizontalCellIterator::SkipInvalidInRow()
{
debugiter("skip empty cells at column %d, row %d\n",
(int)maColPos->mnCol, (int)nRow);
- maColPos++;
+ ++maColPos;
}
}
@@ -2068,7 +2068,7 @@ void ScHorizontalCellIterator::Advance()
assert (mbMore);
assert (maColPos != maColPositions.end());
- maColPos++;
+ ++maColPos;
SkipInvalid();
}
diff --git a/sd/source/core/shapelist.cxx b/sd/source/core/shapelist.cxx
index a265e9c93eff..47ba5e281705 100644
--- a/sd/source/core/shapelist.cxx
+++ b/sd/source/core/shapelist.cxx
@@ -133,7 +133,7 @@ void ShapeList::seekShape( sal_uInt32 nIndex )
{
maIter = maShapeList.begin();
while( nIndex-- && (maIter != maShapeList.end()) )
- maIter++;
+ ++maIter;
}
bool ShapeList::hasMore() const
diff --git a/sd/source/ui/view/ViewShellManager.cxx b/sd/source/ui/view/ViewShellManager.cxx
index ad678affd91b..4587c0550e27 100644
--- a/sd/source/ui/view/ViewShellManager.cxx
+++ b/sd/source/ui/view/ViewShellManager.cxx
@@ -997,7 +997,7 @@ IMPL_LINK(ViewShellManager::Implementation, WindowEventHandler, VclWindowEvent*,
{
for (ActiveShellList::iterator aI(maActiveViewShells.begin());
aI!=maActiveViewShells.end();
- aI++)
+ ++aI)
{
if (pEventWindow == static_cast< ::vcl::Window*>(aI->GetWindow()))
{
diff --git a/svtools/source/control/calendar.cxx b/svtools/source/control/calendar.cxx
index c2dc94ab5fd5..72f2966fa0b6 100644
--- a/svtools/source/control/calendar.cxx
+++ b/svtools/source/control/calendar.cxx
@@ -93,7 +93,7 @@ static void ImplCalendarSelectDateRange( IntDateSet* pTable,
while ( aStartDate <= aEndDate )
{
pTable->insert( aStartDate.GetDate() );
- aStartDate++;
+ ++aStartDate;
}
}
else
@@ -443,7 +443,7 @@ void Calendar::ImplFormat()
aTempDate += nDaysInMonth;
}
Date aTempDate2 = aTempDate;
- aTempDate2--;
+ --aTempDate2;
nDaysInMonth = aTempDate2.GetDaysInMonth();
aTempDate2 -= nDaysInMonth-1;
nWeekDay = (sal_uInt16)aTempDate2.GetDayOfWeek();
@@ -1086,7 +1086,7 @@ void Calendar::ImplMouseSelect( const Date& rDate, sal_uInt16 nHitTest,
Date aTempDate = rDate;
if ( !(nHitTest & CALENDAR_HITTEST_DAY) )
- aTempDate--;
+ --aTempDate;
if ( mbMultiSelection )
{
@@ -1231,7 +1231,7 @@ void Calendar::ImplScroll( bool bPrev )
Date aNewFirstMonth = GetFirstMonth();
if ( bPrev )
{
- aNewFirstMonth--;
+ --aNewFirstMonth;
aNewFirstMonth -= aNewFirstMonth.GetDaysInMonth()-1;
}
else
@@ -1526,11 +1526,11 @@ void Calendar::KeyInput( const KeyEvent& rKEvt )
break;
case KEY_LEFT:
- aNewDate--;
+ --aNewDate;
break;
case KEY_RIGHT:
- aNewDate++;
+ ++aNewDate;
break;
case KEY_UP:
@@ -1897,7 +1897,7 @@ void Calendar::SetCurDate( const Date& rNewDate )
{
Date aFirstDate = GetFirstMonth();
aFirstDate += aFirstDate.GetDaysInMonth();
- aTempDate++;
+ ++aTempDate;
while ( nDateOff > aTempDate.GetDaysInMonth() )
{
aFirstDate += aFirstDate.GetDaysInMonth();
@@ -1957,7 +1957,7 @@ Date Calendar::GetLastMonth() const
sal_uInt16 nMonthCount = GetMonthCount();
for ( sal_uInt16 i = 0; i < nMonthCount; i++ )
aDate += aDate.GetDaysInMonth();
- aDate--;
+ --aDate;
return aDate;
}
@@ -2037,7 +2037,7 @@ Rectangle Calendar::GetDateRect( const Date& rDate ) const
}
else
nDayIndex++;
- aLastDate++;
+ ++aLastDate;
}
}
}
diff --git a/sw/qa/extras/uiwriter/uiwriter.cxx b/sw/qa/extras/uiwriter/uiwriter.cxx
index b2850185d2a1..d76a27ae3ae3 100644
--- a/sw/qa/extras/uiwriter/uiwriter.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter.cxx
@@ -337,7 +337,7 @@ void SwUiWriterTest::testFdo74981()
aIdx = SwNodeIndex(pDoc->GetNodes().GetEndOfContent(), -1);
pTxtNode = aIdx.GetNode().GetTxtNode();
CPPUNIT_ASSERT(pTxtNode->HasHints());
- aIdx--;
+ --aIdx;
pTxtNode = aIdx.GetNode().GetTxtNode();
CPPUNIT_ASSERT(!pTxtNode->HasHints());
}
diff --git a/sw/source/core/access/accmap.cxx b/sw/source/core/access/accmap.cxx
index 5b41a9cae49e..398a8a36bf22 100644
--- a/sw/source/core/access/accmap.cxx
+++ b/sw/source/core/access/accmap.cxx
@@ -1375,7 +1375,7 @@ void SwAccessibleMap::InvalidateShapeInParaSelection()
mapTemp.insert( SwAccessibleContextMap_Impl::value_type( pFrm, xAcc ) );
}
}
- nStartIndex++;
+ ++nStartIndex;
}
}
}
diff --git a/vcl/source/control/field2.cxx b/vcl/source/control/field2.cxx
index efd554952db1..c356a827535d 100644
--- a/vcl/source/control/field2.cxx
+++ b/vcl/source/control/field2.cxx
@@ -1239,12 +1239,12 @@ static void ImplDateIncrementDay( Date& rDate, bool bUp )
if ( bUp )
{
if ( (rDate.GetDay() != 31) || (rDate.GetMonth() != 12) || (rDate.GetYear() != 9999) )
- rDate++;
+ ++rDate;
}
else
{
if ( (rDate.GetDay() != 1 ) || (rDate.GetMonth() != 1) || (rDate.GetYear() != 0) )
- rDate--;
+ --rDate;
}
}
diff --git a/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx b/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx
index be14e0eeeba6..0e2a2cbc2a24 100644
--- a/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx
+++ b/xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx
@@ -127,7 +127,7 @@ bool UnoInterfaceToUniqueIdentifierMapper::findReference( const Reference< XInte
if( (*rIter).second.get() == xRef.get() )
return true;
- rIter++;
+ ++rIter;
}
return false;