From 17a2c9e8e2361de27013a25e51f3a3ca729f1b31 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Mon, 5 Jun 2017 08:34:22 +0200 Subject: clang-tidy performance-unnecessary-value-param Change-Id: I6ed8c54c7c45931d91709cc818f2483c70197192 Reviewed-on: https://gerrit.libreoffice.org/38400 Tested-by: Jenkins Reviewed-by: Noel Grandin --- avmedia/inc/mediacontrol.hxx | 2 +- avmedia/source/framework/MediaControlBase.cxx | 8 ++++---- avmedia/source/framework/mediacontrol.cxx | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) (limited to 'avmedia') diff --git a/avmedia/inc/mediacontrol.hxx b/avmedia/inc/mediacontrol.hxx index a6c247b7c687..99ad5ef91a01 100644 --- a/avmedia/inc/mediacontrol.hxx +++ b/avmedia/inc/mediacontrol.hxx @@ -51,7 +51,7 @@ public: const Size& getMinSizePixel() const; void setState( const MediaItem& rItem ); - void UpdateURLField(MediaItem maItem); + void UpdateURLField( MediaItem const & maItem ); protected: diff --git a/avmedia/source/framework/MediaControlBase.cxx b/avmedia/source/framework/MediaControlBase.cxx index 2aa829dc735f..390f08ff6dbc 100644 --- a/avmedia/source/framework/MediaControlBase.cxx +++ b/avmedia/source/framework/MediaControlBase.cxx @@ -39,7 +39,7 @@ MediaControlBase::MediaControlBase() { } -void MediaControlBase::UpdateTimeField( MediaItem aMediaItem, double fTime ) +void MediaControlBase::UpdateTimeField( MediaItem const & aMediaItem, double fTime ) { if( !aMediaItem.getURL().isEmpty()) { @@ -57,7 +57,7 @@ void MediaControlBase::UpdateTimeField( MediaItem aMediaItem, double fTime ) } } -void MediaControlBase::UpdateVolumeSlider( MediaItem aMediaItem ) +void MediaControlBase::UpdateVolumeSlider( MediaItem const & aMediaItem ) { if( aMediaItem.getURL().isEmpty() ) mpVolumeSlider->Disable(); @@ -70,7 +70,7 @@ void MediaControlBase::UpdateVolumeSlider( MediaItem aMediaItem ) } } -void MediaControlBase::UpdateTimeSlider( MediaItem aMediaItem ) +void MediaControlBase::UpdateTimeSlider( MediaItem const & aMediaItem ) { if( aMediaItem.getURL().isEmpty() ) mpTimeSlider->Disable(); @@ -211,7 +211,7 @@ void MediaControlBase::UpdateToolBoxes(MediaItem aMediaItem) } } -void MediaControlBase::SelectPlayToolBoxItem( MediaItem& aExecItem, MediaItem aItem, sal_uInt16 nId) +void MediaControlBase::SelectPlayToolBoxItem( MediaItem& aExecItem, MediaItem const & aItem, sal_uInt16 nId) { switch( nId ) { diff --git a/avmedia/source/framework/mediacontrol.cxx b/avmedia/source/framework/mediacontrol.cxx index 44d9ce7ce030..bde107a71e24 100644 --- a/avmedia/source/framework/mediacontrol.cxx +++ b/avmedia/source/framework/mediacontrol.cxx @@ -168,7 +168,7 @@ const Size& MediaControl::getMinSizePixel() const return maMinSize; } -void MediaControl::UpdateURLField(MediaItem tempItem) +void MediaControl::UpdateURLField(MediaItem const & tempItem) { const OUString aURL( AvmResId(AVMEDIA_MEDIA_PATH) + ": " + tempItem.getURL() ) ; mpMediaPath->SetText(aURL); -- cgit