diff options
author | Michael Stahl <mst@openoffice.org> | 2012-07-31 15:26:19 +0200 |
---|---|---|
committer | Petr Mladek <pmladek@suse.cz> | 2012-07-31 15:26:19 +0200 |
commit | c096d40a4c7bf3231813126d6dc118292e1da506 (patch) | |
tree | f354c7d14b14e124ddbfe545fc6ae37e5db20f2d | |
parent | 10790f7a0bcf0fdcfafedda8c98a8f6ef845fd38 (diff) |
basic, cui, extensions, filter, vcl: fix some warnings
Signed-off-by: Petr Mladek <pmladek@suse.cz>
-rw-r--r-- | basic/source/runtime/runtime.cxx | 6 | ||||
-rw-r--r-- | cui/source/tabpages/numpages.cxx | 6 | ||||
-rw-r--r-- | extensions/source/scanner/sane.cxx | 2 | ||||
-rw-r--r-- | filter/source/graphicfilter/icgm/chart.cxx | 2 | ||||
-rw-r--r-- | vcl/source/gdi/metric.cxx | 2 | ||||
-rw-r--r-- | vcl/source/gdi/outmap.cxx | 11 |
6 files changed, 18 insertions, 11 deletions
diff --git a/basic/source/runtime/runtime.cxx b/basic/source/runtime/runtime.cxx index 53f104eea8aa..3b70b04ccea2 100644 --- a/basic/source/runtime/runtime.cxx +++ b/basic/source/runtime/runtime.cxx @@ -711,17 +711,17 @@ sal_Bool SbiRuntime::Step() SbiOpcode eOp = (SbiOpcode ) ( *pCode++ ); sal_uInt32 nOp1, nOp2; - if( eOp <= SbOP0_END ) + if (eOp < SbOP0_END) { (this->*( aStep0[ eOp ] ) )(); } - else if( eOp >= SbOP1_START && eOp <= SbOP1_END ) + else if (eOp >= SbOP1_START && eOp < SbOP1_END) { nOp1 = *pCode++; nOp1 |= *pCode++ << 8; nOp1 |= *pCode++ << 16; nOp1 |= *pCode++ << 24; (this->*( aStep1[ eOp - SbOP1_START ] ) )( nOp1 ); } - else if( eOp >= SbOP2_START && eOp <= SbOP2_END ) + else if (eOp >= SbOP2_START && eOp < SbOP2_END) { nOp1 = *pCode++; nOp1 |= *pCode++ << 8; nOp1 |= *pCode++ << 16; nOp1 |= *pCode++ << 24; nOp2 = *pCode++; nOp2 |= *pCode++ << 8; nOp2 |= *pCode++ << 16; nOp2 |= *pCode++ << 24; diff --git a/cui/source/tabpages/numpages.cxx b/cui/source/tabpages/numpages.cxx index dd5d346b95d6..1b28cbc4d8bc 100644 --- a/cui/source/tabpages/numpages.cxx +++ b/cui/source/tabpages/numpages.cxx @@ -2850,6 +2850,12 @@ void SvxNumPositionTabPage::InitControls() nMask <<= 1; } + if (SVX_MAX_NUM <= nLvl) + { + OSL_ENSURE(false, "cannot happen."); + return; + } + if(bSameDistBorderNum) { long nDistBorderNum; diff --git a/extensions/source/scanner/sane.cxx b/extensions/source/scanner/sane.cxx index a26ff3caf0e5..b58d05b56fa0 100644 --- a/extensions/source/scanner/sane.cxx +++ b/extensions/source/scanner/sane.cxx @@ -985,7 +985,7 @@ String Sane::GetOptionUnitName( int n ) String aText; SANE_Unit nUnit = mppOptions[n]->unit; size_t nUnitAsSize = (size_t)nUnit; - if( nUnitAsSize > SAL_N_ELEMENTS( ppUnits ) ) + if (nUnitAsSize >= SAL_N_ELEMENTS( ppUnits )) aText = String::CreateFromAscii( "[unknown units]" ); else aText = String( ppUnits[ nUnit ], osl_getThreadTextEncoding() ); diff --git a/filter/source/graphicfilter/icgm/chart.cxx b/filter/source/graphicfilter/icgm/chart.cxx index 7e34fdfd1fe5..5eb52ab58880 100644 --- a/filter/source/graphicfilter/icgm/chart.cxx +++ b/filter/source/graphicfilter/icgm/chart.cxx @@ -69,7 +69,6 @@ void CGMChart::DeleteTextEntry( TextEntry* pTextEntry ) pTAttr = pTAttr->pNextAttribute; delete pTempTAttr; } - delete pTextEntry; ::std::vector< TextEntry* >::iterator it; for ( it = maTextEntryList.begin(); it < maTextEntryList.end(); ++it ) { @@ -79,6 +78,7 @@ void CGMChart::DeleteTextEntry( TextEntry* pTextEntry ) break; } } + delete pTextEntry; } }; diff --git a/vcl/source/gdi/metric.cxx b/vcl/source/gdi/metric.cxx index d0ad6e984419..27094940df9e 100644 --- a/vcl/source/gdi/metric.cxx +++ b/vcl/source/gdi/metric.cxx @@ -714,7 +714,7 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, CmapResult& rResult ) while( cMin < cEnd ) { int j = 0; - for(; (cMin < cEnd) && (j < NINSIZE); ++cMin ) + for (; (cMin < cEnd) && (j < (NINSIZE-1)); ++cMin) { if( cMin >= 0x0100 ) cCharsInp[ j++ ] = static_cast<sal_Char>(cMin >> 8); diff --git a/vcl/source/gdi/outmap.cxx b/vcl/source/gdi/outmap.cxx index 740d5adb594f..c911c8b0df85 100644 --- a/vcl/source/gdi/outmap.cxx +++ b/vcl/source/gdi/outmap.cxx @@ -61,9 +61,10 @@ DBG_NAMEEX( Region ) // ======================================================================= -static long aImplNumeratorAry[MAP_PIXEL+1] = +static int const s_ImplArySize = MAP_PIXEL+1; +static long aImplNumeratorAry[s_ImplArySize] = { 1, 1, 5, 50, 1, 1, 1, 1, 1, 1, 1 }; -static long aImplDenominatorAry[MAP_PIXEL+1] = +static long aImplDenominatorAry[s_ImplArySize] = { 2540, 254, 127, 127, 1000, 100, 10, 1, 72, 1440, 1 }; // ----------------------------------------------------------------------- @@ -1861,9 +1862,9 @@ Region OutputDevice::PixelToLogic( const Region& rDeviceRegion, #define ENTER3( eUnitSource, eUnitDest ) \ long nNumerator = 1; \ long nDenominator = 1; \ - DBG_ASSERT( eUnitSource <= MAP_PIXEL, "nonpermitted source map unit"); \ - DBG_ASSERT( eUnitDest <= MAP_PIXEL, "nonpermitted destination map unit"); \ - if( (eUnitSource <= MAP_PIXEL) && (eUnitDest <= MAP_PIXEL) ) \ + DBG_ASSERT( eUnitSource < s_ImplArySize, "Invalid source map unit"); \ + DBG_ASSERT( eUnitDest < s_ImplArySize, "Invalid destination map unit"); \ + if( (eUnitSource < s_ImplArySize) && (eUnitDest < s_ImplArySize) ) \ { \ nNumerator = aImplNumeratorAry[eUnitSource] * \ aImplDenominatorAry[eUnitDest]; \ |