diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-06 14:49:15 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-10 09:57:24 +0200 |
| commit | 4250b25c6ae361359300ab6ccde27230f8e01039 (patch) | |
| tree | 916a8420282928a92ede0760d696997550ae0840 /svl | |
| parent | 2ed9a2b641682d8612b5404bd3978ed049aa0266 (diff) | |
teach unnecessaryparen loplugin about identifiers
Change-Id: I5710b51e53779c222cec0bf08cd34bda330fec4b
Reviewed-on: https://gerrit.libreoffice.org/39737
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svl')
| -rw-r--r-- | svl/source/items/itemset.cxx | 8 | ||||
| -rw-r--r-- | svl/source/items/poolio.cxx | 4 | ||||
| -rw-r--r-- | svl/source/numbers/numfmuno.cxx | 4 | ||||
| -rw-r--r-- | svl/source/numbers/zforfind.cxx | 2 |
4 files changed, 9 insertions, 9 deletions
diff --git a/svl/source/items/itemset.cxx b/svl/source/items/itemset.cxx index 62b1ea3cb8c3..d59872fa3799 100644 --- a/svl/source/items/itemset.cxx +++ b/svl/source/items/itemset.cxx @@ -946,7 +946,7 @@ void SfxItemSet::Intersect( const SfxItemSet& rSet ) break; } if( n & 1 ) - nSize += ( *(pWh1) - *(pWh1-1) ) + 1; + nSize += ( *pWh1 - *(pWh1-1) ) + 1; } bool bEqual = *pWh1 == *pWh2; // Also check for 0 @@ -1012,7 +1012,7 @@ void SfxItemSet::Differentiate( const SfxItemSet& rSet ) break; } if( n & 1 ) - nSize += ( *(pWh1) - *(pWh1-1) ) + 1; + nSize += ( *pWh1 - *(pWh1-1) ) + 1; } bool bEqual = *pWh1 == *pWh2; // Also test for 0 @@ -1222,7 +1222,7 @@ void SfxItemSet::MergeValues( const SfxItemSet& rSet ) break; } if( n & 1 ) - nSize += ( *(pWh1) - *(pWh1-1) ) + 1; + nSize += ( *pWh1 - *(pWh1-1) ) + 1; } bool bEqual = *pWh1 == *pWh2; // Also check for 0 @@ -1315,7 +1315,7 @@ sal_uInt16 SfxItemSet::GetWhichByPos( sal_uInt16 nPos ) const { n = ( *(pPtr+1) - *pPtr ) + 1; if( nPos < n ) - return *(pPtr)+nPos; + return *pPtr + nPos; nPos = nPos - n; pPtr += 2; } diff --git a/svl/source/items/poolio.cxx b/svl/source/items/poolio.cxx index d2829f703b16..2f10ef1ba82a 100644 --- a/svl/source/items/poolio.cxx +++ b/svl/source/items/poolio.cxx @@ -524,7 +524,7 @@ SvStream &SfxItemPool::Load(SvStream &rStream) { OSL_FAIL( "SFX_ITEMPOOL_TAG_STARTPOOL_5" ); /*! s.u. */ /*! Set error code and evaluate! */ - (rStream).SetError(SVSTREAM_FILEFORMAT_ERROR); + rStream.SetError(SVSTREAM_FILEFORMAT_ERROR); pImpl->bStreaming = false; return rStream; } @@ -550,7 +550,7 @@ SvStream &SfxItemPool::Load(SvStream &rStream) { OSL_FAIL( "SFX_ITEMPOOL_TAG_TRICK4OLD" ); /*! s.u. */ /*! Set error code and evaluate! */ - (rStream).SetError(SVSTREAM_FILEFORMAT_ERROR); + rStream.SetError(SVSTREAM_FILEFORMAT_ERROR); pImpl->bStreaming = false; return rStream; } diff --git a/svl/source/numbers/numfmuno.cxx b/svl/source/numbers/numfmuno.cxx index efb1555edcda..bef4075994e1 100644 --- a/svl/source/numbers/numfmuno.cxx +++ b/svl/source/numbers/numfmuno.cxx @@ -705,12 +705,12 @@ uno::Any SAL_CALL SvNumberFormatObj::getPropertyValue( const OUString& aProperty else if (aPropertyName == PROPERTYNAME_DECIMALS) { pFormat->GetFormatSpecialInfo( bThousand, bRed, nDecimals, nLeading ); - aRet <<= (sal_Int16)( nDecimals ); + aRet <<= (sal_Int16) nDecimals; } else if (aPropertyName == PROPERTYNAME_LEADING) { pFormat->GetFormatSpecialInfo( bThousand, bRed, nDecimals, nLeading ); - aRet <<= (sal_Int16)( nLeading ); + aRet <<= (sal_Int16) nLeading; } else if (aPropertyName == PROPERTYNAME_NEGRED) { diff --git a/svl/source/numbers/zforfind.cxx b/svl/source/numbers/zforfind.cxx index d022a2b4cd63..ddf0f9983f80 100644 --- a/svl/source/numbers/zforfind.cxx +++ b/svl/source/numbers/zforfind.cxx @@ -480,7 +480,7 @@ bool ImpSvNumberInputScan::StringContainsWord( const OUString& rWhat, if ((nType & (KCharacterType::UPPER | KCharacterType::LOWER | KCharacterType::DIGIT)) != 0) return false; // Alpha or numeric is not word gap. - if ((nType & (KCharacterType::LETTER)) != 0) + if (nType & KCharacterType::LETTER) return true; // Letter other than alpha is new word. (Is it?) return true; // Catch all remaining as gap until we know better. |