diff options
author | Noel Grandin <noel@peralex.com> | 2016-06-27 15:10:26 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2016-06-28 08:35:03 +0000 |
commit | 0f17e343306fd457c542067c09242ddd04ca39e9 (patch) | |
tree | 98a114e5d9385a2f44c4e662d6b67a91dffc04fb /chart2/source/controller/main/ObjectHierarchy.cxx | |
parent | db11f792018f6905733323a4979c6a5e697807ca (diff) |
loplugin:singlevalfields in chart2
Change-Id: I8b8bf093736a191c90d765dce82b00e15bb398d6
Reviewed-on: https://gerrit.libreoffice.org/26708
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'chart2/source/controller/main/ObjectHierarchy.cxx')
-rw-r--r-- | chart2/source/controller/main/ObjectHierarchy.cxx | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/chart2/source/controller/main/ObjectHierarchy.cxx b/chart2/source/controller/main/ObjectHierarchy.cxx index 6b58b1235131..5d86a04b3ad6 100644 --- a/chart2/source/controller/main/ObjectHierarchy.cxx +++ b/chart2/source/controller/main/ObjectHierarchy.cxx @@ -697,8 +697,7 @@ ObjectKeyNavigation::ObjectKeyNavigation( ExplicitValueProvider * pExplicitValueProvider /* = 0 */ ) : m_aCurrentOID( rCurrentOID ), m_xChartDocument( xChartDocument ), - m_pExplicitValueProvider( pExplicitValueProvider ), - m_bStepDownInDiagram( true ) + m_pExplicitValueProvider( pExplicitValueProvider ) { if ( !m_aCurrentOID.isValid() ) { @@ -749,7 +748,7 @@ void ObjectKeyNavigation::setCurrentSelection( const ObjectHierarchy::tOID& rOID bool ObjectKeyNavigation::first() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); ObjectHierarchy::tChildContainer aSiblings( aHierarchy.getSiblings( getCurrentSelection() ) ); bool bResult = !aSiblings.empty(); if( bResult ) @@ -761,7 +760,7 @@ bool ObjectKeyNavigation::first() bool ObjectKeyNavigation::last() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); ObjectHierarchy::tChildContainer aSiblings( aHierarchy.getSiblings( getCurrentSelection() ) ); bool bResult = !aSiblings.empty(); if( bResult ) @@ -773,7 +772,7 @@ bool ObjectKeyNavigation::last() bool ObjectKeyNavigation::next() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); ObjectHierarchy::tChildContainer aSiblings( aHierarchy.getSiblings( getCurrentSelection() ) ); bool bResult = !aSiblings.empty(); if( bResult ) @@ -793,7 +792,7 @@ bool ObjectKeyNavigation::next() bool ObjectKeyNavigation::previous() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); ObjectHierarchy::tChildContainer aSiblings( aHierarchy.getSiblings( getCurrentSelection())); bool bResult = !aSiblings.empty(); if( bResult ) @@ -813,7 +812,7 @@ bool ObjectKeyNavigation::previous() bool ObjectKeyNavigation::up() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); bool bResult = !ObjectHierarchy::isRootNode( getCurrentSelection()); if( bResult ) setCurrentSelection( aHierarchy.getParent( getCurrentSelection())); @@ -822,7 +821,7 @@ bool ObjectKeyNavigation::up() bool ObjectKeyNavigation::down() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); bool bResult = aHierarchy.hasChildren( getCurrentSelection()); if( bResult ) { @@ -835,7 +834,7 @@ bool ObjectKeyNavigation::down() bool ObjectKeyNavigation::veryFirst() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); ObjectHierarchy::tChildContainer aChildren( aHierarchy.getTopLevelChildren()); bool bResult = !aChildren.empty(); if( bResult ) @@ -845,7 +844,7 @@ bool ObjectKeyNavigation::veryFirst() bool ObjectKeyNavigation::veryLast() { - ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, m_bStepDownInDiagram ); + ObjectHierarchy aHierarchy( m_xChartDocument, m_pExplicitValueProvider, true/*StepDownInDiagram*/ ); ObjectHierarchy::tChildContainer aChildren( aHierarchy.getTopLevelChildren()); bool bResult = !aChildren.empty(); if( bResult ) |