diff options
author | Michael Stahl <mstahl@redhat.com> | 2015-03-13 14:16:51 +0100 |
---|---|---|
committer | Michael Stahl <mstahl@redhat.com> | 2015-03-13 16:23:26 +0100 |
commit | 077723111292ea615437f3bc2f1e47cf77d7ad42 (patch) | |
tree | f851f81cea82fcbf8251f8418b3f2a26f3085f64 | |
parent | 386a85ed50a3e4832644072950a30304ba6c58a6 (diff) |
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.cxx | 2 | ||||
-rw-r--r-- | chart2/source/view/main/ChartView.cxx | 3 | ||||
-rw-r--r-- | dbaccess/source/core/api/RowSetCache.cxx | 2 | ||||
-rw-r--r-- | include/basegfx/range/b2dconnectedranges.hxx | 2 | ||||
-rw-r--r-- | include/cppuhelper/interfacecontainer.hxx | 2 | ||||
-rw-r--r-- | package/source/zippackage/ZipPackageFolderEnumeration.cxx | 2 | ||||
-rw-r--r-- | sc/source/core/data/dociter.cxx | 4 | ||||
-rw-r--r-- | sd/source/core/shapelist.cxx | 2 | ||||
-rw-r--r-- | sd/source/ui/view/ViewShellManager.cxx | 2 | ||||
-rw-r--r-- | svtools/source/control/calendar.cxx | 18 | ||||
-rw-r--r-- | sw/qa/extras/uiwriter/uiwriter.cxx | 2 | ||||
-rw-r--r-- | sw/source/core/access/accmap.cxx | 2 | ||||
-rw-r--r-- | vcl/source/control/field2.cxx | 4 | ||||
-rw-r--r-- | xmloff/source/core/unointerfacetouniqueidentifiermapper.cxx | 2 |
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; |