From 87661a7ab3275195ca251188ce4d7825fd038de6 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 2 May 2017 16:23:46 +0200 Subject: [PATCH] loplugin:checkunusedparams in sd Change-Id: I217a4177e9c2cdc2d85100d253f7f38ec8aad756 Reviewed-on: https://gerrit.libreoffice.org/37162 Tested-by: Jenkins Reviewed-by: Noel Grandin --- sd/inc/drawdoc.hxx | 8 ++-- sd/inc/sdmod.hxx | 6 --- sd/inc/sdpage.hxx | 2 +- sd/source/core/drawdoc.cxx | 4 +- sd/source/core/drawdoc3.cxx | 3 +- sd/source/core/drawdoc4.cxx | 4 +- sd/source/core/sdpage.cxx | 21 +++++----- sd/source/filter/eppt/eppt.cxx | 7 +--- sd/source/filter/eppt/eppt.hxx | 2 +- sd/source/filter/eppt/epptbase.hxx | 4 +- sd/source/filter/eppt/epptso.cxx | 20 ++++----- sd/source/filter/eppt/pptexanimations.cxx | 42 +++++++++---------- sd/source/filter/eppt/pptexanimations.hxx | 11 +++-- sd/source/filter/eppt/pptx-stylesheet.cxx | 4 +- sd/source/filter/html/htmlex.cxx | 9 ++-- sd/source/filter/html/htmlex.hxx | 2 +- sd/source/filter/ppt/pptin.cxx | 2 +- sd/source/ui/app/optsitem.cxx | 2 +- sd/source/ui/app/sdmod.cxx | 8 ---- sd/source/ui/app/sdmod2.cxx | 2 +- sd/source/ui/dlg/docprev.cxx | 4 +- sd/source/ui/dlg/sdtreelb.cxx | 4 +- .../framework/factories/BasicViewFactory.cxx | 5 --- sd/source/ui/func/fuexpand.cxx | 2 +- sd/source/ui/func/fuinsfil.cxx | 6 +-- sd/source/ui/func/fupoor.cxx | 4 +- sd/source/ui/func/fusumry.cxx | 2 +- sd/source/ui/inc/DrawViewShell.hxx | 19 ++------- sd/source/ui/inc/FrameView.hxx | 4 +- sd/source/ui/inc/GraphicViewShell.hxx | 1 - sd/source/ui/inc/OutlineView.hxx | 2 +- sd/source/ui/inc/OutlineViewShell.hxx | 2 +- sd/source/ui/inc/PresentationViewShell.hxx | 2 +- sd/source/ui/inc/SlideSorter.hxx | 2 +- sd/source/ui/inc/TabControl.hxx | 2 +- sd/source/ui/inc/View.hxx | 6 +-- sd/source/ui/inc/ViewShell.hxx | 22 +++------- sd/source/ui/inc/optsitem.hxx | 2 +- sd/source/ui/inc/sdtreelb.hxx | 2 +- sd/source/ui/inc/slideshow.hxx | 4 +- sd/source/ui/sidebar/MasterPagesSelector.cxx | 2 +- sd/source/ui/sidebar/PreviewValueSet.cxx | 2 +- sd/source/ui/sidebar/PreviewValueSet.hxx | 2 +- sd/source/ui/slideshow/showwin.cxx | 2 +- sd/source/ui/slideshow/slideshow.cxx | 6 +-- sd/source/ui/slideshow/slideshowimpl.cxx | 8 ++-- sd/source/ui/slideshow/slideshowimpl.hxx | 4 +- .../ui/slidesorter/cache/SlsRequestQueue.cxx | 5 +-- .../ui/slidesorter/cache/SlsRequestQueue.hxx | 3 +- .../slidesorter/controller/SlsClipboard.cxx | 6 +-- .../slidesorter/controller/SlsSlotManager.cxx | 4 +- .../inc/controller/SlsClipboard.hxx | 2 +- .../ui/slidesorter/shell/SlideSorter.cxx | 10 ++--- .../shell/SlideSorterViewShell.cxx | 4 +- sd/source/ui/unoidl/unomodel.cxx | 4 +- sd/source/ui/unoidl/unopage.cxx | 2 +- sd/source/ui/view/Outliner.cxx | 8 ++-- sd/source/ui/view/ViewShellBase.cxx | 8 ++-- sd/source/ui/view/drawview.cxx | 2 +- sd/source/ui/view/drviews8.cxx | 2 +- sd/source/ui/view/drviewsa.cxx | 10 ++--- sd/source/ui/view/drviewse.cxx | 8 ++-- sd/source/ui/view/frmview.cxx | 4 +- sd/source/ui/view/grviewsh.cxx | 2 - sd/source/ui/view/outlnvsh.cxx | 14 +++---- sd/source/ui/view/outlview.cxx | 4 +- sd/source/ui/view/presvish.cxx | 8 ++-- sd/source/ui/view/sdview2.cxx | 8 ++-- sd/source/ui/view/sdwindow.cxx | 2 +- sd/source/ui/view/tabcontr.cxx | 6 +-- sd/source/ui/view/viewshe2.cxx | 14 +++---- sd/source/ui/view/viewshel.cxx | 10 ++--- 72 files changed, 188 insertions(+), 248 deletions(-) diff --git a/sd/inc/drawdoc.hxx b/sd/inc/drawdoc.hxx index 47af1b7c726f..72df74cbe843 100644 --- a/sd/inc/drawdoc.hxx +++ b/sd/inc/drawdoc.hxx @@ -334,7 +334,7 @@ public: SAL_DLLPRIVATE bool InsertBookmarkAsObject(const std::vector &rBookmarkList, const std::vector &rExchangeList, - bool bLink, ::sd::DrawDocShell* pBookmarkDocSh, + ::sd::DrawDocShell* pBookmarkDocSh, Point* pObjPos, bool bCalcObjCount); void CloseBookmarkDoc(); @@ -413,10 +413,10 @@ public: SAL_DLLPRIVATE void ImpOnlineSpellCallback(SpellCallbackInfo* pInfo, SdrObject* pObj, SdrOutliner* pOutl); - SAL_DLLPRIVATE void InsertObject(SdrObject* pObj, SdPage* pPage); - SAL_DLLPRIVATE void RemoveObject(SdrObject* pObj, SdPage* pPage); + SAL_DLLPRIVATE void InsertObject(SdrObject* pObj); + SAL_DLLPRIVATE void RemoveObject(SdrObject* pObj); - SAL_DLLPRIVATE sal_uLong GetLinkCount(); + SAL_DLLPRIVATE sal_uLong GetLinkCount(); SAL_DLLPRIVATE std::vector& GetFrameViewList() { return maFrameViewList; } SdCustomShowList* GetCustomShowList(bool bCreate = false); diff --git a/sd/inc/sdmod.hxx b/sd/inc/sdmod.hxx index c96a1b1749c9..ffd1660a2c9e 100644 --- a/sd/inc/sdmod.hxx +++ b/sd/inc/sdmod.hxx @@ -112,12 +112,6 @@ public: */ OutputDevice* GetVirtualRefDevice() { return mpVirtualRefDevice;} - /** Deprecated alias to GetVirtualRefDevice. - @param rDocShell - Unused dummy parameter. - */ - OutputDevice* GetRefDevice (::sd::DrawDocShell& rDocShell); - SD_DLLPUBLIC SvNumberFormatter* GetNumberFormatter(); // virtual methods for the option dialog diff --git a/sd/inc/sdpage.hxx b/sd/inc/sdpage.hxx index 0fb422bb1546..45d3b0e78262 100644 --- a/sd/inc/sdpage.hxx +++ b/sd/inc/sdpage.hxx @@ -169,7 +169,7 @@ public: sd::ShapeList& GetPresentationShapeList() { return maPresentationShapeList; } void EnsureMasterPageDefaultBackground(); - SdrObject* CreatePresObj(PresObjKind eObjKind, bool bVertical, const ::tools::Rectangle& rRect, bool bInsert=false); + SdrObject* CreatePresObj(PresObjKind eObjKind, bool bVertical, const ::tools::Rectangle& rRect); SdrObject* CreateDefaultPresObj(PresObjKind eObjKind); void DestroyDefaultPresObj(PresObjKind eObjKind); SdrObject* GetPresObj(PresObjKind eObjKind, int nIndex = 1, bool bFuzzySearch = false ); diff --git a/sd/source/core/drawdoc.cxx b/sd/source/core/drawdoc.cxx index 3a4a86e13260..0a383c637615 100644 --- a/sd/source/core/drawdoc.cxx +++ b/sd/source/core/drawdoc.cxx @@ -816,7 +816,7 @@ SdOutliner* SdDrawDocument::GetOutliner(bool bCreateOutliner) mpOutliner = new SdOutliner( this, OutlinerMode::TextObject ); if (mpDocSh) - mpOutliner->SetRefDevice( SD_MOD()->GetRefDevice( *mpDocSh ) ); + mpOutliner->SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); mpOutliner->SetDefTab( nDefaultTabulator ); mpOutliner->SetStyleSheetPool(static_cast(GetStyleSheetPool())); @@ -840,7 +840,7 @@ SdOutliner* SdDrawDocument::GetInternalOutliner(bool bCreateOutliner) mpInternalOutliner->EnableUndo( false ); if (mpDocSh) - mpInternalOutliner->SetRefDevice( SD_MOD()->GetRefDevice( *mpDocSh ) ); + mpInternalOutliner->SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); mpInternalOutliner->SetDefTab( nDefaultTabulator ); mpInternalOutliner->SetStyleSheetPool(static_cast(GetStyleSheetPool())); diff --git a/sd/source/core/drawdoc3.cxx b/sd/source/core/drawdoc3.cxx index 239db2a88dc1..f4030c714cfa 100644 --- a/sd/source/core/drawdoc3.cxx +++ b/sd/source/core/drawdoc3.cxx @@ -336,7 +336,7 @@ void SdDrawDocument::InsertBookmark( if ( bOK && !rBookmarkList.empty() ) { // Insert all object bookmarks - InsertBookmarkAsObject(rBookmarkList, rExchangeList, bLink, + InsertBookmarkAsObject(rBookmarkList, rExchangeList, pBookmarkDocSh, pObjPos, bCalcObjCount); } } @@ -978,7 +978,6 @@ bool SdDrawDocument::InsertBookmarkAsPage( bool SdDrawDocument::InsertBookmarkAsObject( const std::vector &rBookmarkList, const std::vector &rExchangeList, // List of names to use - bool /* bLink */, ::sd::DrawDocShell* pBookmarkDocSh, Point* pObjPos, bool bCalcObjCount) diff --git a/sd/source/core/drawdoc4.cxx b/sd/source/core/drawdoc4.cxx index 7fe0b5b5119f..d11df6530604 100644 --- a/sd/source/core/drawdoc4.cxx +++ b/sd/source/core/drawdoc4.cxx @@ -926,7 +926,7 @@ void SdDrawDocument::SpellObject(SdrTextObj* pObj) } // Object was inserted into model -void SdDrawDocument::InsertObject(SdrObject* pObj, SdPage* /*pPage*/) +void SdDrawDocument::InsertObject(SdrObject* pObj) { if(mpOnlineSpellingList && pObj) { @@ -939,7 +939,7 @@ void SdDrawDocument::InsertObject(SdrObject* pObj, SdPage* /*pPage*/) } // Object removed from model -void SdDrawDocument::RemoveObject(SdrObject* pObj, SdPage* /*pPage*/) +void SdDrawDocument::RemoveObject(SdrObject* pObj) { if(mpOnlineSpellingList && pObj) { diff --git a/sd/source/core/sdpage.cxx b/sd/source/core/sdpage.cxx index 9852f7a4848b..c96a8f122033 100644 --- a/sd/source/core/sdpage.cxx +++ b/sd/source/core/sdpage.cxx @@ -281,7 +281,7 @@ void SdPage::EnsureMasterPageDefaultBackground() /** creates a presentation object with the given PresObjKind on this page. A user call will be set */ -SdrObject* SdPage::CreatePresObj(PresObjKind eObjKind, bool bVertical, const ::tools::Rectangle& rRect, bool /* bInsert */ ) +SdrObject* SdPage::CreatePresObj(PresObjKind eObjKind, bool bVertical, const ::tools::Rectangle& rRect ) { ::svl::IUndoManager* pUndoManager = pModel ? static_cast(pModel)->GetUndoManager() : nullptr; const bool bUndo = pUndoManager && pUndoManager->IsInListAction() && IsInserted(); @@ -846,7 +846,7 @@ void SdPage::CreateTitleAndLayout(bool bInit, bool bCreate ) while( iter != aAreas.end() ) { - SdrPageObj* pPageObj = static_cast(pMasterPage->CreatePresObj(PRESOBJ_HANDOUT, false, (*iter++), true) ); + SdrPageObj* pPageObj = static_cast(pMasterPage->CreatePresObj(PRESOBJ_HANDOUT, false, (*iter++)) ); // #i105146# We want no content to be displayed for PageKind::Handout, // so just never set a page as content pPageObj->SetReferencedPage(nullptr); @@ -974,21 +974,20 @@ void getPresObjProp( const SdPage& rPage, const char* sObjKind, const char* sPag SdrObject* SdPage::CreateDefaultPresObj(PresObjKind eObjKind) { - if( eObjKind == PRESOBJ_TITLE ) { ::tools::Rectangle aTitleRect( GetTitleRect() ); - return CreatePresObj(PRESOBJ_TITLE, false, aTitleRect, true/*bInsert*/); + return CreatePresObj(PRESOBJ_TITLE, false, aTitleRect); } else if( eObjKind == PRESOBJ_OUTLINE ) { ::tools::Rectangle aLayoutRect( GetLayoutRect() ); - return CreatePresObj( PRESOBJ_OUTLINE, false, aLayoutRect, true/*bInsert*/); + return CreatePresObj( PRESOBJ_OUTLINE, false, aLayoutRect); } else if( eObjKind == PRESOBJ_NOTES ) { ::tools::Rectangle aLayoutRect( GetLayoutRect() ); - return CreatePresObj( PRESOBJ_NOTES, false, aLayoutRect, true/*bInsert*/); + return CreatePresObj( PRESOBJ_NOTES, false, aLayoutRect); } else if( (eObjKind == PRESOBJ_FOOTER) || (eObjKind == PRESOBJ_DATETIME) || (eObjKind == PRESOBJ_SLIDENUMBER) || (eObjKind == PRESOBJ_HEADER ) ) { @@ -1021,7 +1020,7 @@ SdrObject* SdPage::CreateDefaultPresObj(PresObjKind eObjKind) else { ::tools::Rectangle aRect( aPos, aSize ); - return CreatePresObj( eObjKind, false, aRect, true/*bInsert*/ ); + return CreatePresObj( eObjKind, false, aRect ); } } else @@ -1048,7 +1047,7 @@ SdrObject* SdPage::CreateDefaultPresObj(PresObjKind eObjKind) aPos.Y() = aPosition.Y() + long( aPageSize.Height() - NOTES_HEADER_FOOTER_HEIGHT ); ::tools::Rectangle aRect( aPos, aSize ); - return CreatePresObj( eObjKind, false, aRect, true/*bInsert*/ ); + return CreatePresObj( eObjKind, false, aRect ); } } else @@ -1687,7 +1686,7 @@ void SdPage::NbcInsertObject(SdrObject* pObj, size_t nPos) { FmFormPage::NbcInsertObject(pObj, nPos); - static_cast(pModel)->InsertObject(pObj, this); + static_cast(pModel)->InsertObject(pObj); SdrLayerID nId = pObj->GetLayer(); if( mbMaster ) @@ -1751,7 +1750,7 @@ void SdPage::onRemoveObject( SdrObject* pObject ) RemovePresObj(pObject); if( pModel ) - static_cast(pModel)->RemoveObject(pObject, this); + static_cast(pModel)->RemoveObject(pObject); removeAnimations( pObject ); } @@ -2425,7 +2424,7 @@ void SdPage::SetObjText(SdrTextObj* pObj, SdrOutliner* pOutliner, PresObjKind eO { SfxItemPool* pPool = static_cast(GetModel())->GetDrawOutliner().GetEmptyItemSet().GetPool(); pOutl = new ::Outliner( pPool, OutlinerMode::OutlineObject ); - pOutl->SetRefDevice( SD_MOD()->GetRefDevice( *static_cast( GetModel() )->GetDocSh() ) ); + pOutl->SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); pOutl->SetEditTextObjectPool(pPool); pOutl->SetStyleSheetPool(static_cast(GetModel()->GetStyleSheetPool())); pOutl->EnableUndo(false); diff --git a/sd/source/filter/eppt/eppt.cxx b/sd/source/filter/eppt/eppt.cxx index 14747c1203a6..2b98faf035af 100644 --- a/sd/source/filter/eppt/eppt.cxx +++ b/sd/source/filter/eppt/eppt.cxx @@ -399,7 +399,6 @@ void PPTWriter::ImplWriteSlideMaster( sal_uInt32 nPageNum, Reference< XPropertyS mpPptEscherEx->BeginAtom(); - bool bFirst = true; bool bSimpleText = false; mpStrm->WriteUInt16( 5 ); // paragraph count @@ -408,13 +407,11 @@ void PPTWriter::ImplWriteSlideMaster( sal_uInt32 nPageNum, Reference< XPropertyS { if ( nInstance >= EPP_TEXTTYPE_CenterBody ) { - bFirst = false; bSimpleText = true; mpStrm->WriteUInt16( nLev ); } - mpStyleSheet->mpParaSheet[ nInstance ]->Write( *mpStrm, mpPptEscherEx, nLev, bFirst, bSimpleText, mXPagePropSet ); - mpStyleSheet->mpCharSheet[ nInstance ]->Write( *mpStrm, mpPptEscherEx, nLev, bFirst, bSimpleText, mXPagePropSet ); - bFirst = false; + mpStyleSheet->mpParaSheet[ nInstance ]->Write( *mpStrm, nLev, bSimpleText, mXPagePropSet ); + mpStyleSheet->mpCharSheet[ nInstance ]->Write( *mpStrm, nLev, bSimpleText, mXPagePropSet ); } mpPptEscherEx->EndAtom( EPP_TxMasterStyleAtom, 0, nInstance ); } diff --git a/sd/source/filter/eppt/eppt.hxx b/sd/source/filter/eppt/eppt.hxx index 90c8ee5f490b..ddba77f1b970 100644 --- a/sd/source/filter/eppt/eppt.hxx +++ b/sd/source/filter/eppt/eppt.hxx @@ -224,7 +224,7 @@ class PPTWriter : public PPTWriterBase, public PPTExBulletProvider void ImplFlipBoundingBox( EscherPropertyContainer& rPropOpt ); bool ImplGetText(); - bool ImplCreatePresentationPlaceholder( const bool bMaster, const PageType PageType, + bool ImplCreatePresentationPlaceholder( const bool bMaster, const sal_uInt32 StyleInstance, const sal_uInt8 PlaceHolderId ); static bool ImplGetEffect( const css::uno::Reference< css::beans::XPropertySet > &, css::presentation::AnimationEffect& eEffect, diff --git a/sd/source/filter/eppt/epptbase.hxx b/sd/source/filter/eppt/epptbase.hxx index 864fd4b9d3c0..cd5c58609c42 100644 --- a/sd/source/filter/eppt/epptbase.hxx +++ b/sd/source/filter/eppt/epptbase.hxx @@ -245,7 +245,7 @@ struct PPTExCharSheet void SetStyleSheet( const css::uno::Reference< css::beans::XPropertySet > &, FontCollection& rFontCollection, int nLevel ); - void Write( SvStream& rSt, PptEscherEx* pEx, sal_uInt16 nLev, bool bFirst, bool bSimpleText, + void Write( SvStream& rSt, sal_uInt16 nLev, bool bSimpleText, const css::uno::Reference< css::beans::XPropertySet > & rPagePropSet ); }; @@ -286,7 +286,7 @@ struct PPTExParaSheet void SetStyleSheet( const css::uno::Reference< css::beans::XPropertySet > &, FontCollection& rFontCollection, int nLevel, const PPTExCharLevel& rCharLevel ); - void Write( SvStream& rSt, PptEscherEx* pEx, sal_uInt16 nLev, bool bFirst, bool bSimpleText, + void Write( SvStream& rSt, sal_uInt16 nLev, bool bSimpleText, const css::uno::Reference< css::beans::XPropertySet > & rPagePropSet ); }; diff --git a/sd/source/filter/eppt/epptso.cxx b/sd/source/filter/eppt/epptso.cxx index 6ec29c413fdb..d4314994741d 100644 --- a/sd/source/filter/eppt/epptso.cxx +++ b/sd/source/filter/eppt/epptso.cxx @@ -407,12 +407,10 @@ bool PPTWriter::ImplCloseDocument() EscherExAtom aTxMasterStyleAtom( aTxMasterStyleAtomStrm, EPP_TxMasterStyleAtom, EPP_TEXTTYPE_Other ); aTxMasterStyleAtomStrm.WriteUInt16( 5 ); // paragraph count sal_uInt16 nLev; - bool bFirst = true; for ( nLev = 0; nLev < 5; nLev++ ) { - mpStyleSheet->mpParaSheet[ EPP_TEXTTYPE_Other ]->Write( aTxMasterStyleAtomStrm, mpPptEscherEx, nLev, bFirst, false, mXPagePropSet ); - mpStyleSheet->mpCharSheet[ EPP_TEXTTYPE_Other ]->Write( aTxMasterStyleAtomStrm, mpPptEscherEx, nLev, bFirst, false, mXPagePropSet ); - bFirst = false; + mpStyleSheet->mpParaSheet[ EPP_TEXTTYPE_Other ]->Write( aTxMasterStyleAtomStrm, nLev, false, mXPagePropSet ); + mpStyleSheet->mpCharSheet[ EPP_TEXTTYPE_Other ]->Write( aTxMasterStyleAtomStrm, nLev, false, mXPagePropSet ); } } @@ -1564,7 +1562,7 @@ bool PPTWriter::ImplGetEffect( const css::uno::Reference< css::beans::XPropertyS return bHasEffect; }; -bool PPTWriter::ImplCreatePresentationPlaceholder( const bool bMasterPage, const PageType /* ePageType */, +bool PPTWriter::ImplCreatePresentationPlaceholder( const bool bMasterPage, const sal_uInt32 nStyleInstance, const sal_uInt8 nPlaceHolderId ) { bool bRet = ImplGetText(); @@ -2228,7 +2226,7 @@ void PPTWriter::ImplWritePage( const PHLayout& rLayout, EscherSolverContainer& a { if ( ( ePageType == NOTICE ) && mbPresObj ) { - if ( ImplCreatePresentationPlaceholder( bMasterPage, ePageType, EPP_TEXTTYPE_Notes, EPP_PLACEHOLDER_MASTERNOTESBODYIMAGE ) ) + if ( ImplCreatePresentationPlaceholder( bMasterPage, EPP_TEXTTYPE_Notes, EPP_PLACEHOLDER_MASTERNOTESBODYIMAGE ) ) continue; else nPlaceHolderAtom = EPP_PLACEHOLDER_NOTESBODY; @@ -2448,7 +2446,7 @@ void PPTWriter::ImplWritePage( const PHLayout& rLayout, EscherSolverContainer& a { if ( ( ePageType == NOTICE ) && mbPresObj ) { - if ( ImplCreatePresentationPlaceholder( bMasterPage, ePageType, EPP_TEXTTYPE_Notes, EPP_PLACEHOLDER_MASTERNOTESSLIDEIMAGE ) ) + if ( ImplCreatePresentationPlaceholder( bMasterPage, EPP_TEXTTYPE_Notes, EPP_PLACEHOLDER_MASTERNOTESSLIDEIMAGE ) ) continue; else nPlaceHolderAtom = EPP_PLACEHOLDER_NOTESSLIDEIMAGE; @@ -2533,7 +2531,7 @@ void PPTWriter::ImplWritePage( const PHLayout& rLayout, EscherSolverContainer& a } else if ( mType == "presentation.Header" ) { - if ( ImplCreatePresentationPlaceholder( bMasterPage, ePageType, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERHEADER ) ) + if ( ImplCreatePresentationPlaceholder( bMasterPage, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERHEADER ) ) continue; else { @@ -2544,7 +2542,7 @@ void PPTWriter::ImplWritePage( const PHLayout& rLayout, EscherSolverContainer& a } else if ( mType == "presentation.Footer" ) { - if ( ImplCreatePresentationPlaceholder( bMasterPage, ePageType, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERFOOTER ) ) + if ( ImplCreatePresentationPlaceholder( bMasterPage, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERFOOTER ) ) continue; else { @@ -2555,7 +2553,7 @@ void PPTWriter::ImplWritePage( const PHLayout& rLayout, EscherSolverContainer& a } else if ( mType == "presentation.DateTime" ) { - if ( ImplCreatePresentationPlaceholder( bMasterPage, ePageType, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERDATE ) ) + if ( ImplCreatePresentationPlaceholder( bMasterPage, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERDATE ) ) continue; else { @@ -2566,7 +2564,7 @@ void PPTWriter::ImplWritePage( const PHLayout& rLayout, EscherSolverContainer& a } else if ( mType == "presentation.SlideNumber" ) { - if ( ImplCreatePresentationPlaceholder( bMasterPage, ePageType, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERSLIDENUMBER ) ) + if ( ImplCreatePresentationPlaceholder( bMasterPage, EPP_TEXTTYPE_Other, EPP_PLACEHOLDER_MASTERSLIDENUMBER ) ) continue; else { diff --git a/sd/source/filter/eppt/pptexanimations.cxx b/sd/source/filter/eppt/pptexanimations.cxx index c9e94b96a484..58fd59517518 100644 --- a/sd/source/filter/eppt/pptexanimations.cxx +++ b/sd/source/filter/eppt/pptexanimations.cxx @@ -335,7 +335,7 @@ void AnimationExporter::doexport( const Reference< XDrawPage >& xPage, SvStream& if( xRootNode.is() ) { processAfterEffectNodes( xRootNode ); - exportNode( rStrm, xRootNode, nullptr, DFF_msofbtAnimGroup, 1, 0, false, AnimationFill::AUTO ); + exportNode( rStrm, xRootNode, DFF_msofbtAnimGroup, 1, 0, false, AnimationFill::AUTO ); } } } @@ -474,7 +474,7 @@ bool AnimationExporter::isEmptyNode( const Reference< XAnimationNode >& xNode ) return true; } -void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > const & xNode_in, const Reference< XAnimationNode >* pParent, const sal_uInt16 nContainerRecType, +void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > const & xNode_in, const sal_uInt16 nContainerRecType, const sal_uInt16 nInstance, const sal_Int32 nGroupLevel, const bool bTakeBackInteractiveSequenceTiming, const sal_Int16 nFDef ) { auto xNode = xNode_in; @@ -502,7 +502,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > { case AnimationNodeType::CUSTOM : { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); exportAnimEvent( rStrm, xNode ); exportAnimValue( rStrm, xNode, false ); @@ -511,7 +511,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > case AnimationNodeType::PAR : { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); sal_Int32 nFlags = nGroupLevel == 2 ? 0x10 : 0; if ( bTakeBackInteractiveSequenceTiming ) @@ -523,7 +523,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > case AnimationNodeType::SEQ : { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); sal_Int16 nNodeType = exportAnimPropertySet( rStrm, xNode ); sal_Int32 nFlags = 12; if ( ( nGroupLevel == 1 ) && ( nNodeType == css::presentation::EffectNodeType::INTERACTIVE_SEQUENCE ) ) @@ -575,7 +575,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > case AnimationNodeType::ANIMATE : { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); exportAnimEvent( rStrm, xNode ); exportAnimValue( rStrm, xNode, false ); @@ -588,7 +588,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > bool bIsAfterEffectNode( isAfterEffectNode( xNode ) ); if( (nGroupLevel != 4) || !bIsAfterEffectNode ) { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); exportAnimateSet( rStrm, xNode, bIsAfterEffectNode ? AFTEREFFECT_SET : AFTEREFFECT_NONE ); exportAnimEvent( rStrm, xNode ); @@ -603,7 +603,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > case AnimationNodeType::ANIMATEMOTION : { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); exportAnimateMotion( rStrm, xNode ); exportAnimEvent( rStrm, xNode ); @@ -619,7 +619,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > if( bIsAfterEffectNode ) xNode = createAfterEffectNodeClone( xNode ); - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); exportAnimateColor( rStrm, xNode, bIsAfterEffectNode ? AFTEREFFECT_COLOR : AFTEREFFECT_NONE ); exportAnimEvent( rStrm, xNode ); @@ -634,7 +634,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > case AnimationNodeType::ANIMATETRANSFORM : { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); exportAnimateTransform( rStrm, xNode ); exportAnimEvent( rStrm, xNode ); @@ -644,7 +644,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > case AnimationNodeType::TRANSITIONFILTER : { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); exportAnimEvent( rStrm, xNode ); exportAnimValue( rStrm, xNode, false ); @@ -654,7 +654,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > case AnimationNodeType::AUDIO : // #i58428# { - exportAnimNode( rStrm, xNode, pParent, nGroupLevel, nFillDefault ); + exportAnimNode( rStrm, xNode, nFillDefault ); exportAnimPropertySet( rStrm, xNode ); Reference< XAudio > xAudio( xNode, UNO_QUERY ); @@ -709,7 +709,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > Reference< XAnimationNode > xAfterEffectNode; if( hasAfterEffectNode( xNode, xAfterEffectNode ) ) { - exportNode( rStrm, xAfterEffectNode, &xNode, DFF_msofbtAnimSubGoup, 1, nGroupLevel + 1, bTakeBackInteractiveSequenceTimingForChild, nFillDefault ); + exportNode( rStrm, xAfterEffectNode, DFF_msofbtAnimSubGoup, 1, nGroupLevel + 1, bTakeBackInteractiveSequenceTimingForChild, nFillDefault ); } Reference< XEnumerationAccess > xEnumerationAccess( xNode, UNO_QUERY ); @@ -729,7 +729,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > nAudioGroup = mnCurrentGroup; } else - exportNode( rStrm, xChildNode, &xNode, DFF_msofbtAnimGroup, 1, nGroupLevel + 1, bTakeBackInteractiveSequenceTimingForChild, nFillDefault ); + exportNode( rStrm, xChildNode, DFF_msofbtAnimGroup, 1, nGroupLevel + 1, bTakeBackInteractiveSequenceTimingForChild, nFillDefault ); } } } @@ -737,7 +737,7 @@ void AnimationExporter::exportNode( SvStream& rStrm, Reference< XAnimationNode > } } if ( xAudioNode.is() ) - exportNode( rStrm, xAudioNode, &xNode, DFF_msofbtAnimGroup, 1, nGroupLevel, bTakeBackInteractiveSequenceTimingForChild, nFillDefault ); + exportNode( rStrm, xAudioNode, DFF_msofbtAnimGroup, 1, nGroupLevel, bTakeBackInteractiveSequenceTimingForChild, nFillDefault ); if( xNode->getType() == AnimationNodeType::ITERATE ) aTarget = Any(); @@ -784,7 +784,7 @@ bool AnimationExporter::GetNodeType( const Reference< XAnimationNode >& xNode, s } void AnimationExporter::exportAnimNode( SvStream& rStrm, const Reference< XAnimationNode >& xNode, - const css::uno::Reference< css::animations::XAnimationNode >*, const sal_Int32, const sal_Int16 nFillDefault ) + const sal_Int16 nFillDefault ) { EscherExAtom aAnimNodeExAtom( rStrm, DFF_msofbtAnimNode ); AnimationNode aAnim; @@ -1055,7 +1055,7 @@ sal_Int16 AnimationExporter::exportAnimPropertySet( SvStream& rStrm, const Refer { bool bAfterEffect = false; if ( *pAny[ DFF_ANIM_AFTEREFFECT ] >>= bAfterEffect ) - exportAnimPropertyByte( rStrm, DFF_ANIM_AFTEREFFECT, int(bAfterEffect), TRANSLATE_NONE ); + exportAnimPropertyByte( rStrm, DFF_ANIM_AFTEREFFECT, int(bAfterEffect) ); } if ( pAny[ DFF_ANIM_RUNTIMECONTEXT ] ) @@ -1145,7 +1145,7 @@ bool AnimationExporter::exportAnimProperty( SvStream& rStrm, const sal_uInt16 nP double fVal = 0.0; if ( rAny >>= fVal ) { - exportAnimPropertyFloat( rStrm, nPropertyId, fVal, eTranslateMode ); + exportAnimPropertyFloat( rStrm, nPropertyId, fVal ); bRet = true; } } @@ -1162,7 +1162,7 @@ bool AnimationExporter::exportAnimProperty( SvStream& rStrm, const sal_uInt16 nP } else { - exportAnimPropertyFloat( rStrm, nPropertyId, fVal, eTranslateMode ); + exportAnimPropertyFloat( rStrm, nPropertyId, fVal ); bRet = true; } } @@ -1195,7 +1195,7 @@ void AnimationExporter::exportAnimPropertyString( SvStream& rStrm, const sal_uIn writeZString( rStrm, aStr ); } -void AnimationExporter::exportAnimPropertyFloat( SvStream& rStrm, const sal_uInt16 nPropertyId, const double& rVal, const TranslateMode ) +void AnimationExporter::exportAnimPropertyFloat( SvStream& rStrm, const sal_uInt16 nPropertyId, const double& rVal ) { EscherExAtom aExAtom( rStrm, DFF_msofbtAnimAttributeValue, nPropertyId ); sal_uInt8 nType = DFF_ANIM_PROP_TYPE_FLOAT; @@ -1212,7 +1212,7 @@ void AnimationExporter::exportAnimPropertyuInt32( SvStream& rStrm, const sal_uIn .WriteUInt32( nVal ); } -void AnimationExporter::exportAnimPropertyByte( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt8 nVal, const TranslateMode ) +void AnimationExporter::exportAnimPropertyByte( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt8 nVal ) { EscherExAtom aExAtom( rStrm, DFF_msofbtAnimAttributeValue, nPropertyId ); sal_uInt8 nType = DFF_ANIM_PROP_TYPE_BYTE; diff --git a/sd/source/filter/eppt/pptexanimations.hxx b/sd/source/filter/eppt/pptexanimations.hxx index 81eace5c933c..d9e38b030534 100644 --- a/sd/source/filter/eppt/pptexanimations.hxx +++ b/sd/source/filter/eppt/pptexanimations.hxx @@ -72,24 +72,23 @@ class AnimationExporter static bool getColorAny( const css::uno::Any& rAny, const sal_Int16 nColorSpace, sal_Int32& rMode, sal_Int32& rA, sal_Int32& rB, sal_Int32& rC ); static bool exportAnimProperty( SvStream& rStrm, const sal_uInt16 nPropertyId, const css::uno::Any& rAny, const TranslateMode eTranslateMode ); static void exportAnimPropertyString( SvStream& rStrm, const sal_uInt16 nPropertyId, const OUString& rVal, const TranslateMode eTranslateMode ); - static void exportAnimPropertyFloat( SvStream& rStrm, const sal_uInt16 nPropertyId, const double& rVal, const TranslateMode eTranslateMode ); + static void exportAnimPropertyFloat( SvStream& rStrm, const sal_uInt16 nPropertyId, const double& rVal ); static void exportAnimPropertyuInt32( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt32 nVal, const TranslateMode eTranslateMode ); - static void exportAnimPropertyByte( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt8 nVal, const TranslateMode eTranslateMode ); + static void exportAnimPropertyByte( SvStream& rStrm, const sal_uInt16 nPropertyId, const sal_uInt8 nVal ); /** if available exportAnimPropertySet @return the css::presentation::EffectNodeType*/ static sal_Int16 exportAnimPropertySet( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode ); static void exportAnimNode( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode, - const css::uno::Reference< css::animations::XAnimationNode >* pParent, const sal_Int32 nGroupLevel, const sal_Int16 nFillDefault ); + const sal_Int16 nFillDefault ); void exportAnimate( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode ); void exportAnimateTarget( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode, const sal_uInt32 nForceAttributeName = 0, int nAfterEffectType = AFTEREFFECT_NONE ); void exportAnimateSet( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode, int nAfterEffectType ); static void exportAnimAction( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode ); void exportAnimEvent( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode, const sal_Int32 nFlags = 0 ); void exportNode( SvStream& rStrm, css::uno::Reference< css::animations::XAnimationNode > const & xNode, - const css::uno::Reference< css::animations::XAnimationNode >* xParent, - const sal_uInt16 nContainerRecType, const sal_uInt16 nInstance, const sal_Int32 nGroupLevel, const bool bTakeBackInteractiveSequenceTiming, - const sal_Int16 nFillDefault ); + const sal_uInt16 nContainerRecType, const sal_uInt16 nInstance, const sal_Int32 nGroupLevel, const bool bTakeBackInteractiveSequenceTiming, + const sal_Int16 nFillDefault ); void exportAnimateTargetElement( SvStream& rStrm, const css::uno::Any& rAny, const bool bCreate2b01Atom ); static void exportAnimateKeyPoints( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimate >& xAnimate ); static void exportAnimValue( SvStream& rStrm, const css::uno::Reference< css::animations::XAnimationNode >& xNode, const bool bExportAlways ); diff --git a/sd/source/filter/eppt/pptx-stylesheet.cxx b/sd/source/filter/eppt/pptx-stylesheet.cxx index 065b723dc860..6a739344dd40 100644 --- a/sd/source/filter/eppt/pptx-stylesheet.cxx +++ b/sd/source/filter/eppt/pptx-stylesheet.cxx @@ -89,7 +89,7 @@ void PPTExCharSheet::SetStyleSheet( const css::uno::Reference< css::beans::XProp rLev.mnFlags = aPortionObj.mnCharAttr; } -void PPTExCharSheet::Write( SvStream& rSt, PptEscherEx*, sal_uInt16 nLev, bool, bool bSimpleText, +void PPTExCharSheet::Write( SvStream& rSt, sal_uInt16 nLev, bool bSimpleText, const css::uno::Reference< css::beans::XPropertySet > & rPagePropSet ) { const PPTExCharLevel& rLev = maCharLevel[ nLev ]; @@ -312,7 +312,7 @@ void PPTExParaSheet::SetStyleSheet( const css::uno::Reference< css::beans::XProp } } -void PPTExParaSheet::Write( SvStream& rSt, PptEscherEx*, sal_uInt16 nLev, bool, bool bSimpleText, +void PPTExParaSheet::Write( SvStream& rSt, sal_uInt16 nLev, bool bSimpleText, const css::uno::Reference< css::beans::XPropertySet > & rPagePropSet ) { const PPTExParaLevel& rLev = maParaLevel[ nLev ]; diff --git a/sd/source/filter/html/htmlex.cxx b/sd/source/filter/html/htmlex.cxx index 413c85ac866f..275b50fe5da0 100644 --- a/sd/source/filter/html/htmlex.cxx +++ b/sd/source/filter/html/htmlex.cxx @@ -674,7 +674,7 @@ void HtmlExport::ExportSingleDocument() if(mbNotes) { SdPage* pNotesPage = maNotesPages[ nSdPage ]; - OUString aNotesStr( CreateTextForNotesPage( pOutliner, pNotesPage, true, maBackColor) ); + OUString aNotesStr( CreateTextForNotesPage( pOutliner, pNotesPage, maBackColor) ); if (!aNotesStr.isEmpty()) { @@ -1127,7 +1127,7 @@ bool HtmlExport::CreateHtmlTextForPresPages() if(mbNotes) { SdPage* pNotesPage = maNotesPages[ nSdPage ]; - OUString aNotesStr( CreateTextForNotesPage( pOutliner, pNotesPage, true, maBackColor) ); + OUString aNotesStr( CreateTextForNotesPage( pOutliner, pNotesPage, maBackColor) ); if (!aNotesStr.isEmpty()) { @@ -1377,7 +1377,6 @@ void HtmlExport::WriteOutlinerParagraph(OUStringBuffer& aStr, SdrOutliner* pOutl // creates a outliner text for a note page OUString HtmlExport::CreateTextForNotesPage( SdrOutliner* pOutliner, SdPage* pPage, - bool, const Color& rBackgroundColor ) { OUStringBuffer aStr; @@ -1662,7 +1661,7 @@ bool HtmlExport::CreateHtmlForPresPages() { SdrOutliner* pOutliner = mpDoc->GetInternalOutliner(); SdPage* pNotesPage = maNotesPages[ nSdPage ]; - OUString aNotesStr( CreateTextForNotesPage( pOutliner, pNotesPage, true, maBackColor) ); + OUString aNotesStr( CreateTextForNotesPage( pOutliner, pNotesPage, maBackColor) ); pOutliner->Clear(); if (!aNotesStr.isEmpty()) @@ -2065,7 +2064,7 @@ bool HtmlExport::CreateNotesPages() aStr.append(CreateBodyTag()); if(pPage) - aStr.append(CreateTextForNotesPage( pOutliner, pPage, true, maBackColor )); + aStr.append(CreateTextForNotesPage( pOutliner, pPage, maBackColor )); aStr.append("\r\n"); diff --git a/sd/source/filter/html/htmlex.hxx b/sd/source/filter/html/htmlex.hxx index e7e5ff6d9d96..754aeb8e5787 100644 --- a/sd/source/filter/html/htmlex.hxx +++ b/sd/source/filter/html/htmlex.hxx @@ -180,7 +180,7 @@ class HtmlExport final OUString CreateTextForTitle( SdrOutliner* pOutliner, SdPage* pPage, const Color& rBackgroundColor ); OUString CreateTextForPage( SdrOutliner* pOutliner, SdPage* pPage, bool bHeadLine, const Color& rBackgroundColor ); - OUString CreateTextForNotesPage( SdrOutliner* pOutliner, SdPage* pPage, bool bHeadLine, const Color& rBackgroundColor ); + OUString CreateTextForNotesPage( SdrOutliner* pOutliner, SdPage* pPage, const Color& rBackgroundColor ); static OUString CreateHTMLCircleArea( sal_uLong nRadius, sal_uLong nCenterX, sal_uLong nCenterY, const OUString& rHRef ); diff --git a/sd/source/filter/ppt/pptin.cxx b/sd/source/filter/ppt/pptin.cxx index 88c469243d76..b11bbd2cb187 100644 --- a/sd/source/filter/ppt/pptin.cxx +++ b/sd/source/filter/ppt/pptin.cxx @@ -2422,7 +2422,7 @@ SdrObject* ImplSdPPTImport::ApplyTextObj( PPTTextObj* pTextObj, SdrTextObj* pObj } else { - SdrObject* pPresObj = pPage->CreatePresObj( ePresObjKind, bVertical, pText->GetLogicRect(), true ); + SdrObject* pPresObj = pPage->CreatePresObj( ePresObjKind, bVertical, pText->GetLogicRect() ); pPresObj->SetUserCall( pPage ); SfxItemSet aSet( pSdrModel->GetItemPool() ); diff --git a/sd/source/ui/app/optsitem.cxx b/sd/source/ui/app/optsitem.cxx index 8848dddce805..cdbcacd330e4 100644 --- a/sd/source/ui/app/optsitem.cxx +++ b/sd/source/ui/app/optsitem.cxx @@ -379,7 +379,7 @@ bool SdOptionsContents::WriteData( Any* pValues ) const |* \************************************************************************/ -SdOptionsContentsItem::SdOptionsContentsItem( SdOptions*, ::sd::FrameView*) +SdOptionsContentsItem::SdOptionsContentsItem() : SfxPoolItem ( ATTR_OPTIONS_CONTENTS ) , maOptionsContents ( 0, false ) { diff --git a/sd/source/ui/app/sdmod.cxx b/sd/source/ui/app/sdmod.cxx index 27ec957c04f5..689ba7569b15 100644 --- a/sd/source/ui/app/sdmod.cxx +++ b/sd/source/ui/app/sdmod.cxx @@ -210,14 +210,6 @@ SvNumberFormatter* SdModule::GetNumberFormatter() return pNumberFormatter; } -/** This method is deprecated and only an alias to - * GetVirtualRefDevice(). The given argument is ignored. - */ -OutputDevice* SdModule::GetRefDevice (::sd::DrawDocShell& ) -{ - return GetVirtualRefDevice(); -} - svtools::ColorConfig& SdModule::GetColorConfig() { return *mpColorConfig; diff --git a/sd/source/ui/app/sdmod2.cxx b/sd/source/ui/app/sdmod2.cxx index ce4dd0dfdc9f..c19ae34d1919 100644 --- a/sd/source/ui/app/sdmod2.cxx +++ b/sd/source/ui/app/sdmod2.cxx @@ -479,7 +479,7 @@ SfxItemSet* SdModule::CreateItemSet( sal_uInt16 nSlot ) pRet->Put( SfxUInt16Item( SID_ATTR_METRIC, (sal_uInt16)nMetric ) ); // TP_OPTIONS_CONTENTS: - pRet->Put( SdOptionsContentsItem( pOptions, pFrameView ) ); + pRet->Put( SdOptionsContentsItem() ); // TP_OPTIONS_MISC: SdOptionsMiscItem aSdOptionsMiscItem( pOptions, pFrameView ); diff --git a/sd/source/ui/dlg/docprev.cxx b/sd/source/ui/dlg/docprev.cxx index 84cce2aa79e7..919867d9cd6d 100644 --- a/sd/source/ui/dlg/docprev.cxx +++ b/sd/source/ui/dlg/docprev.cxx @@ -126,7 +126,7 @@ void SdDocPreviewWin::ImpPaint( OutputDevice* pVDev ) pVDev->DrawRect(::tools::Rectangle( Point(0,0 ), pVDev->GetOutputSize())); } -void SdDocPreviewWin::Paint( vcl::RenderContext& /*rRenderContext*/, const ::tools::Rectangle& rRect ) +void SdDocPreviewWin::Paint( vcl::RenderContext& /*rRenderContext*/, const ::tools::Rectangle& /*rRect*/ ) { if( (!mxSlideShow.is()) || (!mxSlideShow->isRunning() ) ) { @@ -140,7 +140,7 @@ void SdDocPreviewWin::Paint( vcl::RenderContext& /*rRenderContext*/, const ::too } else { - mxSlideShow->paint( rRect ); + mxSlideShow->paint(); } } diff --git a/sd/source/ui/dlg/sdtreelb.cxx b/sd/source/ui/dlg/sdtreelb.cxx index 073185fbecda..6e5a7a14a722 100644 --- a/sd/source/ui/dlg/sdtreelb.cxx +++ b/sd/source/ui/dlg/sdtreelb.cxx @@ -136,7 +136,7 @@ bool SdPageObjsTLB::SdPageObjsTransferable::GetData( const css::datatransfer::Da void SdPageObjsTLB::SdPageObjsTransferable::DragFinished( sal_Int8 nDropAction ) { - mrParent.OnDragFinished( nDropAction ); + mrParent.OnDragFinished(); SdTransferable::DragFinished(nDropAction); } @@ -1238,7 +1238,7 @@ void SdPageObjsTLB::DoDrag() } } -void SdPageObjsTLB::OnDragFinished( sal_uInt8 ) +void SdPageObjsTLB::OnDragFinished() { if( mpFrame->HasChildWindow( SID_NAVIGATOR ) ) { diff --git a/sd/source/ui/framework/factories/BasicViewFactory.cxx b/sd/source/ui/framework/factories/BasicViewFactory.cxx index 0eace0ed525d..a537cb6a5574 100644 --- a/sd/source/ui/framework/factories/BasicViewFactory.cxx +++ b/sd/source/ui/framework/factories/BasicViewFactory.cxx @@ -333,7 +333,6 @@ std::shared_ptr BasicViewFactory::CreateViewShell ( { pViewShell.reset( new DrawViewShell( - &rFrame, *mpBase, &rWindow, PageKind::Standard, @@ -344,7 +343,6 @@ std::shared_ptr BasicViewFactory::CreateViewShell ( { pViewShell.reset( new GraphicViewShell ( - &rFrame, *mpBase, &rWindow, pFrameView)); @@ -364,7 +362,6 @@ std::shared_ptr BasicViewFactory::CreateViewShell ( { pViewShell.reset( new DrawViewShell( - &rFrame, *mpBase, &rWindow, PageKind::Notes, @@ -375,7 +372,6 @@ std::shared_ptr BasicViewFactory::CreateViewShell ( { pViewShell.reset( new DrawViewShell( - &rFrame, *mpBase, &rWindow, PageKind::Handout, @@ -386,7 +382,6 @@ std::shared_ptr BasicViewFactory::CreateViewShell ( { pViewShell.reset( new PresentationViewShell( - &rFrame, *mpBase, &rWindow, pFrameView)); diff --git a/sd/source/ui/func/fuexpand.cxx b/sd/source/ui/func/fuexpand.cxx index 0feb4fd8cbb9..f49a3885a844 100644 --- a/sd/source/ui/func/fuexpand.cxx +++ b/sd/source/ui/func/fuexpand.cxx @@ -94,7 +94,7 @@ void FuExpandPage::DoExecute( SfxRequest& ) pOutl->EnableUndo(false); if (mpDocSh) - pOutl->SetRefDevice( SD_MOD()->GetRefDevice( *mpDocSh ) ); + pOutl->SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); pOutl->SetDefTab( mpDoc->GetDefaultTabulator() ); pOutl->SetStyleSheetPool(static_cast(mpDoc->GetStyleSheetPool())); diff --git a/sd/source/ui/func/fuinsfil.cxx b/sd/source/ui/func/fuinsfil.cxx index d2228597d673..d4ace7dc8e36 100644 --- a/sd/source/ui/func/fuinsfil.cxx +++ b/sd/source/ui/func/fuinsfil.cxx @@ -402,7 +402,7 @@ bool FuInsertFile::InsSDDinDrMode(SfxMedium* pMedium) if( bNameOK ) bOK = mpDoc->InsertBookmarkAsObject( aObjectBookmarkList, aExchangeList, - bLink, nullptr, nullptr, false ); + nullptr, nullptr, false ); if( pDlg->IsRemoveUnnessesaryMasterPages() ) mpDoc->RemoveUnnecessaryMasterPages(); @@ -442,7 +442,7 @@ void FuInsertFile::InsTextOrRTFinDrMode(SfxMedium* pMedium) std::unique_ptr pOutliner(new SdOutliner( mpDoc, OutlinerMode::TextObject )); // set reference device - pOutliner->SetRefDevice( SD_MOD()->GetRefDevice( *mpDocSh ) ); + pOutliner->SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); SdPage* pPage = static_cast(mpViewShell)->GetActualPage(); aLayoutName = pPage->GetLayoutName(); @@ -593,7 +593,7 @@ void FuInsertFile::InsTextOrRTFinOlMode(SfxMedium* pMedium) pOutliner->SetStyleSheetPool(static_cast(mpDoc->GetStyleSheetPool())); // set reference device - pOutliner->SetRefDevice(SD_MOD()->GetRefDevice( *mpDocSh )); + pOutliner->SetRefDevice(SD_MOD()->GetVirtualRefDevice()); pOutliner->SetPaperSize(Size(0x7fffffff, 0x7fffffff)); SvStream* pStream = pMedium->GetInStream(); diff --git a/sd/source/ui/func/fupoor.cxx b/sd/source/ui/func/fupoor.cxx index 2771a542fe6e..8fa2896794ef 100644 --- a/sd/source/ui/func/fupoor.cxx +++ b/sd/source/ui/func/fupoor.cxx @@ -901,7 +901,7 @@ void FuPoor::DoCut() { if (mpView) { - mpView->DoCut(mpWindow); + mpView->DoCut(); } } @@ -912,7 +912,7 @@ void FuPoor::DoCopy() { if (mpView) { - mpView->DoCopy(mpWindow); + mpView->DoCopy(); } } diff --git a/sd/source/ui/func/fusumry.cxx b/sd/source/ui/func/fusumry.cxx index 8417c3c94ae6..e6a63b0f654e 100644 --- a/sd/source/ui/func/fusumry.cxx +++ b/sd/source/ui/func/fusumry.cxx @@ -165,7 +165,7 @@ void FuSummaryPage::DoExecute( SfxRequest& ) pOutl->EnableUndo(false); if (mpDocSh) - pOutl->SetRefDevice(SD_MOD()->GetRefDevice( *mpDocSh )); + pOutl->SetRefDevice(SD_MOD()->GetVirtualRefDevice()); pOutl->SetDefTab( mpDoc->GetDefaultTabulator() ); pOutl->SetStyleSheetPool(static_cast(mpDoc->GetStyleSheetPool())); diff --git a/sd/source/ui/inc/DrawViewShell.hxx b/sd/source/ui/inc/DrawViewShell.hxx index 6178bde590b8..d42b514753d3 100644 --- a/sd/source/ui/inc/DrawViewShell.hxx +++ b/sd/source/ui/inc/DrawViewShell.hxx @@ -88,7 +88,6 @@ public: one view shell to the next. */ DrawViewShell ( - SfxViewFrame* pFrame, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, PageKind ePageKind, @@ -279,7 +278,7 @@ public: */ virtual void ShowUIControls (bool bVisible) override; - void ScannerEvent( const css::lang::EventObject& rEventObject ); + void ScannerEvent(); bool IsLayerModeActive() const { return mbIsLayerModeActive;} @@ -449,21 +448,9 @@ private: virtual void Notify (SfxBroadcaster& rBC, const SfxHint& rHint) override; - /** Stop a running slide show. The frame the show is running in is - destroyed if - a) it is running in its own frame, i.e. is a full screen show and - b) the given flag bCloseFrame is true. - @param bCloseFrame - Be careful with this flag when stopping a full screen show. - When called from the destructor the flag has to be or - otherwise we run into a loop of calls to destructors of the view - and the frame. - When called from other places the flag should be so that - not an empty frame remains. When called with it is the - responsibility of the caller to avoid an illegal reentrant - call. + /** Stop a running slide show. */ - void StopSlideShow (bool bCloseFrame); + void StopSlideShow(); /** Show the context menu for snap lines and points. Because snap lines can not be selected the index of the snap line/point for which the diff --git a/sd/source/ui/inc/FrameView.hxx b/sd/source/ui/inc/FrameView.hxx index 15950ba61fe4..4ae22cc04c2d 100644 --- a/sd/source/ui/inc/FrameView.hxx +++ b/sd/source/ui/inc/FrameView.hxx @@ -165,8 +165,8 @@ public: void SetIsNavigatorShowingAllShapes (const bool bIsNavigatorShowingAllShapes); bool IsNavigatorShowingAllShapes() const { return mbIsNavigatorShowingAllShapes;} - void WriteUserDataSequence ( css::uno::Sequence < css::beans::PropertyValue >&, bool bBrowse ); - void ReadUserDataSequence ( const css::uno::Sequence < css::beans::PropertyValue >&, bool bBrowse ); + void WriteUserDataSequence ( css::uno::Sequence < css::beans::PropertyValue >& ); + void ReadUserDataSequence ( const css::uno::Sequence < css::beans::PropertyValue >& ); private: sal_uInt16 mnRefCount; diff --git a/sd/source/ui/inc/GraphicViewShell.hxx b/sd/source/ui/inc/GraphicViewShell.hxx index 5830ae0dcfcb..34b858df94b8 100644 --- a/sd/source/ui/inc/GraphicViewShell.hxx +++ b/sd/source/ui/inc/GraphicViewShell.hxx @@ -56,7 +56,6 @@ public: one view shell to the next. */ GraphicViewShell ( - SfxViewFrame* pFrame, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameView); diff --git a/sd/source/ui/inc/OutlineView.hxx b/sd/source/ui/inc/OutlineView.hxx index 221ea9396ddf..27a17bb07a81 100644 --- a/sd/source/ui/inc/OutlineView.hxx +++ b/sd/source/ui/inc/OutlineView.hxx @@ -110,7 +110,7 @@ public: sal_uLong GetPaperWidth() { return mnPaperWidth;} - bool PrepareClose(bool bUI = true); + bool PrepareClose(); virtual void GetAttributes( SfxItemSet& rTargetSet, bool bOnlyHardAttr = false ) const override; virtual bool SetAttributes(const SfxItemSet& rSet, bool bReplaceAll = false) override; diff --git a/sd/source/ui/inc/OutlineViewShell.hxx b/sd/source/ui/inc/OutlineViewShell.hxx index dc79fc1739e7..7284d36a03d5 100644 --- a/sd/source/ui/inc/OutlineViewShell.hxx +++ b/sd/source/ui/inc/OutlineViewShell.hxx @@ -155,7 +155,7 @@ private: bool bPastePossible; bool mbInitialized; - void Construct (DrawDocShell* pDocSh); + void Construct(); DECL_LINK( ClipboardChanged, TransferableDataHelper*, void ); }; diff --git a/sd/source/ui/inc/PresentationViewShell.hxx b/sd/source/ui/inc/PresentationViewShell.hxx index 6a4f33e28ebe..c0020685c02e 100644 --- a/sd/source/ui/inc/PresentationViewShell.hxx +++ b/sd/source/ui/inc/PresentationViewShell.hxx @@ -38,7 +38,7 @@ private: static void InitInterface_Impl(); public: - PresentationViewShell( SfxViewFrame* pFrame, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameView); + PresentationViewShell( ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameView); virtual ~PresentationViewShell() override; /** This method is used by a simple class that passes some diff --git a/sd/source/ui/inc/SlideSorter.hxx b/sd/source/ui/inc/SlideSorter.hxx index 87f01e25df8f..97b0336bd149 100644 --- a/sd/source/ui/inc/SlideSorter.hxx +++ b/sd/source/ui/inc/SlideSorter.hxx @@ -249,7 +249,7 @@ private: This method is usually called exactly one time from the constructor. */ - void SetupControls (vcl::Window* pParentWindow); + void SetupControls(); /** This method is usually called exactly one time from the constructor. diff --git a/sd/source/ui/inc/TabControl.hxx b/sd/source/ui/inc/TabControl.hxx index d748fe38d4a5..e72e1a2fe100 100644 --- a/sd/source/ui/inc/TabControl.hxx +++ b/sd/source/ui/inc/TabControl.hxx @@ -101,7 +101,7 @@ private: friend class TabControl::TabControlTransferable; - void DragFinished( sal_Int8 nDropAction ); + void DragFinished(); private: using TabBar::StartDrag; diff --git a/sd/source/ui/inc/View.hxx b/sd/source/ui/inc/View.hxx index 3c1c08f695c5..58915b3e5149 100644 --- a/sd/source/ui/inc/View.hxx +++ b/sd/source/ui/inc/View.hxx @@ -90,8 +90,8 @@ public: virtual bool SetAttributes(const SfxItemSet& rSet, bool bReplaceAll = false); virtual void MarkListHasChanged() override; void SelectAll(); - void DoCut(vcl::Window* pWindow=nullptr); - void DoCopy(vcl::Window* pWindow=nullptr); + void DoCut(); + void DoCopy(); void DoPaste(vcl::Window* pWindow=nullptr); virtual void DoConnect(SdrOle2Obj* pOleObj) override; virtual bool SetStyleSheet(SfxStyleSheet* pStyleSheet, bool bDontRemoveHardAttr = false); @@ -110,7 +110,7 @@ public: sal_uInt16 nLayer); css::uno::Reference - CreateClipboardDataObject (::sd::View*, vcl::Window& rWindow); + CreateClipboardDataObject (); css::uno::Reference CreateDragDataObject (::sd::View*, vcl::Window& rWindow, const Point& rDragPos); diff --git a/sd/source/ui/inc/ViewShell.hxx b/sd/source/ui/inc/ViewShell.hxx index 74ec3b0e6c99..7fcc96cdff92 100644 --- a/sd/source/ui/inc/ViewShell.hxx +++ b/sd/source/ui/inc/ViewShell.hxx @@ -110,7 +110,6 @@ public: ViewShell ( - SfxViewFrame *pFrame, vcl::Window* pParentWindow, ViewShellBase& rViewShellBase); virtual ~ViewShell() override; @@ -180,7 +179,7 @@ public: virtual void MouseButtonUp(const MouseEvent& rMEvt, ::sd::Window* pWin); virtual void MouseButtonDown(const MouseEvent& rMEvt, ::sd::Window* pWin); virtual void Command(const CommandEvent& rCEvt, ::sd::Window* pWin); - bool RequestHelp( const HelpEvent& rEvt, ::sd::Window* pWin ); + bool RequestHelp( const HelpEvent& rEvt ); bool Notify( NotifyEvent& rNEvt, ::sd::Window* pWin ); bool HandleScrollCommand(const CommandEvent& rCEvt, ::sd::Window* pWin); @@ -227,8 +226,8 @@ public: void SetFrameView (FrameView* pFrameView); virtual void ReadFrameViewData(FrameView* pView); virtual void WriteFrameViewData(); - void WriteUserData(OUString& rString); - void ReadUserData(const OUString& rString); + void WriteUserData(); + void ReadUserData(); virtual bool ActivateObject(SdrOle2Obj* pObj, long nVerb); @@ -313,17 +312,8 @@ public: /** Return the border that is drawn around the actual document view. The border contains typically rulers and scroll bars. - @param bOuterResize - When this flag is then the border is used for an - OuterResizePixel(), i.e. there is a given window size and the - border elements are placed inside so that the document view has - the given window size minus the border. - When the flag is then the border is used for an - InnerResizePixel(), i.e. the document view has a given size and - the border is placed outside. In this scenario the parent - window has the size of the document view plus the border. */ - SvBorder GetBorder (bool bOuterResize); + SvBorder GetBorder(); /** Notify the view shell that its parent window has been resized. The ViewShell places and resizes its UI elements accordingly. @@ -503,8 +493,8 @@ protected: virtual ::svl::IUndoManager* ImpGetUndoManager() const; void ImpGetUndoStrings(SfxItemSet &rSet) const; void ImpGetRedoStrings(SfxItemSet &rSet) const; - void ImpSidUndo(bool bDrawViewShell, SfxRequest& rReq); - void ImpSidRedo(bool bDrawViewShell, SfxRequest& rReq); + void ImpSidUndo(SfxRequest& rReq); + void ImpSidRedo(SfxRequest& rReq); DECL_LINK( HScrollHdl, ScrollBar *, void ); DECL_LINK( VScrollHdl, ScrollBar *, void ); diff --git a/sd/source/ui/inc/optsitem.hxx b/sd/source/ui/inc/optsitem.hxx index 9526aee6d539..5bf26e738eff 100644 --- a/sd/source/ui/inc/optsitem.hxx +++ b/sd/source/ui/inc/optsitem.hxx @@ -175,7 +175,7 @@ class SD_DLLPUBLIC SdOptionsContentsItem : public SfxPoolItem { public: - SdOptionsContentsItem( SdOptions* pOpts, ::sd::FrameView* pView ); + SdOptionsContentsItem(); virtual SfxPoolItem* Clone( SfxItemPool *pPool = nullptr ) const override; virtual bool operator==( const SfxPoolItem& ) const override; diff --git a/sd/source/ui/inc/sdtreelb.hxx b/sd/source/ui/inc/sdtreelb.hxx index 76bfa73e0dbc..8541ace2557b 100644 --- a/sd/source/ui/inc/sdtreelb.hxx +++ b/sd/source/ui/inc/sdtreelb.hxx @@ -149,7 +149,7 @@ protected: virtual void RequestingChildren( SvTreeListEntry* pParent ) override; void DoDrag(); - void OnDragFinished( sal_uInt8 nDropAction ); + void OnDragFinished(); /** Return the name of the object. When the object has no user supplied name and the bCreate flag is then a name is created diff --git a/sd/source/ui/inc/slideshow.hxx b/sd/source/ui/inc/slideshow.hxx index ada2329f0df8..b276da8e03df 100644 --- a/sd/source/ui/inc/slideshow.hxx +++ b/sd/source/ui/inc/slideshow.hxx @@ -162,8 +162,8 @@ public: // events void resize( const Size &rSize ); void activate(ViewShellBase& rBase); - void deactivate(ViewShellBase& rBase); - void paint( const ::tools::Rectangle& rRect ); + void deactivate(); + void paint(); bool keyInput(const KeyEvent& rKEvt); diff --git a/sd/source/ui/sidebar/MasterPagesSelector.cxx b/sd/source/ui/sidebar/MasterPagesSelector.cxx index 524061bead98..359b5ad5eda2 100644 --- a/sd/source/ui/sidebar/MasterPagesSelector.cxx +++ b/sd/source/ui/sidebar/MasterPagesSelector.cxx @@ -581,7 +581,7 @@ void MasterPagesSelector::UpdateAllPreviews() mpContainer->RequestPreview(aToken); } } - PreviewValueSet::Rearrange(true); + PreviewValueSet::Rearrange(); } void MasterPagesSelector::ClearPageSet() diff --git a/sd/source/ui/sidebar/PreviewValueSet.cxx b/sd/source/ui/sidebar/PreviewValueSet.cxx index 62eacbabf20e..c15d89d4e231 100644 --- a/sd/source/ui/sidebar/PreviewValueSet.cxx +++ b/sd/source/ui/sidebar/PreviewValueSet.cxx @@ -72,7 +72,7 @@ void PreviewValueSet::Resize() } } -void PreviewValueSet::Rearrange (bool /*bForceRequestResize*/) +void PreviewValueSet::Rearrange() { sal_uInt16 nNewColumnCount (CalculateColumnCount ( GetOutputSizePixel().Width())); diff --git a/sd/source/ui/sidebar/PreviewValueSet.hxx b/sd/source/ui/sidebar/PreviewValueSet.hxx index 880ee9190120..41d5a8fac54e 100644 --- a/sd/source/ui/sidebar/PreviewValueSet.hxx +++ b/sd/source/ui/sidebar/PreviewValueSet.hxx @@ -43,7 +43,7 @@ public: /** Set the number of rows and columns according to the current number of items. Call this method when new items have been inserted. */ - void Rearrange (bool bForceRequestResize = false); + void Rearrange(); protected: virtual void MouseButtonDown (const MouseEvent& rEvent) override; diff --git a/sd/source/ui/slideshow/showwin.cxx b/sd/source/ui/slideshow/showwin.cxx index 3fec3df97e44..48f6c3a321d4 100644 --- a/sd/source/ui/slideshow/showwin.cxx +++ b/sd/source/ui/slideshow/showwin.cxx @@ -254,7 +254,7 @@ void ShowWindow::Paint(vcl::RenderContext& /*rRenderContext*/, const ::tools::Re { if( mxController.is() ) { - mxController->paint(rRect); + mxController->paint(); } else if(mpViewShell ) { diff --git a/sd/source/ui/slideshow/slideshow.cxx b/sd/source/ui/slideshow/slideshow.cxx index 42237d0f83cf..51e2c0bfb094 100644 --- a/sd/source/ui/slideshow/slideshow.cxx +++ b/sd/source/ui/slideshow/slideshow.cxx @@ -977,7 +977,7 @@ void SlideShow::activate( ViewShellBase& rBase ) } -void SlideShow::deactivate( ViewShellBase& /*rBase*/ ) +void SlideShow::deactivate() { mxController->deactivate(); } @@ -987,10 +987,10 @@ bool SlideShow::keyInput(const KeyEvent& rKEvt) return mxController.is() && mxController->keyInput(rKEvt); } -void SlideShow::paint( const ::tools::Rectangle& rRect ) +void SlideShow::paint() { if( mxController.is() ) - mxController->paint( rRect ); + mxController->paint(); } void SlideShow::pause( bool bPause ) diff --git a/sd/source/ui/slideshow/slideshowimpl.cxx b/sd/source/ui/slideshow/slideshowimpl.cxx index 3c48479037c3..8de1e7ac82d2 100644 --- a/sd/source/ui/slideshow/slideshowimpl.cxx +++ b/sd/source/ui/slideshow/slideshowimpl.cxx @@ -1145,7 +1145,7 @@ void SlideshowImpl::onFirstPaint() maUpdateTimer.Start(); } -void SlideshowImpl::paint( const ::tools::Rectangle& /* rRect */ ) +void SlideshowImpl::paint() { if( mxView.is() ) try { @@ -1478,7 +1478,7 @@ void SAL_CALL SlideshowImpl::blankScreen( sal_Int32 nColor ) // XShapeEventListener -void SlideshowImpl::click( const Reference< XShape >& xShape, const css::awt::MouseEvent& /* aOriginalEvent */ ) +void SlideshowImpl::click( const Reference< XShape >& xShape ) { SolarMutexGuard aSolarGuard; @@ -3413,11 +3413,11 @@ void SAL_CALL SlideShowListenerProxy::disposing( const css::lang::EventObject& a // XShapeEventListener -void SAL_CALL SlideShowListenerProxy::click( const Reference< XShape >& xShape, const css::awt::MouseEvent& aOriginalEvent ) +void SAL_CALL SlideShowListenerProxy::click( const Reference< XShape >& xShape, const css::awt::MouseEvent& /*aOriginalEvent*/ ) { SolarMutexGuard aSolarGuard; if( mxController.is() ) - mxController->click(xShape, aOriginalEvent ); + mxController->click(xShape ); } } // namespace ::sd diff --git a/sd/source/ui/slideshow/slideshowimpl.hxx b/sd/source/ui/slideshow/slideshowimpl.hxx index 68778470509c..0a3f5b8c6a69 100644 --- a/sd/source/ui/slideshow/slideshowimpl.hxx +++ b/sd/source/ui/slideshow/slideshowimpl.hxx @@ -223,7 +223,7 @@ public: void slideEnded(const bool bReverse); /// @throws css::uno::RuntimeException void hyperLinkClicked(const OUString & hyperLink); - void click(const css::uno::Reference< css::drawing::XShape > & xShape, const css::awt::MouseEvent & aOriginalEvent); + void click(const css::uno::Reference< css::drawing::XShape > & xShape); bool swipe(const CommandSwipeData &rSwipeData); bool longpress(const CommandLongPressData& rLongPressData); @@ -232,7 +232,7 @@ public: ViewShell* getViewShell() const { return mpViewShell; } - void paint( const ::tools::Rectangle& rRect ); + void paint(); bool keyInput(const KeyEvent& rKEvt); void mouseButtonUp(const MouseEvent& rMEvt); diff --git a/sd/source/ui/slidesorter/cache/SlsRequestQueue.cxx b/sd/source/ui/slidesorter/cache/SlsRequestQueue.cxx index e20e963cb597..8bc8bb3e74da 100644 --- a/sd/source/ui/slidesorter/cache/SlsRequestQueue.cxx +++ b/sd/source/ui/slidesorter/cache/SlsRequestQueue.cxx @@ -99,8 +99,7 @@ RequestQueue::~RequestQueue() void RequestQueue::AddRequest ( CacheKey aKey, - RequestPriorityClass eRequestClass, - bool /*bInsertWithHighestPriority*/) + RequestPriorityClass eRequestClass) { ::osl::MutexGuard aGuard (maMutex); @@ -184,7 +183,7 @@ void RequestQueue::ChangeClass ( Request::DataComparator(aKey))); if (iRequest!=mpRequestQueue->end() && iRequest->meClass!=eNewRequestClass) { - AddRequest(aKey, eNewRequestClass, true); + AddRequest(aKey, eNewRequestClass); } } diff --git a/sd/source/ui/slidesorter/cache/SlsRequestQueue.hxx b/sd/source/ui/slidesorter/cache/SlsRequestQueue.hxx index b8fc2fb85903..7f3946be4954 100644 --- a/sd/source/ui/slidesorter/cache/SlsRequestQueue.hxx +++ b/sd/source/ui/slidesorter/cache/SlsRequestQueue.hxx @@ -56,8 +56,7 @@ public: */ void AddRequest ( CacheKey aKey, - RequestPriorityClass eRequestClass, - bool bInsertWithHighestPriority = false); + RequestPriorityClass eRequestClass); /** Remove the specified request from the queue. @param aKey diff --git a/sd/source/ui/slidesorter/controller/SlsClipboard.cxx b/sd/source/ui/slidesorter/controller/SlsClipboard.cxx index ebc201d46505..d69a340e67d0 100644 --- a/sd/source/ui/slidesorter/controller/SlsClipboard.cxx +++ b/sd/source/ui/slidesorter/controller/SlsClipboard.cxx @@ -636,7 +636,7 @@ sal_Int8 Clipboard::AcceptDrop ( { sal_Int8 nAction (DND_ACTION_NONE); - const Clipboard::DropType eDropType (IsDropAccepted(rTargetHelper)); + const Clipboard::DropType eDropType (IsDropAccepted()); switch (eDropType) { @@ -704,7 +704,7 @@ sal_Int8 Clipboard::ExecuteDrop ( { sal_Int8 nResult = DND_ACTION_NONE; mxUndoContext.reset(); - const Clipboard::DropType eDropType (IsDropAccepted(rTargetHelper)); + const Clipboard::DropType eDropType (IsDropAccepted()); switch (eDropType) { @@ -857,7 +857,7 @@ sal_uInt16 Clipboard::InsertSlides ( return nInsertedPageCount; } -Clipboard::DropType Clipboard::IsDropAccepted (DropTargetHelper&) const +Clipboard::DropType Clipboard::IsDropAccepted() const { const SdTransferable* pDragTransferable = SD_MOD()->pTransferDrag; if (pDragTransferable == nullptr) diff --git a/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx b/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx index 2842a1e2a97d..031fb61f6913 100644 --- a/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx +++ b/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx @@ -405,7 +405,7 @@ void SlotManager::FuSupport (SfxRequest& rRequest) = dynamic_cast(mrSlideSorter.GetViewShell()); if (pViewShell != nullptr) { - pViewShell->ImpSidUndo (false, rRequest); + pViewShell->ImpSidUndo (rRequest); } break; } @@ -416,7 +416,7 @@ void SlotManager::FuSupport (SfxRequest& rRequest) = dynamic_cast(mrSlideSorter.GetViewShell()); if (pViewShell != nullptr) { - pViewShell->ImpSidRedo (false, rRequest); + pViewShell->ImpSidRedo (rRequest); } break; } diff --git a/sd/source/ui/slidesorter/inc/controller/SlsClipboard.hxx b/sd/source/ui/slidesorter/inc/controller/SlsClipboard.hxx index b7d819a6ebf3..bc0ce0836174 100644 --- a/sd/source/ui/slidesorter/inc/controller/SlsClipboard.hxx +++ b/sd/source/ui/slidesorter/inc/controller/SlsClipboard.hxx @@ -186,7 +186,7 @@ private: implementation of proper handling master pages copy-and-paste. */ enum DropType { DT_PAGE, DT_PAGE_FROM_NAVIGATOR, DT_SHAPE, DT_NONE }; - DropType IsDropAccepted (DropTargetHelper& rTargetHelper) const; + DropType IsDropAccepted() const; /** This method contains the code for AcceptDrop() and ExecuteDrop() shapes. There are only minor differences for the two cases at this level. diff --git a/sd/source/ui/slidesorter/shell/SlideSorter.cxx b/sd/source/ui/slidesorter/shell/SlideSorter.cxx index 9a0a758b2370..37a710a45663 100644 --- a/sd/source/ui/slidesorter/shell/SlideSorter.cxx +++ b/sd/source/ui/slidesorter/shell/SlideSorter.cxx @@ -183,7 +183,7 @@ void SlideSorter::Init() pContentWindow->Hide(); // Set view pointer of base class. - SetupControls(pParentWindow); + SetupControls(); mbIsValid = true; } @@ -242,7 +242,7 @@ void SlideSorter::Paint (const ::tools::Rectangle& rRepaintArea) GetContentWindow()); } -void SlideSorter::SetupControls (vcl::Window* ) +void SlideSorter::SetupControls() { GetVerticalScrollBar()->Show(); } @@ -371,16 +371,12 @@ bool SlideSorter::RelocateToWindow (vcl::Window* pParentWindow) ReleaseListeners(); - vcl::Window *pNewWindow = nullptr; if (mpViewShell) { mpViewShell->ViewShell::RelocateToParentWindow(pParentWindow); - pNewWindow = mpViewShell->GetParentWindow(); } - else - pNewWindow = nullptr; - SetupControls(pNewWindow); + SetupControls(); SetupListeners(); // For accessibility we have to shortly hide the content window. This diff --git a/sd/source/ui/slidesorter/shell/SlideSorterViewShell.cxx b/sd/source/ui/slidesorter/shell/SlideSorterViewShell.cxx index d56fb3f52389..1f015f1ae9bc 100644 --- a/sd/source/ui/slidesorter/shell/SlideSorterViewShell.cxx +++ b/sd/source/ui/slidesorter/shell/SlideSorterViewShell.cxx @@ -115,11 +115,11 @@ std::shared_ptr SlideSorterViewShell::Create ( } SlideSorterViewShell::SlideSorterViewShell ( - SfxViewFrame* pFrame, + SfxViewFrame* /*pFrame*/, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameViewArgument) - : ViewShell (pFrame, pParentWindow, rViewShellBase), + : ViewShell (pParentWindow, rViewShellBase), mpSlideSorter(), mbIsArrangeGUIElementsPending(true) { diff --git a/sd/source/ui/unoidl/unomodel.cxx b/sd/source/ui/unoidl/unomodel.cxx index aefeb457483a..6fb83a46f59f 100644 --- a/sd/source/ui/unoidl/unomodel.cxx +++ b/sd/source/ui/unoidl/unomodel.cxx @@ -638,7 +638,7 @@ uno::Reference < container::XIndexAccess > SAL_CALL SdXImpressDocument::getViewD ::sd::FrameView* pFrameView = rList[ i ]; uno::Sequence< beans::PropertyValue > aSeq; - pFrameView->WriteUserDataSequence( aSeq, false ); + pFrameView->WriteUserDataSequence( aSeq ); xCont->insertByIndex( i, uno::makeAny( aSeq ) ); } } @@ -676,7 +676,7 @@ void SAL_CALL SdXImpressDocument::setViewData( const uno::Reference < container: { pFrameView = new ::sd::FrameView( mpDoc ); - pFrameView->ReadUserDataSequence( aSeq, false ); + pFrameView->ReadUserDataSequence( aSeq ); rViews.push_back( pFrameView ); } } diff --git a/sd/source/ui/unoidl/unopage.cxx b/sd/source/ui/unoidl/unopage.cxx index 18c50b8c608d..ce0bf91fa98d 100644 --- a/sd/source/ui/unoidl/unopage.cxx +++ b/sd/source/ui/unoidl/unopage.cxx @@ -533,7 +533,7 @@ SdrObject * SdGenericDrawPage::CreateSdrObject_( const Reference< drawing::XShap } else { - pPresObj = GetPage()->CreatePresObj( eObjKind, false, aRect, true ); + pPresObj = GetPage()->CreatePresObj( eObjKind, false, aRect ); } if( pPresObj ) diff --git a/sd/source/ui/view/Outliner.cxx b/sd/source/ui/view/Outliner.cxx index 11fc88a4f8dd..0cd8df02d24e 100644 --- a/sd/source/ui/view/Outliner.cxx +++ b/sd/source/ui/view/Outliner.cxx @@ -264,7 +264,7 @@ void SdOutliner::PrepareSpelling() sd::ViewShellBase* pBase = dynamic_cast< sd::ViewShellBase *>( SfxViewShell::Current() ); if (pBase != nullptr) SetViewShell (pBase->GetMainViewShell()); - SetRefDevice( SD_MOD()->GetRefDevice( *mpDrawDocument->GetDocSh() ) ); + SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); std::shared_ptr pViewShell (mpWeakViewShell.lock()); if (pViewShell) @@ -342,7 +342,7 @@ void SdOutliner::EndSpelling() if(IsModified()) { if(mpView && dynamic_cast< const sd::OutlineView *>( mpView ) != nullptr) - static_cast(mpView)->PrepareClose(false); + static_cast(mpView)->PrepareClose(); if(mpDrawDocument && !mpDrawDocument->IsChanged()) mpDrawDocument->SetChanged(); } @@ -374,7 +374,7 @@ bool SdOutliner::SpellNextDocument() else { if( dynamic_cast< const sd::OutlineView *>( mpView ) != nullptr) - static_cast(mpView)->PrepareClose(false); + static_cast(mpView)->PrepareClose(); mpDrawDocument->GetDocSh()->SetWaitCursor( true ); Initialize (true); @@ -1612,7 +1612,7 @@ void SdOutliner::PrepareConversion() void SdOutliner::BeginConversion() { - SetRefDevice( SD_MOD()->GetRefDevice( *mpDrawDocument->GetDocSh() ) ); + SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); sd::ViewShellBase* pBase = dynamic_cast( SfxViewShell::Current() ); if (pBase != nullptr) diff --git a/sd/source/ui/view/ViewShellBase.cxx b/sd/source/ui/view/ViewShellBase.cxx index 9487cba4ddb8..308e28f890af 100644 --- a/sd/source/ui/view/ViewShellBase.cxx +++ b/sd/source/ui/view/ViewShellBase.cxx @@ -786,7 +786,7 @@ void ViewShellBase::WriteUserData (OUString& rString, bool bBrowse) // Forward call to main sub shell. ViewShell* pShell = GetMainViewShell().get(); if (pShell != nullptr) - pShell->WriteUserData (rString); + pShell->WriteUserData(); } void ViewShellBase::ReadUserData (const OUString& rString, bool bBrowse) @@ -796,7 +796,7 @@ void ViewShellBase::ReadUserData (const OUString& rString, bool bBrowse) // Forward call to main sub shell. ViewShell* pShell = GetMainViewShell().get(); if (pShell != nullptr) - pShell->ReadUserData (rString); + pShell->ReadUserData(); } SdrView* ViewShellBase::GetDrawView() const @@ -829,7 +829,7 @@ void ViewShellBase::UpdateBorder ( bool bForce /* = false */ ) SvBorder aCurrentBorder (GetBorderPixel()); bool bOuterResize ( ! GetDocShell()->IsInPlaceActive()); SvBorder aBorder (GetBorder(bOuterResize)); - aBorder += pMainViewShell->GetBorder(bOuterResize); + aBorder += pMainViewShell->GetBorder(); if (bForce || (aBorder != aCurrentBorder)) { @@ -1124,7 +1124,7 @@ void ViewShellBase::Implementation::ResizePixel ( // Calculate and set the border before the controls are placed. SvBorder aBorder; if (pMainViewShell != nullptr) - aBorder = pMainViewShell->GetBorder(bOuterResize); + aBorder = pMainViewShell->GetBorder(); aBorder += mrBase.GetBorder(bOuterResize); if (mrBase.GetBorderPixel() != aBorder) mrBase.SetBorderPixel(aBorder); diff --git a/sd/source/ui/view/drawview.cxx b/sd/source/ui/view/drawview.cxx index fd74fd14a497..26ac7a9b4f22 100644 --- a/sd/source/ui/view/drawview.cxx +++ b/sd/source/ui/view/drawview.cxx @@ -473,7 +473,7 @@ void DrawView::CompleteRedraw(OutputDevice* pOutDev, const vcl::Region& rReg, sd if( (pShowWindow == pOutDev) || (xSlideshow->getAnimationMode() == ANIMATIONMODE_PREVIEW) ) { if( pShowWindow == pOutDev && mpViewSh ) - xSlideshow->paint( rReg.GetBoundRect() ); + xSlideshow->paint(); bStandardPaint = false; } } diff --git a/sd/source/ui/view/drviews8.cxx b/sd/source/ui/view/drviews8.cxx index 8c23c9656f3a..ac4bf0203db4 100644 --- a/sd/source/ui/view/drviews8.cxx +++ b/sd/source/ui/view/drviews8.cxx @@ -45,7 +45,7 @@ namespace sd { -void DrawViewShell::ScannerEvent( const css::lang::EventObject& ) +void DrawViewShell::ScannerEvent() { if( mxScannerManager.is() ) { diff --git a/sd/source/ui/view/drviewsa.cxx b/sd/source/ui/view/drviewsa.cxx index 5c8832a5b111..c623adcb6517 100644 --- a/sd/source/ui/view/drviewsa.cxx +++ b/sd/source/ui/view/drviewsa.cxx @@ -95,14 +95,14 @@ public: void ParentDestroyed() { mpParent = nullptr; } }; -void SAL_CALL ScannerEventListener::disposing( const lang::EventObject& rEventObject ) +void SAL_CALL ScannerEventListener::disposing( const lang::EventObject& /*rEventObject*/ ) { if( mpParent ) - mpParent->ScannerEvent( rEventObject ); + mpParent->ScannerEvent(); } -DrawViewShell::DrawViewShell( SfxViewFrame* pFrame, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, PageKind ePageKind, FrameView* pFrameViewArgument ) - : ViewShell (pFrame, pParentWindow, rViewShellBase) +DrawViewShell::DrawViewShell( ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, PageKind ePageKind, FrameView* pFrameViewArgument ) + : ViewShell (pParentWindow, rViewShellBase) , maTabControl(VclPtr::Create(this, pParentWindow)) , mbIsLayerModeActive(false) , mbIsInSwitchPage(false) @@ -156,7 +156,7 @@ DrawViewShell::~DrawViewShell() EndListening (*GetDocSh()); if( SlideShow::IsRunning(*this) ) - StopSlideShow(false); + StopSlideShow(); DisposeFunctions(); diff --git a/sd/source/ui/view/drviewse.cxx b/sd/source/ui/view/drviewse.cxx index 7cd022d38ea1..39688be50520 100644 --- a/sd/source/ui/view/drviewse.cxx +++ b/sd/source/ui/view/drviewse.cxx @@ -731,7 +731,7 @@ void DrawViewShell::FuSupport(SfxRequest& rReq) case SID_PRESENTATION_END: { - StopSlideShow(true); + StopSlideShow(); rReq.Ignore (); } @@ -1385,13 +1385,13 @@ void DrawViewShell::FuSupport(SfxRequest& rReq) case SID_UNDO : { // moved implementation to BaseClass - ImpSidUndo(true, rReq); + ImpSidUndo(rReq); } break; case SID_REDO : { // moved implementation to BaseClass - ImpSidRedo(true, rReq); + ImpSidRedo(rReq); } break; @@ -1615,7 +1615,7 @@ namespace slideshowhelp } } -void DrawViewShell::StopSlideShow (bool /*bCloseFrame*/) +void DrawViewShell::StopSlideShow() { Reference< XPresentation2 > xPresentation( GetDoc()->getPresentation() ); if(xPresentation.is() && xPresentation->isRunning()) diff --git a/sd/source/ui/view/frmview.cxx b/sd/source/ui/view/frmview.cxx index df7efde02afa..e64042487324 100644 --- a/sd/source/ui/view/frmview.cxx +++ b/sd/source/ui/view/frmview.cxx @@ -372,7 +372,7 @@ static OUString createHelpLinesString( const SdrHelpLineList& rHelpLines ) } #define addValue( n, v ) push_back( std::pair< OUString, Any >( OUString( n ), v ) ) -void FrameView::WriteUserDataSequence ( css::uno::Sequence < css::beans::PropertyValue >& rValues, bool ) +void FrameView::WriteUserDataSequence ( css::uno::Sequence < css::beans::PropertyValue >& rValues ) { std::vector< std::pair< OUString, Any > > aUserData; @@ -523,7 +523,7 @@ static void createHelpLinesFromString( const OUString& rLines, SdrHelpLineList& } } -void FrameView::ReadUserDataSequence ( const css::uno::Sequence < css::beans::PropertyValue >& rSequence, bool ) +void FrameView::ReadUserDataSequence ( const css::uno::Sequence < css::beans::PropertyValue >& rSequence ) { const sal_Int32 nLength = rSequence.getLength(); if (nLength) diff --git a/sd/source/ui/view/grviewsh.cxx b/sd/source/ui/view/grviewsh.cxx index ce5991ddcdec..509fbc4f0fd7 100644 --- a/sd/source/ui/view/grviewsh.cxx +++ b/sd/source/ui/view/grviewsh.cxx @@ -30,12 +30,10 @@ namespace sd { GraphicViewShell::GraphicViewShell ( - SfxViewFrame* pFrame, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameView) : DrawViewShell ( - pFrame, rViewShellBase, pParentWindow, PageKind::Standard, diff --git a/sd/source/ui/view/outlnvsh.cxx b/sd/source/ui/view/outlnvsh.cxx index 470a7eed34bc..25bf6e858a9b 100644 --- a/sd/source/ui/view/outlnvsh.cxx +++ b/sd/source/ui/view/outlnvsh.cxx @@ -126,7 +126,7 @@ void OutlineViewShell::InitInterface_Impl() /** * common initialization part of both constructors */ -void OutlineViewShell::Construct(DrawDocShell* ) +void OutlineViewShell::Construct() { bool bModified = GetDoc()->IsChanged(); @@ -180,11 +180,11 @@ Reference OutlineViewShell::CreateSubController() * Default constructor, windows must not center themselves automatically */ OutlineViewShell::OutlineViewShell ( - SfxViewFrame* pFrame, + SfxViewFrame* /*pFrame*/, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameViewArgument) - : ViewShell(pFrame, pParentWindow, rViewShellBase), + : ViewShell(pParentWindow, rViewShellBase), pOlView(nullptr), pLastPage( nullptr ), bPastePossible(false), @@ -198,7 +198,7 @@ OutlineViewShell::OutlineViewShell ( mpFrameView->Connect(); - Construct(GetDocSh()); + Construct(); SetContextName(vcl::EnumContext::GetContextName(vcl::EnumContext::Context::OutlineText)); @@ -635,13 +635,13 @@ void OutlineViewShell::FuSupport(SfxRequest &rReq) case SID_UNDO : { OutlineViewPageChangesGuard aGuard2(pOlView); - ImpSidUndo(false, rReq); + ImpSidUndo(rReq); } break; case SID_REDO : { OutlineViewPageChangesGuard aGuard2(pOlView); - ImpSidRedo(false, rReq); + ImpSidRedo(rReq); } break; @@ -1114,7 +1114,7 @@ bool OutlineViewShell::PrepareClose( bool bUI ) if( !ViewShell::PrepareClose(bUI) ) return false; - return pOlView == nullptr || pOlView->PrepareClose(bUI); + return pOlView == nullptr || pOlView->PrepareClose(); } /** diff --git a/sd/source/ui/view/outlview.cxx b/sd/source/ui/view/outlview.cxx index 5821dde1d610..3a859aa68662 100644 --- a/sd/source/ui/view/outlview.cxx +++ b/sd/source/ui/view/outlview.cxx @@ -96,7 +96,7 @@ OutlineView::OutlineView( DrawDocShell& rDocSh, vcl::Window* pWindow, OutlineVie // initialize Outliner: set Reference Device bInitOutliner = true; mrOutliner.Init( OutlinerMode::OutlineView ); - mrOutliner.SetRefDevice( SD_MOD()->GetRefDevice( rDocSh ) ); + mrOutliner.SetRefDevice( SD_MOD()->GetVirtualRefDevice() ); //viewsize without the width of the image and number in front mnPaperWidth = (mrOutlineViewShell.GetActiveWindow()->GetViewSize().Width() - 4000); mrOutliner.SetPaperSize(Size(mnPaperWidth, 400000000)); @@ -1002,7 +1002,7 @@ SdrTextObj* OutlineView::CreateOutlineTextObject(SdPage* pPage) } /** updates draw model with all changes from outliner model */ -bool OutlineView::PrepareClose(bool) +bool OutlineView::PrepareClose() { ::sd::UndoManager* pDocUndoMgr = dynamic_cast(mpDocSh->GetUndoManager()); if (pDocUndoMgr != nullptr) diff --git a/sd/source/ui/view/presvish.cxx b/sd/source/ui/view/presvish.cxx index a9c63d886caf..206bd66ffea5 100644 --- a/sd/source/ui/view/presvish.cxx +++ b/sd/source/ui/view/presvish.cxx @@ -74,8 +74,8 @@ void PresentationViewShell::InitInterface_Impl() } -PresentationViewShell::PresentationViewShell( SfxViewFrame* pFrame, ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameView) -: DrawViewShell( pFrame, rViewShellBase, pParentWindow, PageKind::Standard, pFrameView) +PresentationViewShell::PresentationViewShell( ViewShellBase& rViewShellBase, vcl::Window* pParentWindow, FrameView* pFrameView) +: DrawViewShell( rViewShellBase, pParentWindow, PageKind::Standard, pFrameView) { if( GetDocSh() && GetDocSh()->GetCreateMode() == SfxObjectCreateMode::EMBEDDED ) maOldVisArea = GetDocSh()->GetVisArea( ASPECT_CONTENT ); @@ -140,11 +140,11 @@ void PresentationViewShell::Activate( bool bIsMDIActivate ) GetDocSh()->Connect( this ); } -void PresentationViewShell::Paint( const ::tools::Rectangle& rRect, ::sd::Window* ) +void PresentationViewShell::Paint( const ::tools::Rectangle& /*rRect*/, ::sd::Window* ) { rtl::Reference< SlideShow > xSlideShow( SlideShow::GetSlideShow( GetViewShellBase() ) ); if( xSlideShow.is() ) - xSlideShow->paint(rRect); + xSlideShow->paint(); } void PresentationViewShell::Resize() diff --git a/sd/source/ui/view/sdview2.cxx b/sd/source/ui/view/sdview2.cxx index 24a0299bf729..76af9466f9d4 100644 --- a/sd/source/ui/view/sdview2.cxx +++ b/sd/source/ui/view/sdview2.cxx @@ -84,7 +84,7 @@ struct SdNavigatorDropEvent : public ExecuteDropEvent {} }; -css::uno::Reference< css::datatransfer::XTransferable > View::CreateClipboardDataObject( View*, vcl::Window& ) +css::uno::Reference< css::datatransfer::XTransferable > View::CreateClipboardDataObject() { // since SdTransferable::CopyToClipboard is called, this // dynamically created object is destroyed automatically @@ -234,7 +234,7 @@ void View::UpdateSelectionClipboard( bool bForceDeselect ) } } -void View::DoCut(vcl::Window* ) +void View::DoCut() { const OutlinerView* pOLV = GetTextEditOutlinerView(); @@ -251,7 +251,7 @@ void View::DoCut(vcl::Window* ) } } -void View::DoCopy (vcl::Window* pWindow) +void View::DoCopy() { const OutlinerView* pOLV = GetTextEditOutlinerView(); @@ -260,7 +260,7 @@ void View::DoCopy (vcl::Window* pWindow) else if( AreObjectsMarked() ) { BrkAction(); - CreateClipboardDataObject( this, *pWindow ); + CreateClipboardDataObject(); } } diff --git a/sd/source/ui/view/sdwindow.cxx b/sd/source/ui/view/sdwindow.cxx index 9d13093e6144..ce5d5ba1fa43 100644 --- a/sd/source/ui/view/sdwindow.cxx +++ b/sd/source/ui/view/sdwindow.cxx @@ -290,7 +290,7 @@ void Window::RequestHelp(const HelpEvent& rEvt) { if ( mpViewShell ) { - if( !mpViewShell->RequestHelp( rEvt, this) ) + if( !mpViewShell->RequestHelp( rEvt ) ) vcl::Window::RequestHelp( rEvt ); } else diff --git a/sd/source/ui/view/tabcontr.cxx b/sd/source/ui/view/tabcontr.cxx index bbb7598bba31..7b60cc3796d5 100644 --- a/sd/source/ui/view/tabcontr.cxx +++ b/sd/source/ui/view/tabcontr.cxx @@ -57,9 +57,9 @@ bool TabControl::TabControlTransferable::GetData( const css::datatransfer::DataF return false; } -void TabControl::TabControlTransferable::DragFinished( sal_Int8 nDropAction ) +void TabControl::TabControlTransferable::DragFinished( sal_Int8 /*nDropAction*/ ) { - mrParent.DragFinished( nDropAction ); + mrParent.DragFinished(); } TabControl::TabControl(DrawViewShell* pViewSh, vcl::Window* pParent) : @@ -163,7 +163,7 @@ void TabControl::StartDrag( sal_Int8, const Point& ) ( new TabControl::TabControlTransferable( *this ) )->StartDrag( this, DND_ACTION_COPYMOVE ); } -void TabControl::DragFinished( sal_Int8 ) +void TabControl::DragFinished() { bInternalMove = false; } diff --git a/sd/source/ui/view/viewshe2.cxx b/sd/source/ui/view/viewshe2.cxx index a3f4f91ec719..3eee7b1e09e4 100644 --- a/sd/source/ui/view/viewshe2.cxx +++ b/sd/source/ui/view/viewshe2.cxx @@ -670,7 +670,7 @@ void ViewShell::SetActiveWindow (::sd::Window* pWin) } } -bool ViewShell::RequestHelp(const HelpEvent& rHEvt, ::sd::Window*) +bool ViewShell::RequestHelp(const HelpEvent& rHEvt) { bool bReturn = false; @@ -890,14 +890,14 @@ const ::tools::Rectangle& ViewShell::GetAllWindowRect() return maAllWindowRectangle; } -void ViewShell::ReadUserData(const OUString&) +void ViewShell::ReadUserData() { // zoom onto VisArea from FrameView GetViewShell()->GetViewFrame()->GetDispatcher()->Execute(SID_SIZE_VISAREA, SfxCallMode::ASYNCHRON | SfxCallMode::RECORD); } -void ViewShell::WriteUserData(OUString&) +void ViewShell::WriteUserData() { // writing of our data is always done in WriteFrameViewData() WriteFrameViewData(); @@ -961,7 +961,7 @@ sal_Int8 ViewShell::ExecuteDrop ( return pView ? pView->ExecuteDrop( rEvt, pTargetWindow, nPage, nLayer ) : DND_ACTION_NONE; } -void ViewShell::WriteUserDataSequence ( css::uno::Sequence < css::beans::PropertyValue >& rSequence, bool bBrowse) +void ViewShell::WriteUserDataSequence ( css::uno::Sequence < css::beans::PropertyValue >& rSequence, bool /*bBrowse*/) { const sal_Int32 nIndex = rSequence.getLength(); rSequence.realloc( nIndex + 1 ); @@ -976,12 +976,12 @@ void ViewShell::WriteUserDataSequence ( css::uno::Sequence < css::beans::Propert rSequence[nIndex].Name = sUNO_View_ViewId; rSequence[nIndex].Value <<= "view" + OUString::number( static_cast(nViewID)); - mpFrameView->WriteUserDataSequence( rSequence, bBrowse ); + mpFrameView->WriteUserDataSequence( rSequence ); } -void ViewShell::ReadUserDataSequence ( const css::uno::Sequence < css::beans::PropertyValue >& rSequence, bool bBrowse ) +void ViewShell::ReadUserDataSequence ( const css::uno::Sequence < css::beans::PropertyValue >& rSequence, bool /*bBrowse*/ ) { - mpFrameView->ReadUserDataSequence( rSequence, bBrowse ); + mpFrameView->ReadUserDataSequence( rSequence ); } void ViewShell::VisAreaChanged(const ::tools::Rectangle& rRect) diff --git a/sd/source/ui/view/viewshel.cxx b/sd/source/ui/view/viewshel.cxx index 25917474f54c..b741231d7986 100644 --- a/sd/source/ui/view/viewshel.cxx +++ b/sd/source/ui/view/viewshel.cxx @@ -140,7 +140,7 @@ SfxViewFrame* ViewShell::GetViewFrame() const /// declare SFX-Slotmap and standard interface -ViewShell::ViewShell( SfxViewFrame*, vcl::Window* pParentWindow, ViewShellBase& rViewShellBase) +ViewShell::ViewShell( vcl::Window* pParentWindow, ViewShellBase& rViewShellBase) : SfxShell(&rViewShellBase) , mpParentWindow(pParentWindow) { @@ -381,7 +381,7 @@ void ViewShell::Deactivate(bool bIsMDIActivate) { rtl::Reference< SlideShow > xSlideShow( SlideShow::GetSlideShow( GetViewShellBase() ) ); if(xSlideShow.is() && xSlideShow->isRunning() ) - xSlideShow->deactivate(GetViewShellBase()); + xSlideShow->deactivate(); if(HasCurrentFunction()) GetCurrentFunction()->Deactivate(); @@ -970,7 +970,7 @@ void ViewShell::Resize() pView->VisAreaChanged(GetActiveWindow()); } -SvBorder ViewShell::GetBorder (bool ) +SvBorder ViewShell::GetBorder() { SvBorder aBorder; @@ -1248,7 +1248,7 @@ public: } }; -void ViewShell::ImpSidUndo(bool, SfxRequest& rReq) +void ViewShell::ImpSidUndo(SfxRequest& rReq) { //The xWatcher keeps the SlideSorter selection in sync //with the page insertions/deletions that Undo may introduce @@ -1299,7 +1299,7 @@ void ViewShell::ImpSidUndo(bool, SfxRequest& rReq) rReq.Done(); } -void ViewShell::ImpSidRedo(bool, SfxRequest& rReq) +void ViewShell::ImpSidRedo(SfxRequest& rReq) { //The xWatcher keeps the SlideSorter selection in sync //with the page insertions/deletions that Undo may introduce