diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-05 13:03:43 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-08 08:29:03 +0200 |
commit | 185ed3ddb8c01ee4465ce559e37113824f57b5c7 (patch) | |
tree | 596455ca4b9dc85666efbf06a1e1e0a3eec3ee2d /sd | |
parent | d33e262a244f351febc9dbe605b05f76cb834eeb (diff) |
teach loplugin:constantparam about simple constructor calls
Change-Id: I7d2a28ab5951fbdb5a427c84e9ac4c1e32ecf9f9
Reviewed-on: https://gerrit.libreoffice.org/37280
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sd')
-rw-r--r-- | sd/source/ui/app/sdmod1.cxx | 2 | ||||
-rw-r--r-- | sd/source/ui/inc/OutlineViewShell.hxx | 2 | ||||
-rw-r--r-- | sd/source/ui/inc/PreviewRenderer.hxx | 7 | ||||
-rw-r--r-- | sd/source/ui/sidebar/MasterPageContainerProviders.cxx | 3 | ||||
-rw-r--r-- | sd/source/ui/tools/PreviewRenderer.cxx | 5 | ||||
-rw-r--r-- | sd/source/ui/view/outlnvsh.cxx | 4 | ||||
-rw-r--r-- | sd/source/ui/view/sdview3.cxx | 6 |
7 files changed, 11 insertions, 18 deletions
diff --git a/sd/source/ui/app/sdmod1.cxx b/sd/source/ui/app/sdmod1.cxx index f90d50d5824d..53d0750ede7b 100644 --- a/sd/source/ui/app/sdmod1.cxx +++ b/sd/source/ui/app/sdmod1.cxx @@ -606,7 +606,7 @@ void OutlineToImpressFinalizer::operator() (bool) sd::OutlineView* pView = static_cast<sd::OutlineView*>(pOutlineShell->GetView()); // mba: the stream can't contain any relative URLs, because we don't // have any information about a BaseURL! - if ( pOutlineShell->ReadRtf(*mpStream, OUString()) == 0 ) + if ( pOutlineShell->ReadRtf(*mpStream) == 0 ) { } diff --git a/sd/source/ui/inc/OutlineViewShell.hxx b/sd/source/ui/inc/OutlineViewShell.hxx index 7284d36a03d5..10ff97c9a074 100644 --- a/sd/source/ui/inc/OutlineViewShell.hxx +++ b/sd/source/ui/inc/OutlineViewShell.hxx @@ -111,7 +111,7 @@ public: virtual bool KeyInput(const KeyEvent& rKEvt, ::sd::Window* pWin) override; virtual void MouseButtonUp(const MouseEvent& rMEvt, ::sd::Window* pWin) override; - sal_uLong ReadRtf(SvStream& rInput, const OUString& rBaseURL); + sal_uLong ReadRtf(SvStream& rInput); virtual void WriteUserDataSequence ( css::uno::Sequence < css::beans::PropertyValue >&, bool bBrowse ) override; virtual void ReadUserDataSequence ( const css::uno::Sequence < css::beans::PropertyValue >&, bool bBrowse ) override; diff --git a/sd/source/ui/inc/PreviewRenderer.hxx b/sd/source/ui/inc/PreviewRenderer.hxx index 7cccf318e29f..3c0e97254b0e 100644 --- a/sd/source/ui/inc/PreviewRenderer.hxx +++ b/sd/source/ui/inc/PreviewRenderer.hxx @@ -57,17 +57,12 @@ public: The page to render. @param nWidth The width of the preview in device coordinates. - @param sSubstitutionText - When the actual preview can not be created for some reason, then - this text is painted in an empty rectangle of the requested size - instead. The high contrast mode of the application is ignored and the preview is rendered in normal mode. */ Image RenderPage ( const SdPage* pPage, - const sal_Int32 nWidth, - const OUString& sSubstitutionText); + const sal_Int32 nWidth); /** Render a page with the given pixel size. @param pPage diff --git a/sd/source/ui/sidebar/MasterPageContainerProviders.cxx b/sd/source/ui/sidebar/MasterPageContainerProviders.cxx index 11283ed50493..c9a2e37b844e 100644 --- a/sd/source/ui/sidebar/MasterPageContainerProviders.cxx +++ b/sd/source/ui/sidebar/MasterPageContainerProviders.cxx @@ -57,8 +57,7 @@ Image PagePreviewProvider::operator () ( // object. aPreview = rRenderer.RenderPage( pPage, - nWidth, - OUString()); + nWidth); } return aPreview; diff --git a/sd/source/ui/tools/PreviewRenderer.cxx b/sd/source/ui/tools/PreviewRenderer.cxx index d8096c27e53c..c41822289095 100644 --- a/sd/source/ui/tools/PreviewRenderer.cxx +++ b/sd/source/ui/tools/PreviewRenderer.cxx @@ -82,8 +82,7 @@ PreviewRenderer::~PreviewRenderer() Image PreviewRenderer::RenderPage ( const SdPage* pPage, - const sal_Int32 nWidth, - const OUString& rSubstitutionText) + const sal_Int32 nWidth) { if (pPage != nullptr) { @@ -96,7 +95,7 @@ Image PreviewRenderer::RenderPage ( return RenderPage ( pPage, Size(nWidth,nHeight), - rSubstitutionText, + OUString(), false/*bObeyHighContrastMode*/); } else diff --git a/sd/source/ui/view/outlnvsh.cxx b/sd/source/ui/view/outlnvsh.cxx index 25bf6e858a9b..7041995282af 100644 --- a/sd/source/ui/view/outlnvsh.cxx +++ b/sd/source/ui/view/outlnvsh.cxx @@ -1750,7 +1750,7 @@ void OutlineViewShell::UpdateOutlineObject( SdPage* pPage, Paragraph* pPara ) /** * Fill Outliner from Stream */ -sal_uLong OutlineViewShell::ReadRtf(SvStream& rInput, const OUString& rBaseURL) +sal_uLong OutlineViewShell::ReadRtf(SvStream& rInput) { sal_uLong bRet = 0; @@ -1759,7 +1759,7 @@ sal_uLong OutlineViewShell::ReadRtf(SvStream& rInput, const OUString& rBaseURL) OutlineViewPageChangesGuard aGuard( pOlView ); OutlineViewModelChangeGuard aGuard2( *pOlView ); - bRet = rOutl.Read( rInput, rBaseURL, EE_FORMAT_RTF, GetDocSh()->GetHeaderAttributes() ); + bRet = rOutl.Read( rInput, OUString(), EE_FORMAT_RTF, GetDocSh()->GetHeaderAttributes() ); SdPage* pPage = GetDoc()->GetSdPage( GetDoc()->GetSdPageCount(PageKind::Standard) - 1, PageKind::Standard ); SfxStyleSheet* pTitleSheet = pPage->GetStyleSheetForPresObj( PRESOBJ_TITLE ); diff --git a/sd/source/ui/view/sdview3.cxx b/sd/source/ui/view/sdview3.cxx index 5fa5158618bf..c57182f1ca13 100644 --- a/sd/source/ui/view/sdview3.cxx +++ b/sd/source/ui/view/sdview3.cxx @@ -1388,7 +1388,7 @@ bool View::InsertData( const TransferableDataHelper& rDataHelper, { xStm->Seek( 0 ); // mba: clipboard always must contain absolute URLs (could be from alien source) - bReturn = SdrView::Paste( *xStm, OUString(), EE_FORMAT_HTML, maDropPos, pPage, nPasteOptions ); + bReturn = SdrView::Paste( *xStm, EE_FORMAT_HTML, maDropPos, pPage, nPasteOptions ); } } @@ -1417,7 +1417,7 @@ bool View::InsertData( const TransferableDataHelper& rDataHelper, if( !bReturn ) // mba: clipboard always must contain absolute URLs (could be from alien source) - bReturn = SdrView::Paste( *xStm, OUString(), EE_FORMAT_BIN, maDropPos, pPage, nPasteOptions ); + bReturn = SdrView::Paste( *xStm, EE_FORMAT_BIN, maDropPos, pPage, nPasteOptions ); } } @@ -1453,7 +1453,7 @@ bool View::InsertData( const TransferableDataHelper& rDataHelper, if( !bReturn ) // mba: clipboard always must contain absolute URLs (could be from alien source) - bReturn = SdrView::Paste( *xStm, OUString(), EE_FORMAT_RTF, maDropPos, pPage, nPasteOptions ); + bReturn = SdrView::Paste( *xStm, EE_FORMAT_RTF, maDropPos, pPage, nPasteOptions ); } } } |