mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

remove unused enum value SplitWindowItemFlags::Invisible

Browse Source 
Change-Id: I4d63601ca3658b4ffd9eaf1c13d4c17503640aa0
Reviewed-on: https://gerrit.libreoffice.org/64041
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
Noel Grandin
2018-11-23 15:18:50 +02:00
parent 361fa01df3
commit 37b3502b5c
3 changed files with 34 additions and 70 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
compilerplugins/clang/unusedenumconstants.readonly.results
View File

@@ -1070,8 +1070,6 @@ include/vcl/ptrstyle.hxx:92
enum PointerStyle Chart enum PointerStyle Chart
include/vcl/salctype.hxx:41 include/vcl/salctype.hxx:41
enum ConvertDataFormat PDF enum ConvertDataFormat PDF
include/vcl/splitwin.hxx:37
enum SplitWindowItemFlags Invisible
include/vcl/vclenum.hxx:37 include/vcl/vclenum.hxx:37
enum MenuItemBits POPUPSELECT enum MenuItemBits POPUPSELECT
include/vcl/vclenum.hxx:143 include/vcl/vclenum.hxx:143

3
include/vcl/splitwin.hxx
View File

@@ -34,11 +34,10 @@ enum class SplitWindowItemFlags
RelativeSize = 0x0002, RelativeSize = 0x0002,
PercentSize = 0x0004, PercentSize = 0x0004,
ColSet = 0x0008, ColSet = 0x0008,
Invisible = 0x0010,
}; };
namespace o3tl namespace o3tl
{ {
template<> struct typed_flags<SplitWindowItemFlags> : is_typed_flags<SplitWindowItemFlags, 0x1f> {}; template<> struct typed_flags<SplitWindowItemFlags> : is_typed_flags<SplitWindowItemFlags, 0x0f> {};
} }
#define SPLITWINDOW_APPEND (sal_uInt16(0xFFFF)) #define SPLITWINDOW_APPEND (sal_uInt16(0xFFFF))

39
vcl/source/window/splitwin.cxx
View File

