summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatteo Casalin <matteo.casalin@yahoo.com>2015-05-31 18:08:28 +0200
committerMatteo Casalin <matteo.casalin@yahoo.com>2015-05-31 23:35:53 +0200
commit18cf9461f58d652296424b25536543fe5371964b (patch)
treed26a00f3769fdd45d9b3739c3c88c6eb22a04871
parentbd54e40d69cd79fc4d79573c23a9dca460d71819 (diff)
Use more proper integer types and range-based for loops
Change-Id: I59dc93687063e1df082d5c96c691dd364cc78ddb
-rw-r--r--sw/source/filter/html/htmlatr.cxx54
1 files changed, 24 insertions, 30 deletions
diff --git a/sw/source/filter/html/htmlatr.cxx b/sw/source/filter/html/htmlatr.cxx
index 5664b4f2d1b6..2b9fa877f9d7 100644
--- a/sw/source/filter/html/htmlatr.cxx
+++ b/sw/source/filter/html/htmlatr.cxx
@@ -356,10 +356,10 @@ SwHTMLFormatInfo::SwHTMLFormatInfo( const SwFormat *pF, SwDoc *pDoc, SwDoc *pTem
aSets[1] = 1;
break;
}
- for( sal_uInt16 i=0; i<4; i++ )
+ for( int i=0; i<4; ++i )
{
const SfxPoolItem& rRef = pFormat->GetFormatAttr( aWhichIds[nRef][i] );
- for( sal_uInt16 j=0; j<2; j++ )
+ for( size_t j=0; j<SAL_N_ELEMENTS(aSets); ++j )
{
const SfxPoolItem& rSet = pFormat->GetFormatAttr( aWhichIds[aSets[j]][i] );
if( rSet != rRef )
@@ -430,7 +430,7 @@ SwHTMLFormatInfo::SwHTMLFormatInfo( const SwFormat *pF, SwDoc *pDoc, SwDoc *pTem
static const sal_uInt16 aWhichIds[3] =
{ RES_CHRATR_LANGUAGE, RES_CHRATR_CJK_LANGUAGE,
RES_CHRATR_CTL_LANGUAGE };
- for( sal_uInt16 i=0; i<3; i++ )
+ for( size_t i=0; i<SAL_N_ELEMENTS(aWhichIds); ++i )
{
if( aWhichIds[i] != nWhichId )
{
@@ -854,7 +854,7 @@ void OutHTML_SwFormat( Writer& rWrt, const SwFormat& rFormat,
{
static const sal_uInt16 aWhichIds[3] = { RES_CHRATR_LANGUAGE, RES_CHRATR_CJK_LANGUAGE, RES_CHRATR_CTL_LANGUAGE };
- for( sal_uInt16 i=0; i<3; i++ )
+ for( size_t i=0; i<SAL_N_ELEMENTS(aWhichIds); ++i )
{
// export language if it differs from the default language only.
const SfxPoolItem *pTmpItem;
@@ -1124,8 +1124,8 @@ class HTMLEndPosLst
// Eine SttEndPos in die Start- und Ende-Listen eintragen bzw. aus
// ihnen loeschen, wobei die Ende-Position bekannt ist
- void _InsertItem( HTMLStartEndPos *pPos, sal_uInt16 nEndPos );
- void _RemoveItem( sal_uInt16 nEndPos );
+ void _InsertItem( HTMLStartEndPos *pPos, HTMLStartEndPositions::size_type nEndPos );
+ void _RemoveItem( HTMLStartEndPositions::size_type nEndPos );
// die "Art" es Attributs ermitteln
HTMLOnOffState GetHTMLItemState( const SfxPoolItem& rItem );
@@ -1140,7 +1140,7 @@ class HTMLEndPosLst
// das Ende eines gesplitteten Items anpassen
void FixSplittedItem( HTMLStartEndPos *pPos, sal_Int32 nNewEnd,
- sal_uInt16 nStartPos );
+ HTMLStartEndPositions::size_type nStartPos );
// Ein Attribut in die Listen eintragen und ggf. aufteilen
void InsertItem( const SfxPoolItem& rItem, sal_Int32 nStart,
@@ -1185,23 +1185,22 @@ public:
bool IsHTMLMode( sal_uLong nMode ) const { return (nHTMLMode & nMode) != 0; }
};
-void HTMLEndPosLst::_InsertItem( HTMLStartEndPos *pPos, sal_uInt16 nEndPos )
+void HTMLEndPosLst::_InsertItem( HTMLStartEndPos *pPos, HTMLStartEndPositions::size_type nEndPos )
{
// In der Start-Liste das Attribut hinter allen vorher und an
// der gleichen Position gestarteten Attributen einfuegen
sal_Int32 nStart = pPos->GetStart();
- sal_uInt16 i;
+ HTMLStartEndPositions::size_type i {0};
- for( i = 0; i < aStartLst.size() &&
- aStartLst[i]->GetStart() <= nStart; i++ )
- ;
+ while( i < aStartLst.size() && aStartLst[i]->GetStart() <= nStart )
+ ++i;
aStartLst.insert( aStartLst.begin() + i, pPos );
// die Position in der Ende-Liste wurde uebergeben
aEndLst.insert( aEndLst.begin() + nEndPos, pPos );
}
-void HTMLEndPosLst::_RemoveItem( sal_uInt16 nEndPos )
+void HTMLEndPosLst::_RemoveItem( HTMLStartEndPositions::size_type nEndPos )
{
HTMLStartEndPos *pPos = aEndLst[nEndPos];
@@ -1369,10 +1368,8 @@ HTMLOnOffState HTMLEndPosLst::GetHTMLItemState( const SfxPoolItem& rItem )
bool HTMLEndPosLst::ExistsOnTagItem( sal_uInt16 nWhich, sal_Int32 nPos )
{
- for( sal_uInt16 i=0; i<aStartLst.size(); i++ )
+ for( auto pTest : aStartLst )
{
- HTMLStartEndPos *pTest = aStartLst[i];
-
if( pTest->GetStart() > nPos )
{
// dieses uns alle folgenden Attribute beginnen spaeter
@@ -1405,10 +1402,8 @@ bool HTMLEndPosLst::ExistsOffTagItem( sal_uInt16 nWhich, sal_Int32 nStartPos,
return false;
}
- for( sal_uInt16 i=0; i<aStartLst.size(); i++ )
+ for( auto pTest : aStartLst )
{
- HTMLStartEndPos *pTest = aStartLst[i];
-
if( pTest->GetStart() > nStartPos )
{
// dieses uns alle folgenden Attribute beginnen spaeter
@@ -1437,7 +1432,7 @@ bool HTMLEndPosLst::ExistsOffTagItem( sal_uInt16 nWhich, sal_Int32 nStartPos,
}
void HTMLEndPosLst::FixSplittedItem( HTMLStartEndPos *pPos, sal_Int32 nNewEnd,
- sal_uInt16 nStartPos )
+ HTMLStartEndPositions::size_type nStartPos )
{
// die End-Position entsprechend fixen
pPos->SetEnd( nNewEnd );
@@ -1451,14 +1446,13 @@ void HTMLEndPosLst::FixSplittedItem( HTMLStartEndPos *pPos, sal_Int32 nNewEnd,
// es wird von nun an als letztes an der entsprechenden Position
// beendet
- HTMLStartEndPositions::size_type nEndPos;
- for( nEndPos=0; nEndPos < aEndLst.size() &&
- aEndLst[nEndPos]->GetEnd() <= nNewEnd; nEndPos++ )
- ;
+ HTMLStartEndPositions::size_type nEndPos {0};
+ while( nEndPos < aEndLst.size() && aEndLst[nEndPos]->GetEnd() <= nNewEnd )
+ ++nEndPos;
aEndLst.insert( aEndLst.begin() + nEndPos, pPos );
// jetzt noch die spaeter gestarteten Attribute anpassen
- for( sal_uInt16 i=nStartPos+1; i<aStartLst.size(); i++ )
+ for( HTMLStartEndPositions::size_type i = nStartPos+1; i<aStartLst.size(); ++i )
{
HTMLStartEndPos *pTest = aStartLst[i];
sal_Int32 nTestEnd = pTest->GetEnd();
@@ -1495,7 +1489,7 @@ void HTMLEndPosLst::FixSplittedItem( HTMLStartEndPos *pPos, sal_Int32 nNewEnd,
void HTMLEndPosLst::InsertItem( const SfxPoolItem& rItem, sal_Int32 nStart,
sal_Int32 nEnd )
{
- sal_uInt16 i;
+ HTMLStartEndPositions::size_type i;
for( i = 0; i < aEndLst.size(); i++ )
{
HTMLStartEndPos *pTest = aEndLst[i];
@@ -1535,7 +1529,7 @@ void HTMLEndPosLst::SplitItem( const SfxPoolItem& rItem, sal_Int32 nStart,
// erstmal muessen wir die alten Items anhand der Startliste suchen
// und die neuen Item-Bereiche festlegen
- for( sal_uInt16 i=0; i<aStartLst.size(); i++ )
+ for( HTMLStartEndPositions::size_type i=0; i<aStartLst.size(); ++i )
{
HTMLStartEndPos *pTest = aStartLst[i];
sal_Int32 nTestStart = pTest->GetStart();
@@ -1932,7 +1926,7 @@ void HTMLEndPosLst::OutStartAttrs( SwHTMLWriter& rHWrt, sal_Int32 nPos,
// Character border attribute must be the first which is written out
// because of border merge.
- sal_uInt16 nCharBoxIndex = 0;
+ HTMLStartEndPositions::size_type nCharBoxIndex = 0;
while( nCharBoxIndex < aStartLst.size() &&
aStartLst[nCharBoxIndex]->GetItem()->Which() != RES_CHRATR_BOX )
{
@@ -1940,7 +1934,7 @@ void HTMLEndPosLst::OutStartAttrs( SwHTMLWriter& rHWrt, sal_Int32 nPos,
}
// die Attribute in der Start-Liste sind aufsteigend sortiert
- for( sal_uInt16 i=0; i< aStartLst.size(); i++ )
+ for( HTMLStartEndPositions::size_type i=0; i< aStartLst.size(); ++i )
{
HTMLStartEndPos *pPos = 0;
if( nCharBoxIndex < aStartLst.size() )
@@ -1989,7 +1983,7 @@ void HTMLEndPosLst::OutEndAttrs( SwHTMLWriter& rHWrt, sal_Int32 nPos,
rHWrt.bTagOn = false;
// die Attribute in der End-Liste sind aufsteigend sortiert
- sal_uInt16 i=0;
+ HTMLStartEndPositions::size_type i {0};
while( i < aEndLst.size() )
{
HTMLStartEndPos *pPos = aEndLst[i];