diff options
author | Caolán McNamara <caolanm@redhat.com> | 2017-01-19 10:41:38 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2017-01-19 15:16:15 +0000 |
commit | 545c6e9d4fa3209eefd3867077c5fe820f0675d0 (patch) | |
tree | 1c19c5411a69e7771e6e50f554f13a54215617bd /basic/source/runtime/methods1.cxx | |
parent | ecd82b5121899fe23f0ccbdd8a89c8b18436f027 (diff) |
clear ARRAY_VS_SINGLETON coverity warnings
experiment to see if this clears them
Change-Id: I91a74e06f621d3012b564cad87586ba74b0367ca
Diffstat (limited to 'basic/source/runtime/methods1.cxx')
-rw-r--r-- | basic/source/runtime/methods1.cxx | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/basic/source/runtime/methods1.cxx b/basic/source/runtime/methods1.cxx index eb0f245577ba..bad68c593f84 100644 --- a/basic/source/runtime/methods1.cxx +++ b/basic/source/runtime/methods1.cxx @@ -832,13 +832,13 @@ RTLFUNC(Array) SbxVariable* pVar = rPar.Get(i+1); SbxVariable* pNew = new SbxEnsureParentVariable(*pVar); pNew->SetFlag( SbxFlagBits::Write ); - short index = static_cast< short >(i); + short aIdx[1]; + aIdx[0] = static_cast< short >(i); if ( bIncIndex ) { - ++index; + ++aIdx[0]; } - // coverity[callee_ptr_arith] - pArray->Put( pNew, &index ); + pArray->Put(pNew, aIdx); } // return array @@ -1724,12 +1724,12 @@ RTLFUNC(Join) OUString aRetStr; short nLower, nUpper; pArr->GetDim( 1, nLower, nUpper ); - for (short i = nLower; i <= nUpper; ++i) + short aIdx[1]; + for (aIdx[0] = nLower; aIdx[0] <= nUpper; ++aIdx[0]) { - // coverity[callee_ptr_arith] - OUString aStr = pArr->Get( &i )->GetOUString(); + OUString aStr = pArr->Get(aIdx)->GetOUString(); aRetStr += aStr; - if( i != nUpper ) + if (aIdx[0] != nUpper) { aRetStr += aDelim; } |