loplugin:constantparam in slideshow

Change-Id: If6c9b8e551d6e6dc857f634e1e8db092541c1031

6 files changed, 10 insertions, 16 deletions

diff --git a/slideshow/source/engine/screenupdater.cxx b/slideshow/source/engine/screenupdater.cxx
index 71288685ce99..781277fe5a36 100644
--- a/slideshow/source/engine/screenupdater.cxx
+++ b/slideshow/source/engine/screenupdater.cxx
@@ -212,9 +212,9 @@ namespace internal
         }
     }
 
-    ::std::shared_ptr<ScreenUpdater::UpdateLock> ScreenUpdater::createLock (const bool bStartLocked)
+    ::std::shared_ptr<ScreenUpdater::UpdateLock> ScreenUpdater::createLock()
     {
-        return ::std::shared_ptr<ScreenUpdater::UpdateLock>(new ::UpdateLock(*this, bStartLocked));
+        return ::std::shared_ptr<ScreenUpdater::UpdateLock>(new ::UpdateLock(*this, false/*bStartLocked*/));
     }
 
 
diff --git a/slideshow/source/engine/slide/shapemanagerimpl.cxx b/slideshow/source/engine/slide/shapemanagerimpl.cxx
index f9b62a8eff4b..6b655769e90d 100644
--- a/slideshow/source/engine/slide/shapemanagerimpl.cxx
+++ b/slideshow/source/engine/slide/shapemanagerimpl.cxx
@@ -50,7 +50,7 @@ ShapeManagerImpl::ShapeManagerImpl( EventMultiplexer&            rMultiplexer,
 {
 }
 
-void ShapeManagerImpl::activate( bool bSlideBackgoundPainted )
+void ShapeManagerImpl::activate()
 {
     if( !mbEnabled )
     {
@@ -72,7 +72,7 @@ void ShapeManagerImpl::activate( bool bSlideBackgoundPainted )
             this->cursorChanged( rListener.first, rListener.second );
 
         if( mpLayerManager )
-            mpLayerManager->activate( bSlideBackgoundPainted );
+            mpLayerManager->activate( true/*bSlideBackgoundPainted*/ );
     }
 }
 
diff --git a/slideshow/source/engine/slide/shapemanagerimpl.hxx b/slideshow/source/engine/slide/shapemanagerimpl.hxx
index 01ea7745557f..63dc1c79254e 100644
--- a/slideshow/source/engine/slide/shapemanagerimpl.hxx
+++ b/slideshow/source/engine/slide/shapemanagerimpl.hxx
@@ -71,13 +71,11 @@ public:
 
     /** Enables event listening.
 
-        @param bSlideBackgoundPainted
-        When true, the initial slide content on the background layer
+        The initial slide content on the background layer
         is already rendered (e.g. from a previous slide
-        transition). When false, slide renders initial content of
-        slide.
+        transition).
      */
-    void activate( bool bSlideBackgoundPainted );
+    void activate();
 
     /** Disables event listening.
      */
diff --git a/slideshow/source/engine/slide/slideimpl.cxx b/slideshow/source/engine/slide/slideimpl.cxx
index 863b6d2efc0e..ab9aa833271e 100644
--- a/slideshow/source/engine/slide/slideimpl.cxx
+++ b/slideshow/source/engine/slide/slideimpl.cxx
@@ -432,7 +432,7 @@ void SlideImpl::show( bool bSlideBackgoundPainted )
     // slide. Also enables LayerManager to record updates. Currently,
     // never let LayerManager render initial slide content, use
     // buffered slide bitmaps instead.
-    mpShapeManager->activate( true );
+    mpShapeManager->activate();
 
 
     // render slide to screen, if requested
diff --git a/slideshow/source/engine/slideshowimpl.cxx b/slideshow/source/engine/slideshowimpl.cxx
index da94f03319b3..f826b4458f7a 100644
--- a/slideshow/source/engine/slideshowimpl.cxx
+++ b/slideshow/source/engine/slideshowimpl.cxx
@@ -1240,7 +1240,7 @@ sal_Bool SlideShowImpl::previousEffect() throw (uno::RuntimeException, std::exce
     else
     {
         return maEffectRewinder.rewind(
-            maScreenUpdater.createLock(false),
+            maScreenUpdater.createLock(),
             [this]() { return this->redisplayCurrentSlide(); },
             [this]() { return this->rewindEffectToPreviousSlide(); } );
     }
diff --git a/slideshow/source/inc/screenupdater.hxx b/slideshow/source/inc/screenupdater.hxx
index 384e676c6f50..e975f3a0b0ac 100644
--- a/slideshow/source/inc/screenupdater.hxx
+++ b/slideshow/source/inc/screenupdater.hxx
@@ -110,12 +110,8 @@ namespace slideshow
 
             /** Call this method to create a lock instead of calling
                 lockUpdates() and unlockUpdates() directly.
-                @param bStartLocked
-                    When <TRUE/> then the UpdateLock is created already
-                    locked. When <FALSE/> then Activate() has to be called in order
-                    to lock the lock.
             */
-            ::std::shared_ptr<UpdateLock> createLock (const bool bStartLocked);
+            ::std::shared_ptr<UpdateLock> createLock();
 
             /** Lock updates to prevent intermediate repaints.
             */