diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-03-08 15:41:42 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-03-09 07:28:34 +0100 |
commit | 32c288d862d98aac2f726360be9f2ca53db06ccf (patch) | |
tree | 7ff3d9d9aa7ccb49a4dc8f790ec5750fc23eb953 /svtools | |
parent | 313744d4f23f242f5ee979d229fe394211ff7ffd (diff) |
loplugin:unusedfields
Change-Id: Ifb2c9a2d1d7dcc0ed3e8458c1a13933ccababd4c
Reviewed-on: https://gerrit.libreoffice.org/68939
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svtools')
-rw-r--r-- | svtools/source/dialogs/addresstemplate.cxx | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/svtools/source/dialogs/addresstemplate.cxx b/svtools/source/dialogs/addresstemplate.cxx index 56d90626f25b..d297231d65df 100644 --- a/svtools/source/dialogs/addresstemplate.cxx +++ b/svtools/source/dialogs/addresstemplate.cxx @@ -433,8 +433,6 @@ void AssignmentPersistentData::ImplCommit() m_xTransientDataSource; /// current scroll pos in the field list sal_Int32 nFieldScrollPos; - /// the index within m_pFields of the last visible list box. This is redundant, it could be extracted from other members - sal_Int32 nLastVisibleListIndex; /// indicates that we've an odd field number. This member is for efficiency only, it's redundant. bool bOddFieldNumber : 1; /// indicates that we're working with the real persistent configuration @@ -454,7 +452,6 @@ void AssignmentPersistentData::ImplCommit() :pFieldLabels{{nullptr}} ,pFields{{nullptr}} ,nFieldScrollPos(0) - ,nLastVisibleListIndex(0) ,bOddFieldNumber(false) ,bWorkingPersistent( true ) ,pConfigData( new AssignmentPersistentData ) @@ -467,7 +464,6 @@ void AssignmentPersistentData::ImplCommit() ,pFields{{nullptr}} ,m_xTransientDataSource( _rxTransientDS ) ,nFieldScrollPos(0) - ,nLastVisibleListIndex(0) ,bOddFieldNumber(false) ,bWorkingPersistent( false ) ,pConfigData( new AssigmentTransientData( _rDataSourceName, _rTableName, _rFields ) ) @@ -954,7 +950,6 @@ void AssignmentPersistentData::ImplCommit() auto pLeftAssignment = m_pImpl->aFieldAssignments.cbegin() + 2 * _nPos; auto pRightAssignment = pLeftAssignment + 1; - m_pImpl->nLastVisibleListIndex = -1; // loop for (sal_Int32 i=0; i<FIELD_PAIRS_VISIBLE; ++i) { @@ -986,11 +981,6 @@ void AssignmentPersistentData::ImplCommit() implSelectField(pLeftListControl->get(), *pLeftAssignment); implSelectField(pRightListControl->get(), *pRightAssignment); - // the index of the last visible list box - ++m_pImpl->nLastVisibleListIndex; // the left hand side box is always visible - if (!bHideRightColumn) - ++m_pImpl->nLastVisibleListIndex; - // increment ... if ( i < FIELD_PAIRS_VISIBLE - 1 ) { // (not in the very last round, here the +=2 could result in an invalid |