@@ -401,7 +401,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
sal_uInt16 nMins; sal_uInt16 nMins;
sal_uInt16 nCalcItems; sal_uInt16 nCalcItems;
size_t nItems = pSet->mvItems.size(); size_t nItems = pSet->mvItems.size();
sal_uInt16 nVisItems;
sal_uInt16 nAbsItems; sal_uInt16 nAbsItems;
long nCalcSize; long nCalcSize;
long nPos; long nPos;
@@ -409,20 +408,12 @@ static void ImplCalcSet( ImplSplitSet* pSet,
std::vector< ImplSplitItem >& rItems = pSet->mvItems; std::vector< ImplSplitItem >& rItems = pSet->mvItems;
bool bEmpty; bool bEmpty;
// get number of visible items
nVisItems = 0;
for ( auto& rItem : rItems )
{
if ( !(rItem.mnBits & SplitWindowItemFlags::Invisible) )
nVisItems++;
}
// calculate sizes // calculate sizes
if ( bRows ) if ( bRows )
nCalcSize = nSetHeight; nCalcSize = nSetHeight;
else else
nCalcSize = nSetWidth; nCalcSize = nSetWidth;
nCalcSize -= (nVisItems-1)*pSet->mnSplitSize; nCalcSize -= (rItems.size()-1)*pSet->mnSplitSize;
if ( pSet->mbCalcPix || (pSet->mnLastSize != nCalcSize) ) if ( pSet->mbCalcPix || (pSet->mnLastSize != nCalcSize) )
{ {
long nPercentFactor = 10; long nPercentFactor = 10;
@@ -432,8 +423,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
long nAbsSize = 0; long nAbsSize = 0;
long nCurSize = 0; long nCurSize = 0;
for ( auto& rItem : rItems ) for ( auto& rItem : rItems )
{
if ( !(rItem.mnBits & SplitWindowItemFlags::Invisible) )
{ {
if ( rItem.mnBits & SplitWindowItemFlags::RelativeSize ) if ( rItem.mnBits & SplitWindowItemFlags::RelativeSize )
nRelCount += rItem.mnSize; nRelCount += rItem.mnSize;
@@ -442,7 +431,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
else else
nAbsSize += rItem.mnSize; nAbsSize += rItem.mnSize;
} }
}
// map relative values to percentages (percentage here one tenth of a procent) // map relative values to percentages (percentage here one tenth of a procent)
nPercent *= nPercentFactor; nPercent *= nPercentFactor;
if ( nRelCount ) if ( nRelCount )
@@ -466,9 +454,7 @@ static void ImplCalcSet( ImplSplitSet* pSet,
long nSizeDelta = nCalcSize-nAbsSize; long nSizeDelta = nCalcSize-nAbsSize;
for ( auto& rItem : rItems ) for ( auto& rItem : rItems )
{ {
if ( rItem.mnBits & SplitWindowItemFlags::Invisible ) if ( rItem.mnBits & SplitWindowItemFlags::RelativeSize )
rItem.mnPixSize = 0;
else if ( rItem.mnBits & SplitWindowItemFlags::RelativeSize )
{ {
if ( nSizeDelta <= 0 ) if ( nSizeDelta <= 0 )
rItem.mnPixSize = 0; rItem.mnPixSize = 0;
@@ -499,8 +485,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
// first resize absolute items relative // first resize absolute items relative
for ( auto& rItem : rItems ) for ( auto& rItem : rItems )
{
if ( !(rItem.mnBits & SplitWindowItemFlags::Invisible) )
{ {
if ( !(rItem.mnBits & (SplitWindowItemFlags::RelativeSize | SplitWindowItemFlags::PercentSize)) ) if ( !(rItem.mnBits & (SplitWindowItemFlags::RelativeSize | SplitWindowItemFlags::PercentSize)) )
{ {
@@ -508,15 +492,12 @@ static void ImplCalcSet( ImplSplitSet* pSet,
nSizeWinSize += rItem.mnPixSize; nSizeWinSize += rItem.mnPixSize;
} }
} }
}
// do not compensate rounding errors here // do not compensate rounding errors here
if ( (nAbsItems < static_cast<sal_uInt16>(std::abs( nSizeDelta ))) && nSizeWinSize ) if ( (nAbsItems < static_cast<sal_uInt16>(std::abs( nSizeDelta ))) && nSizeWinSize )
{ {
long nNewSizeWinSize = 0; long nNewSizeWinSize = 0;
for ( auto& rItem : rItems ) for ( auto& rItem : rItems )
{
if ( !(rItem.mnBits & SplitWindowItemFlags::Invisible) )
{ {
if ( !(rItem.mnBits & (SplitWindowItemFlags::RelativeSize | SplitWindowItemFlags::PercentSize)) ) if ( !(rItem.mnBits & (SplitWindowItemFlags::RelativeSize | SplitWindowItemFlags::PercentSize)) )
{ {
@@ -524,7 +505,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
nNewSizeWinSize += rItem.mnPixSize; nNewSizeWinSize += rItem.mnPixSize;
} }
} }
}
nSizeDelta -= nNewSizeWinSize-nSizeWinSize; nSizeDelta -= nNewSizeWinSize-nSizeWinSize;
} }
@@ -545,8 +525,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
if ( j >= 2 ) if ( j >= 2 )
rItem.mbSubSize = true; rItem.mbSubSize = true;
else else
{
if ( !(rItem.mnBits & SplitWindowItemFlags::Invisible) )
{ {
if ( (nSizeDelta > 0) || rItem.mnPixSize ) if ( (nSizeDelta > 0) || rItem.mnPixSize )
{ {
@@ -559,7 +537,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
} }
} }
} }
}
if ( rItem.mbSubSize ) if ( rItem.mbSubSize )
nCalcItems++; nCalcItems++;
@@ -574,9 +551,7 @@ static void ImplCalcSet( ImplSplitSet* pSet,
nMins = 0; nMins = 0;
for ( auto& rItem : rItems ) for ( auto& rItem : rItems )
{ {
if ( rItem.mnBits & SplitWindowItemFlags::Invisible ) if ( rItem.mbSubSize )
nMins++;
else if ( rItem.mbSubSize )
{ {
long* pSize = &(rItem.mnPixSize); long* pSize = &(rItem.mnPixSize);
long nTempErr; long nTempErr;
@@ -644,10 +619,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
rItems[i].mnOldWidth = rItems[i].mnWidth; rItems[i].mnOldWidth = rItems[i].mnWidth;
rItems[i].mnOldHeight = rItems[i].mnHeight; rItems[i].mnOldHeight = rItems[i].mnHeight;
if ( rItems[i].mnBits & SplitWindowItemFlags::Invisible )
bEmpty = true;
else
{
bEmpty = false; bEmpty = false;
if ( bDown ) if ( bDown )
{ {
@@ -660,7 +631,6 @@ static void ImplCalcSet( ImplSplitSet* pSet,
if ( nPos < nMaxPos ) if ( nPos < nMaxPos )
bEmpty = true; bEmpty = true;
} }
}
if ( bEmpty ) if ( bEmpty )
{ {
@@ -705,14 +675,11 @@ static void ImplCalcSet( ImplSplitSet* pSet,
} }
} }
if ( !(rItems[i].mnBits & SplitWindowItemFlags::Invisible) )
{
if ( !bDown ) if ( !bDown )
nPos -= pSet->mnSplitSize; nPos -= pSet->mnSplitSize;
else else
nPos += rItems[i].mnPixSize+pSet->mnSplitSize; nPos += rItems[i].mnPixSize+pSet->mnSplitSize;
} }
}
// calculate Sub-Set's // calculate Sub-Set's
for ( auto& rItem : rItems ) for ( auto& rItem : rItems )