diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2016-07-07 15:58:30 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2016-07-07 18:59:55 +0200 |
commit | 768a6fc9c0ce926fc026eb0c93001b3058798917 (patch) | |
tree | 448dfc402ce7518d1bf46c992ceb4fd1b99714fd /sdext/source | |
parent | b46492d06e389728bee7a2f2a8560c3891ef099b (diff) |
loplugin:passstuffbyref also for {css::uno,rtl}::Reference
Change-Id: I3774f16c09b89fc6888bff3a388c290c32bb8a14
Diffstat (limited to 'sdext/source')
-rw-r--r-- | sdext/source/minimizer/optimizerdialog.cxx | 2 | ||||
-rw-r--r-- | sdext/source/minimizer/optimizerdialog.hxx | 2 | ||||
-rw-r--r-- | sdext/source/minimizer/unodialog.cxx | 8 | ||||
-rw-r--r-- | sdext/source/minimizer/unodialog.hxx | 2 | ||||
-rw-r--r-- | sdext/source/pdfimport/tree/genericelements.hxx | 2 | ||||
-rw-r--r-- | sdext/source/pdfimport/tree/pdfiprocessor.cxx | 2 | ||||
-rw-r--r-- | sdext/source/pdfimport/tree/pdfiprocessor.hxx | 2 |
7 files changed, 10 insertions, 10 deletions
diff --git a/sdext/source/minimizer/optimizerdialog.cxx b/sdext/source/minimizer/optimizerdialog.cxx index 9b3d7c723024..700765df0e90 100644 --- a/sdext/source/minimizer/optimizerdialog.cxx +++ b/sdext/source/minimizer/optimizerdialog.cxx @@ -196,7 +196,7 @@ void OptimizerDialog::UpdateConfiguration() } -OptimizerDialog::OptimizerDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame >& rxFrame, Reference< XDispatch > rxStatusDispatcher ) : +OptimizerDialog::OptimizerDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame >& rxFrame, Reference< XDispatch > const & rxStatusDispatcher ) : UnoDialog( rxContext, rxFrame ), ConfigurationAccess( rxContext, nullptr ), mnCurrentStep( 0 ), diff --git a/sdext/source/minimizer/optimizerdialog.hxx b/sdext/source/minimizer/optimizerdialog.hxx index 330ba80539ec..4b01f1792852 100644 --- a/sdext/source/minimizer/optimizerdialog.hxx +++ b/sdext/source/minimizer/optimizerdialog.hxx @@ -60,7 +60,7 @@ class OptimizerDialog : public UnoDialog, public ConfigurationAccess public: OptimizerDialog( const css::uno::Reference< css::uno::XComponentContext >& rxContext, css::uno::Reference< css::frame::XFrame >& rxFrame, - css::uno::Reference< css::frame::XDispatch > rxStatusDispatcher ); + css::uno::Reference< css::frame::XDispatch > const & rxStatusDispatcher ); ~OptimizerDialog(); void execute(); diff --git a/sdext/source/minimizer/unodialog.cxx b/sdext/source/minimizer/unodialog.cxx index d01bce9e3e05..9bb02189c567 100644 --- a/sdext/source/minimizer/unodialog.cxx +++ b/sdext/source/minimizer/unodialog.cxx @@ -83,15 +83,15 @@ void UnoDialog::endExecute( bool bStatus ) } -Reference< XWindowPeer > UnoDialog::createWindowPeer( Reference< XWindowPeer > xParentPeer ) +Reference< XWindowPeer > UnoDialog::createWindowPeer( Reference< XWindowPeer > const & xParentPeer ) throw ( Exception ) { mxDialog->setVisible( false ); Reference< XToolkit > xToolkit( Toolkit::create( mxContext ), UNO_QUERY_THROW ); - if ( !xParentPeer.is() ) - xParentPeer = xToolkit->getDesktopWindow(); mxReschedule.set( xToolkit, UNO_QUERY ); - mxDialog->createPeer( xToolkit, xParentPeer ); + mxDialog->createPeer( + xToolkit, + xParentPeer.is() ? xParentPeer : xToolkit->getDesktopWindow()); // xWindowPeer = xControl.getPeer(); return mxDialog->getPeer(); } diff --git a/sdext/source/minimizer/unodialog.hxx b/sdext/source/minimizer/unodialog.hxx index f80f12295025..9ae526ec6351 100644 --- a/sdext/source/minimizer/unodialog.hxx +++ b/sdext/source/minimizer/unodialog.hxx @@ -60,7 +60,7 @@ public: void execute(); void endExecute( bool bStatus ); - css::uno::Reference< css::awt::XWindowPeer > createWindowPeer( css::uno::Reference< css::awt::XWindowPeer > xParentPeer ) + css::uno::Reference< css::awt::XWindowPeer > createWindowPeer( css::uno::Reference< css::awt::XWindowPeer > const & xParentPeer ) throw ( css::uno::Exception ); css::uno::Reference< css::uno::XInterface > insertControlModel( const OUString& rServiceName, const OUString& rName, diff --git a/sdext/source/pdfimport/tree/genericelements.hxx b/sdext/source/pdfimport/tree/genericelements.hxx index 55b6b00306af..21b2c884823f 100644 --- a/sdext/source/pdfimport/tree/genericelements.hxx +++ b/sdext/source/pdfimport/tree/genericelements.hxx @@ -50,7 +50,7 @@ namespace pdfi PDFIProcessor& _rProcessor, const css::uno::Reference< css::task::XStatusIndicator>& _xStatusIndicator, - css::uno::Reference< css::uno::XComponentContext > xContext) + css::uno::Reference< css::uno::XComponentContext > const & xContext) : rEmitter(_rEmitter), rStyles(_rStyles), diff --git a/sdext/source/pdfimport/tree/pdfiprocessor.cxx b/sdext/source/pdfimport/tree/pdfiprocessor.cxx index 69c9066008b7..e15f4ad27185 100644 --- a/sdext/source/pdfimport/tree/pdfiprocessor.cxx +++ b/sdext/source/pdfimport/tree/pdfiprocessor.cxx @@ -53,7 +53,7 @@ namespace pdfi { PDFIProcessor::PDFIProcessor( const uno::Reference< task::XStatusIndicator >& xStat , - css::uno::Reference< css::uno::XComponentContext > xContext) : + css::uno::Reference< css::uno::XComponentContext > const & xContext) : m_xContext(xContext), prevCharWidth(0), diff --git a/sdext/source/pdfimport/tree/pdfiprocessor.hxx b/sdext/source/pdfimport/tree/pdfiprocessor.hxx index 672b3337d901..bcce1006232d 100644 --- a/sdext/source/pdfimport/tree/pdfiprocessor.hxx +++ b/sdext/source/pdfimport/tree/pdfiprocessor.hxx @@ -71,7 +71,7 @@ namespace pdfi enum DocumentTextDirecion { LrTb, RlTb, TbLr }; explicit PDFIProcessor( const css::uno::Reference< css::task::XStatusIndicator >& xStat, - css::uno::Reference< css::uno::XComponentContext > xContext) ; + css::uno::Reference< css::uno::XComponentContext > const & xContext) ; /// TEMP - enable writer-like text:p on doc level void enableToplevelText(); |