diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2015-10-07 16:15:27 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2015-10-07 16:35:36 +0200 |
| commit | 91d2b58e759eadf7cbc360da829a3ffc8e1e7d2f (patch) | |
| tree | a34c0d3d5eef9bdce37fd2777906ddb24fe85d63 | |
| parent | bb36996c6eef24cc84cdf134957201e3a19d5c5a (diff) | |
clang-analyzer-deadcode.DeadStores
Change-Id: If0d69a8d3bfaef732e436fad6a91851095fdb3be
| -rw-r--r-- | svl/source/items/itemset.cxx | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/svl/source/items/itemset.cxx b/svl/source/items/itemset.cxx index 2b6be1719eb7..d9363c14b725 100644 --- a/svl/source/items/itemset.cxx +++ b/svl/source/items/itemset.cxx @@ -941,7 +941,6 @@ void SfxItemSet::Intersect( const SfxItemSet& rSet ) } // Test whether the Which Ranges are different - bool bEqual = true; sal_uInt16* pWh1 = m_pWhichRanges; sal_uInt16* pWh2 = rSet.m_pWhichRanges; sal_uInt16 nSize = 0; @@ -950,13 +949,12 @@ void SfxItemSet::Intersect( const SfxItemSet& rSet ) { if( *pWh1 != *pWh2 ) { - bEqual = false; break; } if( n & 1 ) nSize += ( *(pWh1) - *(pWh1-1) ) + 1; } - bEqual = *pWh1 == *pWh2; // Also check for 0 + bool bEqual = *pWh1 == *pWh2; // Also check for 0 // If the Ranges are identical, we can easily process it if( bEqual ) @@ -1009,7 +1007,6 @@ void SfxItemSet::Differentiate( const SfxItemSet& rSet ) return; // Test whether the Which Ranges are different - bool bEqual = true; sal_uInt16* pWh1 = m_pWhichRanges; sal_uInt16* pWh2 = rSet.m_pWhichRanges; sal_uInt16 nSize = 0; @@ -1018,13 +1015,12 @@ void SfxItemSet::Differentiate( const SfxItemSet& rSet ) { if( *pWh1 != *pWh2 ) { - bEqual = false; break; } if( n & 1 ) nSize += ( *(pWh1) - *(pWh1-1) ) + 1; } - bEqual = *pWh1 == *pWh2; // Also test for 0 + bool bEqual = *pWh1 == *pWh2; // Also test for 0 // If the Ranges are identical, we can easily process it if( bEqual ) @@ -1221,7 +1217,6 @@ void SfxItemSet::MergeValues( const SfxItemSet& rSet, bool bIgnoreDefaults ) assert( GetPool() == rSet.GetPool() && "MergeValues with different Pools" ); // Test if the which Ranges are different - bool bEqual = true; sal_uInt16* pWh1 = m_pWhichRanges; sal_uInt16* pWh2 = rSet.m_pWhichRanges; sal_uInt16 nSize = 0; @@ -1230,13 +1225,12 @@ void SfxItemSet::MergeValues( const SfxItemSet& rSet, bool bIgnoreDefaults ) { if( *pWh1 != *pWh2 ) { - bEqual = false; break; } if( n & 1 ) nSize += ( *(pWh1) - *(pWh1-1) ) + 1; } - bEqual = *pWh1 == *pWh2; // Also check for 0 + bool bEqual = *pWh1 == *pWh2; // Also check for 0 // If the Ranges match, they are easier to process! if( bEqual ) |