summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNoel Grandin <noel@peralex.com>2016-03-18 08:09:25 +0200
committerNoel Grandin <noel@peralex.com>2016-03-18 11:32:16 +0200
commitf4c59e5364e449bd8ce5420ff57331677ff859b9 (patch)
tree563d04a3dc7724f0597b32022e31a6ca5074aac3
parentcd6d590e2e687c3852a4e7bda9571d499b86d8d4 (diff)
loplugin:constantparam in sd
Change-Id: Idad86a005a7df1af38722fa0e909ed9b6e0858fa
-rw-r--r--sd/inc/drawdoc.hxx2
-rw-r--r--sd/inc/sdcgmfilter.hxx3
-rw-r--r--sd/inc/sdhtmlfilter.hxx3
-rw-r--r--sd/inc/sdpptwrp.hxx3
-rw-r--r--sd/inc/sdxmlwrp.hxx1
-rw-r--r--sd/source/core/drawdoc3.cxx3
-rw-r--r--sd/source/filter/cgm/sdcgmfilter.cxx4
-rw-r--r--sd/source/filter/html/sdhtmlfilter.cxx4
-rw-r--r--sd/source/filter/sdpptwrp.cxx4
-rw-r--r--sd/source/filter/xml/sdxmlwrp.cxx4
-rw-r--r--sd/source/ui/dlg/filedlg.cxx13
-rw-r--r--sd/source/ui/docshell/docshel4.cxx26
-rw-r--r--sd/source/ui/inc/ViewShell.hxx5
-rw-r--r--sd/source/ui/slidesorter/cache/SlsBitmapCache.cxx13
-rw-r--r--sd/source/ui/slidesorter/cache/SlsBitmapCache.hxx10
-rw-r--r--sd/source/ui/slidesorter/cache/SlsPageCache.cxx10
-rw-r--r--sd/source/ui/slidesorter/controller/SlideSorterController.cxx6
-rw-r--r--sd/source/ui/slidesorter/controller/SlsAnimator.cxx3
-rw-r--r--sd/source/ui/slidesorter/controller/SlsCurrentSlideManager.cxx5
-rw-r--r--sd/source/ui/slidesorter/controller/SlsDragAndDropContext.cxx5
-rw-r--r--sd/source/ui/slidesorter/controller/SlsDragAndDropContext.hxx3
-rw-r--r--sd/source/ui/slidesorter/controller/SlsVisibleAreaManager.cxx1
-rw-r--r--sd/source/ui/slidesorter/inc/cache/SlsPageCache.hxx12
-rw-r--r--sd/source/ui/slidesorter/inc/controller/SlsAnimator.hxx3
-rw-r--r--sd/source/ui/slidesorter/inc/controller/SlsCurrentSlideManager.hxx7
-rw-r--r--sd/source/ui/slidesorter/inc/model/SlsPageEnumeration.hxx13
-rw-r--r--sd/source/ui/slidesorter/model/SlsPageEnumeration.cxx17
-rw-r--r--sd/source/ui/slidesorter/view/SlsFramePainter.cxx6
-rw-r--r--sd/source/ui/slidesorter/view/SlsFramePainter.hxx5
-rw-r--r--sd/source/ui/slidesorter/view/SlsInsertAnimator.cxx1
-rw-r--r--sd/source/ui/slidesorter/view/SlsPageObjectPainter.cxx4
-rw-r--r--sd/source/ui/unoidl/SdUnoDrawView.cxx2
-rw-r--r--sd/source/ui/unoidl/SdUnoSlideView.cxx3
-rw-r--r--sd/source/ui/view/sdview2.cxx2
-rw-r--r--sd/source/ui/view/viewshe2.cxx9
-rw-r--r--sd/source/ui/view/viewshel.cxx6
36 files changed, 75 insertions, 146 deletions
diff --git a/sd/inc/drawdoc.hxx b/sd/inc/drawdoc.hxx
index af5798889d37..a2434fa5057b 100644
--- a/sd/inc/drawdoc.hxx
+++ b/sd/inc/drawdoc.hxx
@@ -264,7 +264,7 @@ public:
SAL_DLLPRIVATE void InsertBookmark(const std::vector<OUString> &rBookmarkList,
std::vector<OUString> &rExchangeList, bool bLink,
bool bReplace, sal_uInt16 nPgPos, bool bNoDialogs,
- ::sd::DrawDocShell* pBookmarkDocSh, bool bCopy,
+ ::sd::DrawDocShell* pBookmarkDocSh,
Point* pObjPos);
SAL_DLLPRIVATE bool IsStartWithPresentation() const { return mbStartWithPresentation;}
diff --git a/sd/inc/sdcgmfilter.hxx b/sd/inc/sdcgmfilter.hxx
index 7f11bbf535d7..550819b940df 100644
--- a/sd/inc/sdcgmfilter.hxx
+++ b/sd/inc/sdcgmfilter.hxx
@@ -28,8 +28,7 @@ class SdCGMFilter : public SdFilter
public:
SdCGMFilter (
SfxMedium& rMedium,
- ::sd::DrawDocShell& rDocShell,
- bool bShowProgress );
+ ::sd::DrawDocShell& rDocShell );
virtual ~SdCGMFilter();
bool Import();
diff --git a/sd/inc/sdhtmlfilter.hxx b/sd/inc/sdhtmlfilter.hxx
index e5482844bbfa..e696c8a786a3 100644
--- a/sd/inc/sdhtmlfilter.hxx
+++ b/sd/inc/sdhtmlfilter.hxx
@@ -29,8 +29,7 @@ class SdHTMLFilter
public:
SdHTMLFilter (
SfxMedium& rMedium,
- ::sd::DrawDocShell& rDocShell,
- bool bShowProgress);
+ ::sd::DrawDocShell& rDocShell);
virtual ~SdHTMLFilter();
virtual bool Export() override;
diff --git a/sd/inc/sdpptwrp.hxx b/sd/inc/sdpptwrp.hxx
index f333ced302d8..3f74e7c11b2f 100644
--- a/sd/inc/sdpptwrp.hxx
+++ b/sd/inc/sdpptwrp.hxx
@@ -28,8 +28,7 @@ class SdPPTFilter : public SdFilter
public:
SdPPTFilter (
SfxMedium& rMedium,
- ::sd::DrawDocShell& rDocShell,
- bool bShowProgress);
+ ::sd::DrawDocShell& rDocShell);
virtual ~SdPPTFilter();
/// these methods are necessary for the export to PowerPoint
diff --git a/sd/inc/sdxmlwrp.hxx b/sd/inc/sdxmlwrp.hxx
index 5aafa00c98c0..f3488f7d8c93 100644
--- a/sd/inc/sdxmlwrp.hxx
+++ b/sd/inc/sdxmlwrp.hxx
@@ -36,7 +36,6 @@ public:
SdXMLFilter(
SfxMedium& rMedium,
::sd::DrawDocShell& rDocShell,
- bool bShowProgress,
SdXMLFilterMode eFilterMode = SDXMLMODE_Normal,
sal_uLong nStoreVer = SOFFICE_FILEFORMAT_8 );
virtual ~SdXMLFilter();
diff --git a/sd/source/core/drawdoc3.cxx b/sd/source/core/drawdoc3.cxx
index f1559fae0461..58d80cdeb223 100644
--- a/sd/source/core/drawdoc3.cxx
+++ b/sd/source/core/drawdoc3.cxx
@@ -285,7 +285,6 @@ void SdDrawDocument::InsertBookmark(
sal_uInt16 nInsertPos, // Insertion position of pages
bool bNoDialogs, // Don't show dialogs
::sd::DrawDocShell* pBookmarkDocSh, // If set, this is the source document
- bool bCopy, // Copy the pages?
Point* pObjPos) // Insertion position of objects
{
bool bOK = true;
@@ -330,7 +329,7 @@ void SdDrawDocument::InsertBookmark(
{
// Insert all page bookmarks
bOK = InsertBookmarkAsPage(rBookmarkList, &rExchangeList, bLink, bReplace,
- nInsertPos, bNoDialogs, pBookmarkDocSh, bCopy, true, false);
+ nInsertPos, bNoDialogs, pBookmarkDocSh, true/*bCopy*/, true, false);
}
if ( bOK && !rBookmarkList.empty() )
diff --git a/sd/source/filter/cgm/sdcgmfilter.cxx b/sd/source/filter/cgm/sdcgmfilter.cxx
index 776d4db4d579..f6b5e1fd9cf9 100644
--- a/sd/source/filter/cgm/sdcgmfilter.cxx
+++ b/sd/source/filter/cgm/sdcgmfilter.cxx
@@ -48,8 +48,8 @@ extern "C" sal_uInt32 ImportCGM( OUString const &, Reference< XModel > const &,
#endif
-SdCGMFilter::SdCGMFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell, bool bShowProgress ) :
- SdFilter( rMedium, rDocShell, bShowProgress )
+SdCGMFilter::SdCGMFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell ) :
+ SdFilter( rMedium, rDocShell, true/*bShowProgress*/ )
{
}
diff --git a/sd/source/filter/html/sdhtmlfilter.cxx b/sd/source/filter/html/sdhtmlfilter.cxx
index c4bc08522ac9..56c74335e676 100644
--- a/sd/source/filter/html/sdhtmlfilter.cxx
+++ b/sd/source/filter/html/sdhtmlfilter.cxx
@@ -39,8 +39,8 @@
#include "sdhtmlfilter.hxx"
-SdHTMLFilter::SdHTMLFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell, bool bShowProgress ) :
- SdFilter( rMedium, rDocShell, bShowProgress )
+SdHTMLFilter::SdHTMLFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell ) :
+ SdFilter( rMedium, rDocShell, true/*bShowProgress*/ )
{
}
diff --git a/sd/source/filter/sdpptwrp.cxx b/sd/source/filter/sdpptwrp.cxx
index 98a74536cde6..6f6b2254794f 100644
--- a/sd/source/filter/sdpptwrp.cxx
+++ b/sd/source/filter/sdpptwrp.cxx
@@ -61,8 +61,8 @@ extern "C" sal_Bool SaveVBA( SfxObjectShell&, SvMemoryStream*& );
#endif
-SdPPTFilter::SdPPTFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell, bool bShowProgress ) :
- SdFilter( rMedium, rDocShell, bShowProgress ),
+SdPPTFilter::SdPPTFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell ) :
+ SdFilter( rMedium, rDocShell, true/*bShowProgress*/ ),
pBas ( nullptr )
{
}
diff --git a/sd/source/filter/xml/sdxmlwrp.cxx b/sd/source/filter/xml/sdxmlwrp.cxx
index 91ba7a6c7cb0..8416ca56f8bd 100644
--- a/sd/source/filter/xml/sdxmlwrp.cxx
+++ b/sd/source/filter/xml/sdxmlwrp.cxx
@@ -157,8 +157,8 @@ XML_SERVICES* getServices( bool bImport, bool bDraw, sal_uLong nStoreVer )
}
-SdXMLFilter::SdXMLFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell, bool bShowProgress, SdXMLFilterMode eFilterMode, sal_uLong nStoreVer ) :
- SdFilter( rMedium, rDocShell, bShowProgress ), meFilterMode( eFilterMode ), mnStoreVer( nStoreVer )
+SdXMLFilter::SdXMLFilter( SfxMedium& rMedium, ::sd::DrawDocShell& rDocShell, SdXMLFilterMode eFilterMode, sal_uLong nStoreVer ) :
+ SdFilter( rMedium, rDocShell, true/*bShowProgress*/ ), meFilterMode( eFilterMode ), mnStoreVer( nStoreVer )
{
}
diff --git a/sd/source/ui/dlg/filedlg.cxx b/sd/source/ui/dlg/filedlg.cxx
index ae6aa5818687..3b05fa2f4cd8 100644
--- a/sd/source/ui/dlg/filedlg.cxx
+++ b/sd/source/ui/dlg/filedlg.cxx
@@ -64,8 +64,8 @@ private:
DECL_LINK_TYPED( IsMusicStoppedHdl, Idle *, void );
public:
- SdFileDialog_Imp( const short nDialogType, bool bUsableSelection );
- virtual ~SdFileDialog_Imp();
+ SdFileDialog_Imp( const short nDialogType );
+ virtual ~SdFileDialog_Imp();
ErrCode Execute();
@@ -213,11 +213,10 @@ void SdFileDialog_Imp::CheckSelectionState()
}
}
-SdFileDialog_Imp::SdFileDialog_Imp( const short nDialogType,
- bool bUsableSelection ) :
+SdFileDialog_Imp::SdFileDialog_Imp( const short nDialogType ) :
FileDialogHelper( nDialogType, 0 ),
mnPlaySoundEvent( nullptr ),
- mbUsableSelection( bUsableSelection ),
+ mbUsableSelection( false ),
mbLabelPlaying(false)
{
maUpdateIdle.SetIdleHdl(LINK(this, SdFileDialog_Imp, IsMusicStoppedHdl));
@@ -278,9 +277,7 @@ ErrCode SdFileDialog_Imp::Execute()
// these are simple forwarders
SdOpenSoundFileDialog::SdOpenSoundFileDialog() :
- mpImpl(
- new SdFileDialog_Imp(
- css::ui::dialogs::TemplateDescription::FILEOPEN_PLAY, false ) )
+ mpImpl( new SdFileDialog_Imp( css::ui::dialogs::TemplateDescription::FILEOPEN_PLAY ) )
{
OUString aDescr;
aDescr = SD_RESSTR(STR_ALL_FILES);
diff --git a/sd/source/ui/docshell/docshel4.cxx b/sd/source/ui/docshell/docshel4.cxx
index 71ac8e4e45dd..a2552e2ec565 100644
--- a/sd/source/ui/docshell/docshel4.cxx
+++ b/sd/source/ui/docshell/docshel4.cxx
@@ -287,7 +287,7 @@ bool DrawDocShell::Load( SfxMedium& rMedium )
bRet = SfxObjectShell::Load( rMedium );
if( bRet )
{
- bRet = SdXMLFilter( rMedium, *this, true, SDXMLMODE_Normal, SotStorage::GetVersion( rMedium.GetStorage() ) ).Import( nError );
+ bRet = SdXMLFilter( rMedium, *this, SDXMLMODE_Normal, SotStorage::GetVersion( rMedium.GetStorage() ) ).Import( nError );
}
if( bRet )
@@ -351,7 +351,7 @@ bool DrawDocShell::LoadFrom( SfxMedium& rMedium )
// TODO/LATER: nobody is interested in the error code?!
ErrCode nError = ERRCODE_NONE;
- bool bRet = SdXMLFilter( rMedium, *this, true, SDXMLMODE_Organizer, SotStorage::GetVersion( rMedium.GetStorage() ) ).Import( nError );
+ bool bRet = SdXMLFilter( rMedium, *this, SDXMLMODE_Organizer, SotStorage::GetVersion( rMedium.GetStorage() ) ).Import( nError );
// tell SFX to change viewshell when in preview mode
if( IsPreview() )
@@ -450,7 +450,7 @@ bool DrawDocShell::ConvertFrom( SfxMedium& rMedium )
|| aFilterName == pFilterPowerPoint97AutoPlay)
{
mpDoc->StopWorkStartupDelay();
- bRet = SdPPTFilter( rMedium, *this, true ).Import();
+ bRet = SdPPTFilter( rMedium, *this ).Import();
}
else if (aFilterName.indexOf("impress8") >= 0 ||
aFilterName.indexOf("draw8") >= 0)
@@ -459,7 +459,7 @@ bool DrawDocShell::ConvertFrom( SfxMedium& rMedium )
mpDoc->CreateFirstPages();
mpDoc->StopWorkStartupDelay();
ErrCode nError = ERRCODE_NONE;
- bRet = SdXMLFilter( rMedium, *this, true ).Import( nError );
+ bRet = SdXMLFilter( rMedium, *this ).Import( nError );
}
else if (aFilterName.indexOf("StarOffice XML (Draw)") >= 0 ||
@@ -469,13 +469,13 @@ bool DrawDocShell::ConvertFrom( SfxMedium& rMedium )
mpDoc->CreateFirstPages();
mpDoc->StopWorkStartupDelay();
ErrCode nError = ERRCODE_NONE;
- bRet = SdXMLFilter( rMedium, *this, true, SDXMLMODE_Normal, SOFFICE_FILEFORMAT_60 ).Import( nError );
+ bRet = SdXMLFilter( rMedium, *this, SDXMLMODE_Normal, SOFFICE_FILEFORMAT_60 ).Import( nError );
}
else if( aFilterName == "CGM - Computer Graphics Metafile" )
{
mpDoc->CreateFirstPages();
mpDoc->StopWorkStartupDelay();
- bRet = SdCGMFilter( rMedium, *this, true ).Import();
+ bRet = SdCGMFilter( rMedium, *this ).Import();
}
else
{
@@ -521,7 +521,7 @@ bool DrawDocShell::Save()
bool bRet = SfxObjectShell::Save();
if( bRet )
- bRet = SdXMLFilter( *GetMedium(), *this, true, SDXMLMODE_Normal, SotStorage::GetVersion( GetMedium()->GetStorage() ) ).Export();
+ bRet = SdXMLFilter( *GetMedium(), *this, SDXMLMODE_Normal, SotStorage::GetVersion( GetMedium()->GetStorage() ) ).Export();
return bRet;
}
@@ -555,7 +555,7 @@ bool DrawDocShell::SaveAs( SfxMedium& rMedium )
bool bRet = SfxObjectShell::SaveAs( rMedium );
if( bRet )
- bRet = SdXMLFilter( rMedium, *this, true, SDXMLMODE_Normal, SotStorage::GetVersion( rMedium.GetStorage() ) ).Export();
+ bRet = SdXMLFilter( rMedium, *this, SDXMLMODE_Normal, SotStorage::GetVersion( rMedium.GetStorage() ) ).Export();
if( GetError() == ERRCODE_NONE )
SetError( nVBWarning, OSL_LOG_PREFIX );
@@ -578,26 +578,26 @@ bool DrawDocShell::ConvertTo( SfxMedium& rMedium )
if( aTypeName.indexOf( "graphic_HTML" ) >= 0 )
{
- pFilter = new SdHTMLFilter( rMedium, *this, true );
+ pFilter = new SdHTMLFilter( rMedium, *this );
}
else if( aTypeName.indexOf( "MS_PowerPoint_97" ) >= 0 )
{
- pFilter = new SdPPTFilter( rMedium, *this, true );
+ pFilter = new SdPPTFilter( rMedium, *this );
static_cast<SdPPTFilter*>(pFilter)->PreSaveBasic();
}
else if ( aTypeName.indexOf( "CGM_Computer_Graphics_Metafile" ) >= 0 )
{
- pFilter = new SdCGMFilter( rMedium, *this, true );
+ pFilter = new SdCGMFilter( rMedium, *this );
}
else if( aTypeName.indexOf( "draw8" ) >= 0 ||
aTypeName.indexOf( "impress8" ) >= 0 )
{
- pFilter = new SdXMLFilter( rMedium, *this, true );
+ pFilter = new SdXMLFilter( rMedium, *this );
}
else if( aTypeName.indexOf( "StarOffice_XML_Impress" ) >= 0 ||
aTypeName.indexOf( "StarOffice_XML_Draw" ) >= 0 )
{
- pFilter = new SdXMLFilter( rMedium, *this, true, SDXMLMODE_Normal, SOFFICE_FILEFORMAT_60 );
+ pFilter = new SdXMLFilter( rMedium, *this, SDXMLMODE_Normal, SOFFICE_FILEFORMAT_60 );
}
else
{
diff --git a/sd/source/ui/inc/ViewShell.hxx b/sd/source/ui/inc/ViewShell.hxx
index 21ccfde765fa..e4e2ad46c80d 100644
--- a/sd/source/ui/inc/ViewShell.hxx
+++ b/sd/source/ui/inc/ViewShell.hxx
@@ -112,8 +112,7 @@ public:
ViewShell (
SfxViewFrame *pFrame,
vcl::Window* pParentWindow,
- ViewShellBase& rViewShellBase,
- bool bAllowCenter = true);
+ ViewShellBase& rViewShellBase);
virtual ~ViewShell();
/** The Init method has to be called from the outside directly
@@ -298,7 +297,7 @@ public:
// Move these two methods from DrawViewShell to enable slide show view
void NotifyAccUpdate();
void fireSwitchCurrentPage(sal_Int32 pageIndex);
- void SetWinViewPos(const Point& rWinPos, bool bUpdate);
+ void SetWinViewPos(const Point& rWinPos);
Point GetWinViewPos() const;
Point GetViewOrigin() const;
diff --git a/sd/source/ui/slidesorter/cache/SlsBitmapCache.cxx b/sd/source/ui/slidesorter/cache/SlsBitmapCache.cxx
index 93e527d162f9..9ca737da0c5f 100644
--- a/sd/source/ui/slidesorter/cache/SlsBitmapCache.cxx
+++ b/sd/source/ui/slidesorter/cache/SlsBitmapCache.cxx
@@ -115,7 +115,7 @@ typedef ::std::vector<
//===== BitmapCache =========================================================
-BitmapCache::BitmapCache (const sal_Int32 nMaximalNormalCacheSize)
+BitmapCache::BitmapCache ()
: maMutex(),
mpBitmapContainer(new CacheBitmapContainer()),
mnNormalCacheSize(0),
@@ -125,14 +125,9 @@ BitmapCache::BitmapCache (const sal_Int32 nMaximalNormalCacheSize)
mpCacheCompactor(),
mbIsFull(false)
{
- if (nMaximalNormalCacheSize > 0)
- mnMaximalNormalCacheSize = nMaximalNormalCacheSize;
- else
- {
- Any aCacheSize (CacheConfiguration::Instance()->GetValue("CacheSize"));
- if (aCacheSize.has<sal_Int32>())
- aCacheSize >>= mnMaximalNormalCacheSize;
- }
+ Any aCacheSize (CacheConfiguration::Instance()->GetValue("CacheSize"));
+ if (aCacheSize.has<sal_Int32>())
+ aCacheSize >>= mnMaximalNormalCacheSize;
mpCacheCompactor = CacheCompactor::Create(*this,mnMaximalNormalCacheSize);
}
diff --git a/sd/source/ui/slidesorter/cache/SlsBitmapCache.hxx b/sd/source/ui/slidesorter/cache/SlsBitmapCache.hxx
index ff875ae59424..7b399ad22f12 100644
--- a/sd/source/ui/slidesorter/cache/SlsBitmapCache.hxx
+++ b/sd/source/ui/slidesorter/cache/SlsBitmapCache.hxx
@@ -57,13 +57,11 @@ public:
typedef ::std::vector<CacheKey> CacheIndex;
/** Create a new cache for bitmap objects.
- @param nMaximalNormalCacheSize
- When a size larger then zero is given then that size is used.
- Otherwise the default value from the configuration is used.
- When that does not exist either then a internal default value is
- used.
+ The default value from the configuration is used.
+ When that does not exist then a internal default value is
+ used.
*/
- explicit BitmapCache (const sal_Int32 nMaximalNormalCacheSize = 0);
+ explicit BitmapCache ();
/** The destructor clears the cache and releases all bitmaps still in it.
*/
diff --git a/sd/source/ui/slidesorter/cache/SlsPageCache.cxx b/sd/source/ui/slidesorter/cache/SlsPageCache.cxx
index 07f4f62e2ce0..b10dbd66255d 100644
--- a/sd/source/ui/slidesorter/cache/SlsPageCache.cxx
+++ b/sd/source/ui/slidesorter/cache/SlsPageCache.cxx
@@ -60,10 +60,9 @@ Bitmap PageCache::GetPreviewBitmap (
}
Bitmap PageCache::GetMarkedPreviewBitmap (
- const CacheKey aKey,
- const bool bResize)
+ const CacheKey aKey)
{
- return mpImplementation->GetMarkedPreviewBitmap(aKey, bResize);
+ return mpImplementation->GetMarkedPreviewBitmap(aKey, false/*bResize*/);
}
void PageCache::SetMarkedPreviewBitmap (
@@ -79,10 +78,9 @@ void PageCache::RequestPreviewBitmap (const CacheKey aKey)
}
void PageCache::InvalidatePreviewBitmap (
- const CacheKey aKey,
- const bool bRequestPreview)
+ const CacheKey aKey)
{
- if (mpImplementation->InvalidatePreviewBitmap(aKey) && bRequestPreview)
+ if (mpImplementation->InvalidatePreviewBitmap(aKey))
RequestPreviewBitmap(aKey);
}
diff --git a/sd/source/ui/slidesorter/controller/SlideSorterController.cxx b/sd/source/ui/slidesorter/controller/SlideSorterController.cxx
index 2069d2473dd6..703d0c5fed36 100644
--- a/sd/source/ui/slidesorter/controller/SlideSorterController.cxx
+++ b/sd/source/ui/slidesorter/controller/SlideSorterController.cxx
@@ -900,8 +900,7 @@ void SlideSorterController::CheckForMasterPageAssignment()
if (pDescriptor->UpdateMasterPage())
{
mrView.GetPreviewCache()->InvalidatePreviewBitmap (
- pDescriptor->GetPage(),
- true);
+ pDescriptor->GetPage());
}
}
}
@@ -917,8 +916,7 @@ void SlideSorterController::CheckForSlideTransitionAssignment()
if (pDescriptor->UpdateTransitionFlag())
{
mrView.GetPreviewCache()->InvalidatePreviewBitmap (
- pDescriptor->GetPage(),
- true);
+ pDescriptor->GetPage());
}
}
}
diff --git a/sd/source/ui/slidesorter/controller/SlsAnimator.cxx b/sd/source/ui/slidesorter/controller/SlsAnimator.cxx
index 7268cd7227c7..6d13918a6e37 100644
--- a/sd/source/ui/slidesorter/controller/SlsAnimator.cxx
+++ b/sd/source/ui/slidesorter/controller/SlsAnimator.cxx
@@ -99,7 +99,6 @@ void Animator::Dispose()
Animator::AnimationId Animator::AddAnimation (
const AnimationFunctor& rAnimation,
- const sal_Int32 nStartOffset,
const sal_Int32 nDuration,
const FinishFunctor& rFinishFunctor)
{
@@ -112,7 +111,7 @@ Animator::AnimationId Animator::AddAnimation (
std::shared_ptr<Animation> pAnimation (
new Animation(
rAnimation,
- nStartOffset / 1000.0,
+ 0,
nDuration / 1000.0,
maElapsedTime.getElapsedTime(),
++mnNextAnimationId,
diff --git a/sd/source/ui/slidesorter/controller/SlsCurrentSlideManager.cxx b/sd/source/ui/slidesorter/controller/SlsCurrentSlideManager.cxx
index a8e6a4925b81..da5e6e086fd7 100644
--- a/sd/source/ui/slidesorter/controller/SlsCurrentSlideManager.cxx
+++ b/sd/source/ui/slidesorter/controller/SlsCurrentSlideManager.cxx
@@ -115,10 +115,9 @@ void CurrentSlideManager::AcquireCurrentSlide (const sal_Int32 nSlideIndex)
}
void CurrentSlideManager::SwitchCurrentSlide (
- const sal_Int32 nSlideIndex,
- const bool bUpdateSelection)
+ const sal_Int32 nSlideIndex)
{
- SwitchCurrentSlide(mrSlideSorter.GetModel().GetPageDescriptor(nSlideIndex), bUpdateSelection);
+ SwitchCurrentSlide(mrSlideSorter.GetModel().GetPageDescriptor(nSlideIndex), true/*bUpdateSelection*/);
}
void CurrentSlideManager::SwitchCurrentSlide (
diff --git a/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.cxx b/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.cxx
index 05090c81f26f..b27701d9b779 100644
--- a/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.cxx
+++ b/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.cxx
@@ -111,8 +111,7 @@ void DragAndDropContext::UpdatePosition (
void DragAndDropContext::SetTargetSlideSorter (
const Point& rMousePosition,
- const InsertionIndicatorHandler::Mode eMode,
- const bool bIsOverSourceView)
+ const InsertionIndicatorHandler::Mode eMode)
{
if (mpTargetSlideSorter != nullptr)
{
@@ -126,7 +125,7 @@ void DragAndDropContext::SetTargetSlideSorter (
if (mpTargetSlideSorter != nullptr)
{
mpTargetSlideSorter->GetController().GetInsertionIndicatorHandler()->Start(
- bIsOverSourceView);
+ false/*bIsOverSourceView*/);
mpTargetSlideSorter->GetController().GetInsertionIndicatorHandler()->UpdatePosition(
rMousePosition,
eMode);
diff --git a/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.hxx b/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.hxx
index 1e14c7039da3..3fa783ea896d 100644
--- a/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.hxx
+++ b/sd/source/ui/slidesorter/controller/SlsDragAndDropContext.hxx
@@ -62,8 +62,7 @@ public:
void SetTargetSlideSorter (
const Point& rMousePosition = Point(0,0),
- const InsertionIndicatorHandler::Mode eMode = InsertionIndicatorHandler::UnknownMode,
- const bool bIsOverSourceView = false);
+ const InsertionIndicatorHandler::Mode eMode = InsertionIndicatorHandler::UnknownMode);
private:
SlideSorter* mpTargetSlideSorter;
diff --git a/sd/source/ui/slidesorter/controller/SlsVisibleAreaManager.cxx b/sd/source/ui/slidesorter/controller/SlsVisibleAreaManager.cxx
index 3de85b3ccb58..644a4371eb1f 100644
--- a/sd/source/ui/slidesorter/controller/SlsVisibleAreaManager.cxx
+++ b/sd/source/ui/slidesorter/controller/SlsVisibleAreaManager.cxx
@@ -135,7 +135,6 @@ void VisibleAreaManager::MakeVisible()
{
mnScrollAnimationId = mrSlideSorter.GetController().GetAnimator()->AddAnimation(
aAnimation,
- 0,
300);
}
else
diff --git a/sd/source/ui/slidesorter/inc/cache/SlsPageCache.hxx b/sd/source/ui/slidesorter/inc/cache/SlsPageCache.hxx
index 90f0d3090c0e..47caa1d814f7 100644
--- a/sd/source/ui/slidesorter/inc/cache/SlsPageCache.hxx
+++ b/sd/source/ui/slidesorter/inc/cache/SlsPageCache.hxx
@@ -108,8 +108,7 @@ public:
const bool bResize);
Bitmap GetMarkedPreviewBitmap (
- const CacheKey aKey,
- const bool bResize);
+ const CacheKey aKey);
void SetMarkedPreviewBitmap (
const CacheKey aKey,
const Bitmap& rBitmap);
@@ -123,13 +122,10 @@ public:
/** Tell the cache that the bitmap associated with the given request
data is not up-to-date anymore. This will invalidate all previews
in other caches that represent the same page as well.
- @param bRequestPreview
- When <TRUE/> then a new preview is requested and will lead
- eventually to a repaint of the associated page object.
+ A new preview is requested and will lead
+ eventually to a repaint of the associated page object.
*/
- void InvalidatePreviewBitmap (
- const CacheKey aKey,
- const bool bRequestPreview);
+ void InvalidatePreviewBitmap (const CacheKey aKey);
/** Call this method when all preview bitmaps have to be generated anew.
This is the case when the size of the page objects on the screen has
diff --git a/sd/source/ui/slidesorter/inc/controller/SlsAnimator.hxx b/sd/source/ui/slidesorter/inc/controller/SlsAnimator.hxx
index 0128b41917a1..9b90f36fba3a 100644
--- a/sd/source/ui/slidesorter/inc/controller/SlsAnimator.hxx
+++ b/sd/source/ui/slidesorter/inc/controller/SlsAnimator.hxx
@@ -68,14 +68,11 @@ public:
the specified duration.
@param rAnimation
The animation operation.
- @param nStartOffset
- Time in milli seconds before the animation is started.
@param nDuration
The duration in milli seconds.
*/
AnimationId AddAnimation (
const AnimationFunctor& rAnimation,
- const sal_Int32 nStartOffset,
const sal_Int32 nDuration,
const FinishFunctor& rFinishFunctor = FinishFunctor());
diff --git a/sd/source/ui/slidesorter/inc/controller/SlsCurrentSlideManager.hxx b/sd/source/ui/slidesorter/inc/controller/SlsCurrentSlideManager.hxx
index cb3f2b42771a..c27da2dc9c74 100644
--- a/sd/source/ui/slidesorter/inc/controller/SlsCurrentSlideManager.hxx
+++ b/sd/source/ui/slidesorter/inc/controller/SlsCurrentSlideManager.hxx
@@ -62,13 +62,10 @@ public:
slide change has been initiated by someone else.
@param nSlideIndex
Zero based index in the range [0,number-of-slides).
- @param bUpdateSelection
- When <TRUE/> then the page selection is cleared and only the new
+ The page selection is cleared and only the new
current slide is selected.
*/
- void SwitchCurrentSlide (
- const sal_Int32 nSlideIndex,
- const bool bUpdateSelection = false);
+ void SwitchCurrentSlide (const sal_Int32 nSlideIndex);
void SwitchCurrentSlide (
const model::SharedPageDescriptor& rpSlide,
const bool bUpdateSelection = false);
diff --git a/sd/source/ui/slidesorter/inc/model/SlsPageEnumeration.hxx b/sd/source/ui/slidesorter/inc/model/SlsPageEnumeration.hxx
index 226bb5b987e1..3afaadc5f611 100644
--- a/sd/source/ui/slidesorter/inc/model/SlsPageEnumeration.hxx
+++ b/sd/source/ui/slidesorter/inc/model/SlsPageEnumeration.hxx
@@ -65,19 +65,6 @@ public:
virtual ~PageEnumeration();
- /** Create a new enumeration object. The ownership of the
- implementation object goes to the new object. Use this copy
- constructor only when you know what you are doing. When in doubt,
- use the one argument version.
- @param bCloneImpl
- When <TRUE/> is given this constructor behaves exactly like its
- one argument version. When <FALSE/> is given then the
- implementation object is not copied but moved from the given
- enumeration to the newly created one. The given enumeration
- thus becomes empty.
- */
- PageEnumeration (PageEnumeration& rEnumeration, bool bCloneImpl);
-
/** Create and return an exact copy of the called object.
*/
virtual ::std::unique_ptr<Enumeration<SharedPageDescriptor> > Clone() override;
diff --git a/sd/source/ui/slidesorter/model/SlsPageEnumeration.cxx b/sd/source/ui/slidesorter/model/SlsPageEnumeration.cxx
index 2a01da30910d..ff6a398a531e 100644
--- a/sd/source/ui/slidesorter/model/SlsPageEnumeration.cxx
+++ b/sd/source/ui/slidesorter/model/SlsPageEnumeration.cxx
@@ -85,21 +85,6 @@ PageEnumeration::PageEnumeration (
{
}
-PageEnumeration::PageEnumeration (
- PageEnumeration& rEnumeration,
- bool bCloneImpl)
-{
-
- if( bCloneImpl )
- {
- mpImpl = rEnumeration.mpImpl->Clone();
- }
- else
- {
- mpImpl = std::move(rEnumeration.mpImpl);
- }
-}
-
PageEnumeration::PageEnumeration (const PageEnumeration& rEnumeration )
: sd::slidesorter::model::Enumeration<sd::slidesorter::model::SharedPageDescriptor>()
{
@@ -120,7 +105,7 @@ PageEnumeration& PageEnumeration::operator= (
::std::unique_ptr<Enumeration<SharedPageDescriptor> > PageEnumeration::Clone()
{
return ::std::unique_ptr<Enumeration<SharedPageDescriptor> >(
- new PageEnumeration (*this, true));
+ new PageEnumeration (*this));
}
bool PageEnumeration::HasMoreElements() const
diff --git a/sd/source/ui/slidesorter/view/SlsFramePainter.cxx b/sd/source/ui/slidesorter/view/SlsFramePainter.cxx
index f1c0140697aa..bb297c922a05 100644
--- a/sd/source/ui/slidesorter/view/SlsFramePainter.cxx
+++ b/sd/source/ui/slidesorter/view/SlsFramePainter.cxx
@@ -73,8 +73,7 @@ void FramePainter::PaintFrame (
}
void FramePainter::AdaptColor (
- const Color aNewColor,
- const bool bEraseCenter)
+ const Color aNewColor)
{
// Get the source color.
if (maCenter.maBitmap.IsEmpty())
@@ -86,8 +85,7 @@ void FramePainter::AdaptColor (
Bitmap::ReleaseAccess(pReadAccess);
// Erase the center bitmap.
- if (bEraseCenter)
- maCenter.maBitmap.SetEmpty();
+ maCenter.maBitmap.SetEmpty();
// Replace the color in all bitmaps.
maTopLeft.maBitmap.Replace(aSourceColor, aNewColor);
diff --git a/sd/source/ui/slidesorter/view/SlsFramePainter.hxx b/sd/source/ui/slidesorter/view/SlsFramePainter.hxx
index 614ec0a70577..5ce2ee6d06d0 100644
--- a/sd/source/ui/slidesorter/view/SlsFramePainter.hxx
+++ b/sd/source/ui/slidesorter/view/SlsFramePainter.hxx
@@ -38,10 +38,9 @@ public:
/** Special functionality that takes the color from the center
bitmap and replaces that color in all bitmaps by the given new
color. Alpha values are not modified.
- @param bClearCenterBitmap
- When <TRUE/> then the center bitmap is erased.
+ The center bitmap is erased.
*/
- void AdaptColor (const Color aNewColor, const bool bClearCenterBitmap);
+ void AdaptColor (const Color aNewColor);
private:
/** Bitmap with offset that is used when the bitmap is painted. The bitmap
diff --git a/sd/source/ui/slidesorter/view/SlsInsertAnimator.cxx b/sd/source/ui/slidesorter/view/SlsInsertAnimator.cxx
index f12f9999e07e..41e878e4fdbd 100644
--- a/sd/source/ui/slidesorter/view/SlsInsertAnimator.cxx
+++ b/sd/source/ui/slidesorter/view/SlsInsertAnimator.cxx
@@ -393,7 +393,6 @@ void PageObjectRun::RestartAnimation()
auto sharedThis(shared_from_this());
mnAnimationId = mrAnimatorAccess.GetAnimator()->AddAnimation(
[this] (double const val) { (*this)(val); },
- 0,
300,
[sharedThis] () { sharedThis->mrAnimatorAccess.RemoveRun(sharedThis); }
);
diff --git a/sd/source/ui/slidesorter/view/SlsPageObjectPainter.cxx b/sd/source/ui/slidesorter/view/SlsPageObjectPainter.cxx
index 515aaa896dd7..e9d3f6e68d66 100644
--- a/sd/source/ui/slidesorter/view/SlsPageObjectPainter.cxx
+++ b/sd/source/ui/slidesorter/view/SlsPageObjectPainter.cxx
@@ -57,7 +57,7 @@ PageObjectPainter::PageObjectPainter (
sal_uInt16 nHue, nSat, nBri;
aColor.RGBtoHSB(nHue, nSat, nBri);
aColor = Color::HSBtoRGB(nHue, 28, 65);
- mpFocusBorderPainter->AdaptColor(aColor, true);
+ mpFocusBorderPainter->AdaptColor(aColor);
}
PageObjectPainter::~PageObjectPainter()
@@ -191,7 +191,7 @@ Bitmap PageObjectPainter::GetPreviewBitmap (
{
PageObjectLayouter *pPageObjectLayouter = mrLayouter.GetPageObjectLayouter().get();
- Bitmap aMarkedPreview (mpCache->GetMarkedPreviewBitmap(pPage,false));
+ Bitmap aMarkedPreview (mpCache->GetMarkedPreviewBitmap(pPage));
const Rectangle aPreviewBox (pPageObjectLayouter->GetBoundingBox(
rpDescriptor,
PageObjectLayouter::Preview,
diff --git a/sd/source/ui/unoidl/SdUnoDrawView.cxx b/sd/source/ui/unoidl/SdUnoDrawView.cxx
index d2e6ea118d89..e7cfe2bb4788 100644
--- a/sd/source/ui/unoidl/SdUnoDrawView.cxx
+++ b/sd/source/ui/unoidl/SdUnoDrawView.cxx
@@ -487,7 +487,7 @@ void SdUnoDrawView::SetViewOffset(const awt::Point& rWinPos )
{
Point aWinPos( rWinPos.X, rWinPos.Y );
aWinPos += mrDrawViewShell.GetViewOrigin();
- mrDrawViewShell.SetWinViewPos( aWinPos, true );
+ mrDrawViewShell.SetWinViewPos( aWinPos );
}
awt::Point SdUnoDrawView::GetViewOffset() const
diff --git a/sd/source/ui/unoidl/SdUnoSlideView.cxx b/sd/source/ui/unoidl/SdUnoSlideView.cxx
index cd3a8b2f570e..c220a74910f0 100644
--- a/sd/source/ui/unoidl/SdUnoSlideView.cxx
+++ b/sd/source/ui/unoidl/SdUnoSlideView.cxx
@@ -134,8 +134,7 @@ void SAL_CALL SdUnoSlideView::setCurrentPage (
if (xProperties->getPropertyValue("Number") >>= nPageNumber)
{
mrSlideSorter.GetController().GetCurrentSlideManager()->SwitchCurrentSlide(
- nPageNumber-1,
- true);
+ nPageNumber-1);
}
}
}
diff --git a/sd/source/ui/view/sdview2.cxx b/sd/source/ui/view/sdview2.cxx
index 486535fa248d..4cc7c56ca93b 100644
--- a/sd/source/ui/view/sdview2.cxx
+++ b/sd/source/ui/view/sdview2.cxx
@@ -845,7 +845,7 @@ IMPL_LINK_TYPED( View, ExecuteNavigatorDrop, void*, p, void )
mrDoc.InsertBookmark( aBookmarkList, aExchangeList,
bLink, bReplace, nPgPos, false,
&pPageObjsTransferable->GetDocShell(),
- true, &aPos );
+ &aPos );
}
}
diff --git a/sd/source/ui/view/viewshe2.cxx b/sd/source/ui/view/viewshe2.cxx
index 692262c69836..8e8aeab739eb 100644
--- a/sd/source/ui/view/viewshe2.cxx
+++ b/sd/source/ui/view/viewshe2.cxx
@@ -982,17 +982,14 @@ void ViewShell::VisAreaChanged(const Rectangle& rRect)
GetViewShell()->VisAreaChanged(rRect);
}
-void ViewShell::SetWinViewPos(const Point& rWinPos, bool bUpdate)
+void ViewShell::SetWinViewPos(const Point& rWinPos)
{
if (mpContentWindow.get() != nullptr)
{
mpContentWindow->SetWinViewPos(rWinPos);
- if ( bUpdate )
- {
- mpContentWindow->UpdateMapOrigin();
- mpContentWindow->Invalidate();
- }
+ mpContentWindow->UpdateMapOrigin();
+ mpContentWindow->Invalidate();
}
if (mbHasRulers)
diff --git a/sd/source/ui/view/viewshel.cxx b/sd/source/ui/view/viewshel.cxx
index 67a38fe3bd3f..9e2889c93e07 100644
--- a/sd/source/ui/view/viewshel.cxx
+++ b/sd/source/ui/view/viewshel.cxx
@@ -135,9 +135,9 @@ SfxViewFrame* ViewShell::GetViewFrame() const
/// declare SFX-Slotmap and standard interface
-ViewShell::ViewShell( SfxViewFrame*, vcl::Window* pParentWindow, ViewShellBase& rViewShellBase, bool bAllowCenter)
+ViewShell::ViewShell( SfxViewFrame*, vcl::Window* pParentWindow, ViewShellBase& rViewShellBase)
: SfxShell(&rViewShellBase)
-, mbCenterAllowed(bAllowCenter)
+, mbCenterAllowed(true)
, mpParentWindow(pParentWindow)
{
construct();
@@ -796,7 +796,7 @@ bool ViewShell::HandleScrollCommand(const CommandEvent& rCEvt, ::sd::Window* pWi
SetZoom( nNewZoom );
// Keep mouse at same doc point before zoom
Point aNewMousePos = GetActiveWindow()->PixelToLogic(rCEvt.GetMousePosPixel());
- SetWinViewPos(GetWinViewPos() - (aNewMousePos - aOldMousePos), true);
+ SetWinViewPos(GetWinViewPos() - (aNewMousePos - aOldMousePos));
Invalidate( SID_ATTR_ZOOM );
Invalidate( SID_ATTR_ZOOMSLIDER );