summaryrefslogtreecommitdiff
path: root/vcl
diff options
context:
space:
mode:
authorNoel Grandin <noel@peralex.com>2015-11-25 08:42:22 +0200
committerNoel Grandin <noel@peralex.com>2015-11-25 13:57:29 +0200
commit1522d4b36aa4a8c6f61788b897c24ddfb065185b (patch)
treea73772f50a4620c6fdb02a65f16794ba88592cc0 /vcl
parenta0d112a53023758a28d180ffd12766b4d325bf52 (diff)
loplugin:unusedfields in include/vcl
and fixed bug in sepia filter, where it was using the wrong member of the union to get the percentage Change-Id: I56b76496a3ac711adec12dd8c08b28d69644d66f
Diffstat (limited to 'vcl')
-rw-r--r--vcl/source/gdi/animate.cxx1
-rw-r--r--vcl/source/gdi/bitmap4.cxx6
-rw-r--r--vcl/source/window/builder.cxx2
-rw-r--r--vcl/source/window/dockmgr.cxx2
4 files changed, 3 insertions, 8 deletions
diff --git a/vcl/source/gdi/animate.cxx b/vcl/source/gdi/animate.cxx
index 911e0ca0a051..d6d9ea1071c7 100644
--- a/vcl/source/gdi/animate.cxx
+++ b/vcl/source/gdi/animate.cxx
@@ -802,7 +802,6 @@ SvStream& ReadAnimation( SvStream& rIStm, Animation& rAnimation )
AInfo::AInfo() : pOutDev( nullptr ),
pViewData( nullptr ),
nExtraData( 0L ),
- bWithSize( false ),
bPause( false ) {}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/vcl/source/gdi/bitmap4.cxx b/vcl/source/gdi/bitmap4.cxx
index 3d56db313825..2b5723cc1af7 100644
--- a/vcl/source/gdi/bitmap4.cxx
+++ b/vcl/source/gdi/bitmap4.cxx
@@ -678,16 +678,14 @@ bool Bitmap::ImplSepia( const BmpFilterParam* pFilterParam )
if( pReadAcc )
{
long nSepiaPercent = ( pFilterParam && pFilterParam->meFilter == BMP_FILTER_SEPIA ) ?
- pFilterParam->mcSolarGreyThreshold : 10;
+ pFilterParam->mnSepiaPercent : 10;
const long nSepia = 10000 - 100 * SAL_BOUND( nSepiaPercent, 0, 100 );
BitmapPalette aSepiaPal( 256 );
- DBG_ASSERT( nSepiaPercent <= 100, "Bitmap::ImplSepia(): sepia value out of range; defaulting to 100%" );
-
for( sal_uInt16 i = 0; i < 256; i++ )
{
BitmapColor& rCol = aSepiaPal[ i ];
- const sal_uInt8 cSepiaValue = (sal_uInt8) ( ( nSepia * i ) / 10000 );
+ const sal_uInt8 cSepiaValue = (sal_uInt8) ( nSepia * i / 10000 );
rCol.SetRed( (sal_uInt8) i );
rCol.SetGreen( cSepiaValue );
diff --git a/vcl/source/window/builder.cxx b/vcl/source/window/builder.cxx
index 11efd79cc089..2ee7ab9dc1ac 100644
--- a/vcl/source/window/builder.cxx
+++ b/vcl/source/window/builder.cxx
@@ -2572,7 +2572,7 @@ void VclBuilder::handleMenuObject(PopupMenu *pParent, xmlreader::XmlReader &read
void VclBuilder::handleSizeGroup(xmlreader::XmlReader &reader, const OString &rID)
{
- m_pParserState->m_aSizeGroups.push_back(SizeGroup(rID));
+ m_pParserState->m_aSizeGroups.push_back(SizeGroup());
SizeGroup &rSizeGroup = m_pParserState->m_aSizeGroups.back();
int nLevel = 1;
diff --git a/vcl/source/window/dockmgr.cxx b/vcl/source/window/dockmgr.cxx
index 20950c6f23e6..305878e74a53 100644
--- a/vcl/source/window/dockmgr.cxx
+++ b/vcl/source/window/dockmgr.cxx
@@ -811,10 +811,8 @@ ImplDockingWindowWrapper::ImplDockingWindowWrapper( const vcl::Window *pWindow )
, mbFloatPrevented(false)
, mbDockable(true)
, mbDocking(false)
- , mbDragFull(false)
, mbLastFloatMode(false)
, mbStartFloat(false)
- , mbTrackDock(false)
, mbPinned(false)
, mbRollUp(false)
, mbDockBtn(false)