diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-21 13:13:39 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-22 14:07:49 +0200 |
commit | 2b4cb88c029cc085e06effb32a2631033ccd8a50 (patch) | |
tree | 26c84dc341b233bb94c90d0fbc01788ed62a0bfa | |
parent | 939f9bd0b502ab5dc240be0c6f334becb1c944c8 (diff) |
loplugin:unusedfields in basctl..connectivity
Change-Id: I2f10daadb84e48eaf96f6cc63899b2f4fce7326e
Reviewed-on: https://gerrit.libreoffice.org/40295
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
-rw-r--r-- | basctl/source/basicide/bastype3.hxx | 1 | ||||
-rw-r--r-- | basctl/source/basicide/bastypes.cxx | 1 | ||||
-rw-r--r-- | chart2/source/view/charttypes/BubbleChart.cxx | 8 | ||||
-rw-r--r-- | chart2/source/view/charttypes/BubbleChart.hxx | 1 | ||||
-rw-r--r-- | comphelper/source/misc/accimplaccess.cxx | 6 | ||||
-rw-r--r-- | compilerplugins/clang/unusedfields.readonly.results | 6 | ||||
-rw-r--r-- | connectivity/source/drivers/postgresql/pq_connection.hxx | 2 | ||||
-rw-r--r-- | connectivity/source/drivers/postgresql/pq_databasemetadata.cxx | 2 | ||||
-rw-r--r-- | include/comphelper/accimplaccess.hxx | 6 | ||||
-rw-r--r-- | toolkit/source/controls/accessiblecontrolcontext.cxx | 2 |
10 files changed, 3 insertions, 32 deletions
diff --git a/basctl/source/basicide/bastype3.hxx b/basctl/source/basicide/bastype3.hxx index 580da6a96595..9305685cdb35 100644 --- a/basctl/source/basicide/bastype3.hxx +++ b/basctl/source/basicide/bastype3.hxx @@ -31,7 +31,6 @@ class ExtendedEdit : public Edit private: Accelerator aAcc; Link<Accelerator&,void> aAccHdl; - Link<ExtendedEdit*,void> aLoseFocusHdl; protected: DECL_LINK( EditAccHdl, Accelerator&, void ); diff --git a/basctl/source/basicide/bastypes.cxx b/basctl/source/basicide/bastypes.cxx index 4873d8663b34..91515535a257 100644 --- a/basctl/source/basicide/bastypes.cxx +++ b/basctl/source/basicide/bastypes.cxx @@ -423,7 +423,6 @@ ExtendedEdit::ExtendedEdit(vcl::Window* pParent, WinBits nStyle) IMPL_LINK_NOARG(ExtendedEdit, ImplGetFocusHdl, Control&, void) { Application::InsertAccel( &aAcc ); - aLoseFocusHdl.Call( this ); } IMPL_LINK_NOARG(ExtendedEdit, ImplLoseFocusHdl, Control&, void) diff --git a/chart2/source/view/charttypes/BubbleChart.cxx b/chart2/source/view/charttypes/BubbleChart.cxx index e78738a0e85e..19c278fff217 100644 --- a/chart2/source/view/charttypes/BubbleChart.cxx +++ b/chart2/source/view/charttypes/BubbleChart.cxx @@ -46,7 +46,6 @@ using namespace ::com::sun::star::chart2; BubbleChart::BubbleChart( const uno::Reference<XChartType>& xChartTypeModel , sal_Int32 nDimensionCount ) : VSeriesPlotter( xChartTypeModel, nDimensionCount, false ) - , m_bShowNegativeValues(false) , m_fBubbleSizeScaling(1.0) , m_fMaxLogicBubbleSize( 0.0 ) , m_fBubbleSizeFactorToScreen( 1.0 ) @@ -89,8 +88,6 @@ void BubbleChart::calculateMaximumLogicBubbleSize() continue; double fSize = pSeries->getBubble_Size( nIndex ); - if( m_bShowNegativeValues ) - fSize = fabs(fSize); if( fSize > fMaxSize ) fMaxSize = fSize; } @@ -124,9 +121,6 @@ drawing::Direction3D BubbleChart::transformToScreenBubbleSize( double fLogicSize if( ::rtl::math::isNan(fLogicSize) || ::rtl::math::isInf(fLogicSize) ) return aRet; - if( m_bShowNegativeValues ) - fLogicSize = fabs(fLogicSize); - double fMaxSize = m_fMaxLogicBubbleSize; double fMaxRadius = sqrt( fMaxSize / F_PI ); @@ -251,7 +245,7 @@ void BubbleChart::createShapes() double fLogicY = pSeries->getYValue(nIndex); double fBubbleSize = pSeries->getBubble_Size( nIndex ); - if( !m_bShowNegativeValues && fBubbleSize<0.0 ) + if( fBubbleSize<0.0 ) continue; if( fBubbleSize == 0.0 || ::rtl::math::isNan(fBubbleSize) ) diff --git a/chart2/source/view/charttypes/BubbleChart.hxx b/chart2/source/view/charttypes/BubbleChart.hxx index 6958e54d87ff..80cf1a4be282 100644 --- a/chart2/source/view/charttypes/BubbleChart.hxx +++ b/chart2/source/view/charttypes/BubbleChart.hxx @@ -53,7 +53,6 @@ private: //methods private: //member - bool m_bShowNegativeValues;//input parameter double m_fBubbleSizeScaling;//input parameter double m_fMaxLogicBubbleSize;//calculated values diff --git a/comphelper/source/misc/accimplaccess.cxx b/comphelper/source/misc/accimplaccess.cxx index 96fdbd96f381..9593b787d3a6 100644 --- a/comphelper/source/misc/accimplaccess.cxx +++ b/comphelper/source/misc/accimplaccess.cxx @@ -38,7 +38,6 @@ namespace comphelper struct OAccImpl_Impl { Reference< XAccessible > m_xAccParent; - sal_Int64 m_nForeignControlledStates; }; OAccessibleImplementationAccess::OAccessibleImplementationAccess( ) @@ -58,11 +57,6 @@ namespace comphelper } - sal_Int64 OAccessibleImplementationAccess::implGetForeignControlledStates( ) const - { - return m_pImpl->m_nForeignControlledStates; - } - const Sequence< sal_Int8 > OAccessibleImplementationAccess::getUnoTunnelImplementationId() { static cppu::OImplementationId implID; diff --git a/compilerplugins/clang/unusedfields.readonly.results b/compilerplugins/clang/unusedfields.readonly.results index fafa2d3a0c7d..f13d83aece78 100644 --- a/compilerplugins/clang/unusedfields.readonly.results +++ b/compilerplugins/clang/unusedfields.readonly.results @@ -1,5 +1,3 @@ -basctl/source/basicide/bastype3.hxx:34 - basctl::ExtendedEdit aLoseFocusHdl Link<class basctl::ExtendedEdit *, void> basegfx/source/polygon/b2dtrapezoid.cxx:201 basegfx::trapezoidhelper::PointBlockAllocator maFirstStackBlock class basegfx::B2DPoint [32] basic/qa/cppunit/basictest.hxx:27 @@ -56,8 +54,6 @@ canvas/source/vcl/impltools.hxx:117 vclcanvas::tools::LocalGuard aSolarGuard class SolarMutexGuard chart2/source/model/main/DataPoint.hxx:108 chart::DataPoint m_bNoParentPropAllowed _Bool -chart2/source/view/charttypes/BubbleChart.hxx:56 - chart::BubbleChart m_bShowNegativeValues _Bool chart2/source/view/inc/GL3DRenderer.hxx:54 chart::opengl3D::MaterialParameters pad float chart2/source/view/inc/GL3DRenderer.hxx:55 @@ -68,8 +64,6 @@ chart2/source/view/inc/GL3DRenderer.hxx:65 chart::opengl3D::LightSource pad2 float chart2/source/view/inc/GL3DRenderer.hxx:66 chart::opengl3D::LightSource pad3 float -comphelper/source/misc/accimplaccess.cxx:41 - comphelper::OAccImpl_Impl m_nForeignControlledStates sal_Int64 connectivity/source/drivers/evoab2/EApi.h:122 (anonymous) address_format char * connectivity/source/drivers/evoab2/EApi.h:125 diff --git a/connectivity/source/drivers/postgresql/pq_connection.hxx b/connectivity/source/drivers/postgresql/pq_connection.hxx index bebabb04be82..c49cc8938454 100644 --- a/connectivity/source/drivers/postgresql/pq_connection.hxx +++ b/connectivity/source/drivers/postgresql/pq_connection.hxx @@ -97,7 +97,6 @@ struct ConnectionSettings maxIndexKeys(0), pTablesImpl(nullptr), pViewsImpl(nullptr), - showSystemColumns( false ), logFile( nullptr ), m_nLogLevel(LogLevel::Info) {} @@ -113,7 +112,6 @@ struct ConnectionSettings Views *pViewsImpl; // needed to implement renaming of tables / views OUString user; OUString catalog; - bool showSystemColumns; FILE *logFile; LogLevel m_nLogLevel; }; diff --git a/connectivity/source/drivers/postgresql/pq_databasemetadata.cxx b/connectivity/source/drivers/postgresql/pq_databasemetadata.cxx index 1e0443e651f4..7c440d7eb334 100644 --- a/connectivity/source/drivers/postgresql/pq_databasemetadata.cxx +++ b/connectivity/source/drivers/postgresql/pq_databasemetadata.cxx @@ -1563,7 +1563,7 @@ css::uno::Reference< XResultSet > DatabaseMetaData::getColumns( while( rs->next() ) { - if( m_pSettings->showSystemColumns || ! isSystemColumn( xRow->getShort( 12 ) ) ) + if( ! isSystemColumn( xRow->getShort( 12 ) ) ) { OUString sNewSchema( xRow->getString(1) ); OUString sNewTable( xRow->getString(2) ); diff --git a/include/comphelper/accimplaccess.hxx b/include/comphelper/accimplaccess.hxx index 6d5322548e13..943c12b864c1 100644 --- a/include/comphelper/accimplaccess.hxx +++ b/include/comphelper/accimplaccess.hxx @@ -70,12 +70,6 @@ namespace comphelper const css::uno::Reference< css::accessibility::XAccessible >& implGetForeignControlledParent( ) const; - /** retrieves the set of currently set states which are controlled by a foreign instance - @return - a bit mask, where a set bit 2^n means that the AccessibleStateType n has been set - */ - sal_Int64 implGetForeignControlledStates( ) const; - protected: OAccessibleImplementationAccess( ); virtual ~OAccessibleImplementationAccess( ); diff --git a/toolkit/source/controls/accessiblecontrolcontext.cxx b/toolkit/source/controls/accessiblecontrolcontext.cxx index bd9d72958a8b..cde540b577d9 100644 --- a/toolkit/source/controls/accessiblecontrolcontext.cxx +++ b/toolkit/source/controls/accessiblecontrolcontext.cxx @@ -176,7 +176,7 @@ namespace toolkit if ( isAlive() ) { // no own states, only the ones which are foreign controlled - pStateSet = new ::utl::AccessibleStateSetHelper( implGetForeignControlledStates() ); + pStateSet = new ::utl::AccessibleStateSetHelper( 0 ); } else { // only the DEFUNC state if we're already disposed |