diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-08-15 15:01:55 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-08-15 20:23:50 +0200 |
commit | 2dfb192edfd1ab10e9d39e265e26ac6db93ac755 (patch) | |
tree | 2b8faf5b57cfeb9ceb636783406caf64694a0ff9 /toolkit/source/controls/controlmodelcontainerbase.cxx | |
parent | 3c6cb83b80b502975dce89c02401a064872b5ea3 (diff) |
loplugin:sequenceloop in test..toolkit
Change-Id: Ic8dad06c535b0af713bfe7cd46e601c8ea7ba6c7
Reviewed-on: https://gerrit.libreoffice.org/77531
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'toolkit/source/controls/controlmodelcontainerbase.cxx')
-rw-r--r-- | toolkit/source/controls/controlmodelcontainerbase.cxx | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/toolkit/source/controls/controlmodelcontainerbase.cxx b/toolkit/source/controls/controlmodelcontainerbase.cxx index e08d8c24e3e8..f81468526b13 100644 --- a/toolkit/source/controls/controlmodelcontainerbase.cxx +++ b/toolkit/source/controls/controlmodelcontainerbase.cxx @@ -914,7 +914,7 @@ void ControlModelContainerBase::implUpdateGroupStructure() maGroups.clear(); - Sequence< Reference< XControlModel > > aControlModels = getControlModels(); + const Sequence< Reference< XControlModel > > aControlModels = getControlModels(); // in extreme we have as much groups as controls maGroups.reserve( aControlModels.getLength() ); @@ -1436,7 +1436,7 @@ sal_Bool ControlContainerBase::setModel( const Reference< XControlModel >& rxMod if ( getModel().is() ) { - Sequence< Reference< XControl > > aControls = getControls(); + const Sequence< Reference< XControl > > aControls = getControls(); for ( const Reference< XControl >& rCtrl : aControls ) removeControl( rCtrl ); @@ -1460,7 +1460,7 @@ sal_Bool ControlContainerBase::setModel( const Reference< XControlModel >& rxMod Reference< XNameAccess > xNA( getModel(), UNO_QUERY ); if ( xNA.is() ) { - Sequence< OUString > aNames = xNA->getElementNames(); + const Sequence< OUString > aNames = xNA->getElementNames(); Reference< XControlModel > xCtrlModel; for( const OUString& rName : aNames ) @@ -1818,7 +1818,7 @@ ControlModelContainerBase::updateUserFormChildren( const Reference< XNameContain // global list of containers if ( xProps.is() ) xProps->setPropertyValue( GetPropertyName( BASEPROPERTY_USERFORMCONTAINEES ), uno::makeAny( uno::Reference< XNameContainer >() ) ); - Sequence< OUString > aChildNames = xChildContainer->getElementNames(); + const Sequence< OUString > aChildNames = xChildContainer->getElementNames(); for ( const auto& rName : aChildNames ) updateUserFormChildren( xAllChildren, rName, Operation, Reference< XControlModel > () ); } @@ -1834,7 +1834,7 @@ ControlModelContainerBase::updateUserFormChildren( const Reference< XNameContain Reference< XPropertySet > xProps( xChildContainer, UNO_QUERY ); if ( xProps.is() ) xProps->setPropertyValue( GetPropertyName( BASEPROPERTY_USERFORMCONTAINEES ), uno::makeAny( xAllChildren ) ); - Sequence< OUString > aChildNames = xChildContainer->getElementNames(); + const Sequence< OUString > aChildNames = xChildContainer->getElementNames(); for ( const auto& rName : aChildNames ) { Reference< XControlModel > xChildTarget( xChildContainer->getByName( rName ), UNO_QUERY ); |