diff options
| author | Noel Grandin <noel@peralex.com> | 2016-04-11 14:09:58 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2016-04-11 15:12:10 +0200 |
| commit | 28c96a09105f87d76441234a6e77f2ac1e1473df (patch) | |
| tree | 6a1391cbcc3ab5a7f49379d292da0f81fc1cba0c | |
| parent | e9cded40c8727cec7d0d29219c14ef0d0eef5195 (diff) | |
clang-tidy performance-unnecessary-value-param in toolkit
Change-Id: Ic08723ef15f2bc31031e3887b9916ecc7e2205da
| -rw-r--r-- | include/toolkit/controls/roadmapcontrol.hxx | 8 | ||||
| -rw-r--r-- | toolkit/source/controls/roadmapcontrol.cxx | 8 | ||||
| -rw-r--r-- | toolkit/source/controls/unocontrolcontainer.cxx | 2 |
3 files changed, 9 insertions, 9 deletions
diff --git a/include/toolkit/controls/roadmapcontrol.hxx b/include/toolkit/controls/roadmapcontrol.hxx index f948dc2e08c7..6b97158a9e21 100644 --- a/include/toolkit/controls/roadmapcontrol.hxx +++ b/include/toolkit/controls/roadmapcontrol.hxx @@ -79,10 +79,10 @@ namespace toolkit ContainerListenerMultiplexer maContainerListeners; RoadmapItemHolderList maRoadmapItems; - void MakeRMItemValidation( sal_Int32 Index, css::uno::Reference< XInterface > xRoadmapItem ); - css::container::ContainerEvent GetContainerEvent(sal_Int32 Index, css::uno::Reference< XInterface > ); - void SetRMItemDefaultProperties( const sal_Int32 Index, css::uno::Reference< XInterface > ); - static sal_Int16 GetCurrentItemID( css::uno::Reference< css::beans::XPropertySet > xPropertySet ); + void MakeRMItemValidation( sal_Int32 Index, const css::uno::Reference< XInterface >& xRoadmapItem ); + css::container::ContainerEvent GetContainerEvent(sal_Int32 Index, const css::uno::Reference< XInterface >& ); + void SetRMItemDefaultProperties( const sal_Int32 Index, const css::uno::Reference< XInterface >& ); + static sal_Int16 GetCurrentItemID( const css::uno::Reference< css::beans::XPropertySet >& xPropertySet ); sal_Int32 GetUniqueID(); diff --git a/toolkit/source/controls/roadmapcontrol.cxx b/toolkit/source/controls/roadmapcontrol.cxx index 83476bd46872..6febbff43d34 100644 --- a/toolkit/source/controls/roadmapcontrol.cxx +++ b/toolkit/source/controls/roadmapcontrol.cxx @@ -192,7 +192,7 @@ static void lcl_throwIndexOutOfBoundsException( ) } - void UnoControlRoadmapModel::MakeRMItemValidation( sal_Int32 Index, Reference< XInterface > xRoadmapItem ) + void UnoControlRoadmapModel::MakeRMItemValidation( sal_Int32 Index, const Reference< XInterface >& xRoadmapItem ) { if ((Index > (sal_Int32)maRoadmapItems.size()) || ( Index < 0 ) ) lcl_throwIndexOutOfBoundsException( ); @@ -205,7 +205,7 @@ static void lcl_throwIndexOutOfBoundsException( ) } - void UnoControlRoadmapModel::SetRMItemDefaultProperties( const sal_Int32 , Reference< XInterface > xRoadmapItem) + void UnoControlRoadmapModel::SetRMItemDefaultProperties( const sal_Int32 , const Reference< XInterface >& xRoadmapItem) { Any aAny; Reference< XPropertySet > xPropertySet( xRoadmapItem, UNO_QUERY ); @@ -255,7 +255,7 @@ static void lcl_throwIndexOutOfBoundsException( ) } - ContainerEvent UnoControlRoadmapModel::GetContainerEvent(sal_Int32 Index, Reference< XInterface > xRoadmapItem) + ContainerEvent UnoControlRoadmapModel::GetContainerEvent(sal_Int32 Index, const Reference< XInterface >& xRoadmapItem) { ContainerEvent aEvent; aEvent.Source = *this; @@ -265,7 +265,7 @@ static void lcl_throwIndexOutOfBoundsException( ) } - sal_Int16 UnoControlRoadmapModel::GetCurrentItemID( Reference< XPropertySet > xPropertySet ) + sal_Int16 UnoControlRoadmapModel::GetCurrentItemID( const Reference< XPropertySet >& xPropertySet ) { Any aAny = xPropertySet->getPropertyValue( GetPropertyName( BASEPROPERTY_CURRENTITEMID ) ); sal_Int16 n_CurrentItemID = 0; diff --git a/toolkit/source/controls/unocontrolcontainer.cxx b/toolkit/source/controls/unocontrolcontainer.cxx index aec738fe0980..5bfd89ac1582 100644 --- a/toolkit/source/controls/unocontrolcontainer.cxx +++ b/toolkit/source/controls/unocontrolcontainer.cxx @@ -308,7 +308,7 @@ OUString UnoControlHolderList::impl_getFreeName_throw() void implUpdateVisibility ( sal_Int32 nDialogStep, - uno::Reference< awt::XControlContainer > xControlContainer + const uno::Reference< awt::XControlContainer >& xControlContainer ) { uno::Sequence< uno::Reference< awt::XControl > > |