[Libreoffice-commits] core.git: extensions/source scripting/source sdext/source

Noel Grandin noel.grandin at collabora.co.uk
Mon Jul 31 11:05:48 UTC 2017


 extensions/source/abpilot/datasourcehandling.cxx   |    2 +-
 extensions/source/dbpilots/controlwizard.cxx       |    2 +-
 extensions/source/dbpilots/controlwizard.hxx       |    2 +-
 extensions/source/propctrlr/usercontrol.cxx        |    2 +-
 extensions/source/propctrlr/usercontrol.hxx        |    2 +-
 extensions/source/scanner/grid.cxx                 |    8 ++++----
 extensions/source/update/ui/updatecheckui.cxx      |    4 ++--
 scripting/source/dlgprov/dlgevtatt.cxx             |    2 +-
 scripting/source/dlgprov/dlgevtatt.hxx             |    2 +-
 scripting/source/inc/util/MiscUtils.hxx            |    2 +-
 scripting/source/stringresource/stringresource.cxx |    4 ++--
 scripting/source/stringresource/stringresource.hxx |    4 ++--
 sdext/source/minimizer/informationdialog.cxx       |    2 +-
 sdext/source/minimizer/informationdialog.hxx       |    4 ++--
 sdext/source/minimizer/optimizerdialog.cxx         |    2 +-
 sdext/source/minimizer/optimizerdialog.hxx         |    2 +-
 sdext/source/minimizer/unodialog.cxx               |    2 +-
 sdext/source/minimizer/unodialog.hxx               |    2 +-
 18 files changed, 25 insertions(+), 25 deletions(-)

New commits:
commit 76badcaf7a426458be55355427d6ef689d62ac69
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date:   Mon Jul 31 10:36:32 2017 +0200

    loplugin:constparams in extensions,scripting,sdext
    
    Change-Id: I3d47e144db17358bf6e2e5e3f4ad29de888c01b1
    Reviewed-on: https://gerrit.libreoffice.org/40583
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/extensions/source/abpilot/datasourcehandling.cxx b/extensions/source/abpilot/datasourcehandling.cxx
index dbefd25ae7d3..bb3eae445187 100644
--- a/extensions/source/abpilot/datasourcehandling.cxx
+++ b/extensions/source/abpilot/datasourcehandling.cxx
@@ -55,7 +55,7 @@ namespace
 {
 
 /// Returns the URL of this object shell.
-OUString lcl_getOwnURL(SfxObjectShell* pObjectShell)
+OUString lcl_getOwnURL(SfxObjectShell const * pObjectShell)
 {
     OUString aRet;
 
diff --git a/extensions/source/dbpilots/controlwizard.cxx b/extensions/source/dbpilots/controlwizard.cxx
index e3b9b80bcd07..d7e4881d6b08 100644
--- a/extensions/source/dbpilots/controlwizard.cxx
+++ b/extensions/source/dbpilots/controlwizard.cxx
@@ -630,7 +630,7 @@ namespace dbp
     }
 
 
-    void OControlWizard::commitControlSettings(OControlWizardSettings* _pSettings)
+    void OControlWizard::commitControlSettings(OControlWizardSettings const * _pSettings)
     {
         DBG_ASSERT(m_aContext.xObjectModel.is(), "OControlWizard::commitControlSettings: have no control model to work with!");
         if (!m_aContext.xObjectModel.is())
diff --git a/extensions/source/dbpilots/controlwizard.hxx b/extensions/source/dbpilots/controlwizard.hxx
index faebb6edd7f2..4ae22c35f5d9 100644
--- a/extensions/source/dbpilots/controlwizard.hxx
+++ b/extensions/source/dbpilots/controlwizard.hxx
@@ -126,7 +126,7 @@ namespace dbp
         // with some common data extracted from the control model
         void initControlSettings(OControlWizardSettings* _pSettings);
         // commit the control-relevant settings
-        void commitControlSettings(OControlWizardSettings* _pSettings);
+        void commitControlSettings(OControlWizardSettings const * _pSettings);
 
         bool needDatasourceSelection();
 
diff --git a/extensions/source/propctrlr/usercontrol.cxx b/extensions/source/propctrlr/usercontrol.cxx
index 66186a56cdbc..a030a867deed 100644
--- a/extensions/source/propctrlr/usercontrol.cxx
+++ b/extensions/source/propctrlr/usercontrol.cxx
@@ -141,7 +141,7 @@ namespace pcr
     }
 
 
-    double OFormatSampleControl::getPreviewValue(SvNumberFormatter* _pNF,sal_Int32 _nFormatKey)
+    double OFormatSampleControl::getPreviewValue(SvNumberFormatter const * _pNF, sal_Int32 _nFormatKey)
     {
         const SvNumberformat* pEntry = _pNF->GetEntry(_nFormatKey);
         DBG_ASSERT( pEntry, "OFormattedNumericControl::SetFormatDescription: invalid format key!" );
diff --git a/extensions/source/propctrlr/usercontrol.hxx b/extensions/source/propctrlr/usercontrol.hxx
index 6fc75e2e007f..06e01d8a00ef 100644
--- a/extensions/source/propctrlr/usercontrol.hxx
+++ b/extensions/source/propctrlr/usercontrol.hxx
@@ -77,7 +77,7 @@ namespace pcr
         * \param _nFormatKey the format key
         * \return current date or time or the value 1234.56789
         */
-        static double getPreviewValue(SvNumberFormatter* _pNF,sal_Int32 _nFormatKey);
+        static double getPreviewValue(SvNumberFormatter const * _pNF, sal_Int32 _nFormatKey);
 
     private:
         static double getPreviewValue( const SvNumberformat& i_rEntry );
diff --git a/extensions/source/scanner/grid.cxx b/extensions/source/scanner/grid.cxx
index 9867fc6d000a..ca51e40f32bb 100644
--- a/extensions/source/scanner/grid.cxx
+++ b/extensions/source/scanner/grid.cxx
@@ -57,7 +57,7 @@ class GridWindow : public vcl::Window
             rRenderContext.DrawBitmapEx(maPos - aOffset, rBitmapEx);
         }
 
-        bool isHit(vcl::Window& rWin, const Point& rPos)
+        bool isHit(vcl::Window const & rWin, const Point& rPos)
         {
             const Point aOffset(rWin.PixelToLogic(Point(mnOffX, mnOffY)));
             const tools::Rectangle aTarget(maPos - aOffset, maPos + aOffset);
@@ -111,7 +111,7 @@ class GridWindow : public vcl::Window
     void computeExtremes();
     static void computeChunk( double fMin, double fMax, double& fChunkOut, double& fMinChunkOut );
     void computeNew();
-    static double interpolate( double x, double* pNodeX, double* pNodeY, int nNodes );
+    static double interpolate( double x, double const * pNodeX, double const * pNodeY, int nNodes );
 
     virtual void MouseMove( const MouseEvent& ) override;
     virtual void MouseButtonDown( const MouseEvent& ) override;
@@ -414,8 +414,8 @@ void GridWindow::computeNew()
 
 double GridWindow::interpolate(
     double x,
-    double* pNodeX,
-    double* pNodeY,
+    double const * pNodeX,
+    double const * pNodeY,
     int nNodes )
 {
     // compute Lagrange interpolation
diff --git a/extensions/source/update/ui/updatecheckui.cxx b/extensions/source/update/ui/updatecheckui.cxx
index bf530dbaffd2..b271c8f4eea2 100644
--- a/extensions/source/update/ui/updatecheckui.cxx
+++ b/extensions/source/update/ui/updatecheckui.cxx
@@ -159,7 +159,7 @@ private:
     VclPtr<BubbleWindow> GetBubbleWindow();
     void            RemoveBubbleWindow( bool bRemoveIcon );
     void            AddMenuBarIcon( SystemWindow* pSysWin, bool bAddEventHdl );
-    Image           GetBubbleImage( OUString &rURL );
+    Image           GetBubbleImage( OUString const &rURL );
 
 public:
     explicit        UpdateCheckUI(const uno::Reference<uno::XComponentContext>&);
@@ -238,7 +238,7 @@ UpdateCheckUI::supportsService( OUString const & serviceName )
     return cppu::supportsService(this, serviceName);
 }
 
-Image UpdateCheckUI::GetBubbleImage( OUString &rURL )
+Image UpdateCheckUI::GetBubbleImage( OUString const &rURL )
 {
     Image aImage;
 
diff --git a/scripting/source/dlgprov/dlgevtatt.cxx b/scripting/source/dlgprov/dlgevtatt.cxx
index 9de5689b8bdc..8b88edb4bff4 100644
--- a/scripting/source/dlgprov/dlgevtatt.cxx
+++ b/scripting/source/dlgprov/dlgevtatt.cxx
@@ -194,7 +194,7 @@ namespace dlgprov
             throw RuntimeException(); // more text info here please
         return it->second;
     }
-    Reference< XScriptEventsSupplier > DialogEventsAttacherImpl::getFakeVbaEventsSupplier( const Reference< XControl >& xControl, OUString& sControlName )
+    Reference< XScriptEventsSupplier > DialogEventsAttacherImpl::getFakeVbaEventsSupplier( const Reference< XControl >& xControl, OUString const & sControlName )
     {
         Reference< XScriptEventsSupplier > xEventsSupplier;
         Reference< XMultiComponentFactory > xSMgr( m_xContext->getServiceManager() );
diff --git a/scripting/source/dlgprov/dlgevtatt.hxx b/scripting/source/dlgprov/dlgevtatt.hxx
index 392ffcdc9c70..814d71dbde7b 100644
--- a/scripting/source/dlgprov/dlgevtatt.hxx
+++ b/scripting/source/dlgprov/dlgevtatt.hxx
@@ -54,7 +54,7 @@ namespace dlgprov
         css::uno::Reference< css::script::XEventAttacher > m_xEventAttacher;
         /// @throws css::uno::RuntimeException
         css::uno::Reference< css::script::XScriptListener > getScriptListenerForKey( const OUString& sScriptName );
-        css::uno::Reference< css::script::XScriptEventsSupplier > getFakeVbaEventsSupplier( const css::uno::Reference< css::awt::XControl>& xControl, OUString& sCodeName );
+        css::uno::Reference< css::script::XScriptEventsSupplier > getFakeVbaEventsSupplier( const css::uno::Reference< css::awt::XControl>& xControl, OUString const & sCodeName );
         void nestedAttachEvents( const css::uno::Sequence< css::uno::Reference< css::uno::XInterface > >& Objects, const css::uno::Any& Helper, OUString& sDialogCodeName );
         void attachEventsToControl( const css::uno::Reference< css::awt::XControl>& xControl, const css::uno::Reference< css::script::XScriptEventsSupplier >& events, const css::uno::Any& Helper  );
     public:
diff --git a/scripting/source/inc/util/MiscUtils.hxx b/scripting/source/inc/util/MiscUtils.hxx
index e503eb4faeea..5e0b1c8848e4 100644
--- a/scripting/source/inc/util/MiscUtils.hxx
+++ b/scripting/source/inc/util/MiscUtils.hxx
@@ -126,7 +126,7 @@ static css::uno::Reference< css::frame::XModel > tDocUrlToModel( const OUString&
 }
 
 
-static css::uno::Any getUCBProperty( ::ucbhelper::Content& content, OUString& prop )
+static css::uno::Any getUCBProperty( ::ucbhelper::Content& content, OUString const & prop )
 {
     css::uno::Any result;
     try
diff --git a/scripting/source/stringresource/stringresource.cxx b/scripting/source/stringresource/stringresource.cxx
index 100af7be69ea..6681920e4dc5 100644
--- a/scripting/source/stringresource/stringresource.cxx
+++ b/scripting/source/stringresource/stringresource.cxx
@@ -1701,7 +1701,7 @@ OUString implGetNameScemeForLocaleItem( const LocaleItem* pLocaleItem )
 }
 
 OUString StringResourcePersistenceImpl::implGetFileNameForLocaleItem
-    ( LocaleItem* pLocaleItem, const OUString& aNameBase )
+    ( LocaleItem const * pLocaleItem, const OUString& aNameBase )
 {
     OUString aFileName = aNameBase;
     if( aFileName.isEmpty() )
@@ -1712,7 +1712,7 @@ OUString StringResourcePersistenceImpl::implGetFileNameForLocaleItem
 }
 
 OUString StringResourcePersistenceImpl::implGetPathForLocaleItem
-    ( LocaleItem* pLocaleItem, const OUString& aNameBase,
+    ( LocaleItem const * pLocaleItem, const OUString& aNameBase,
       const OUString& aLocation, bool bDefaultFile )
 {
     OUString aFileName = implGetFileNameForLocaleItem( pLocaleItem, aNameBase );
diff --git a/scripting/source/stringresource/stringresource.hxx b/scripting/source/stringresource/stringresource.hxx
index c4754b02864a..952dfd4b5d70 100644
--- a/scripting/source/stringresource/stringresource.hxx
+++ b/scripting/source/stringresource/stringresource.hxx
@@ -223,8 +223,8 @@ protected:
     virtual void implLoadAllLocales() override;
 
     void implScanLocaleNames( const css::uno::Sequence< OUString >& aContentSeq );
-    static OUString implGetFileNameForLocaleItem( LocaleItem* pLocaleItem, const OUString& aNameBase );
-    static OUString implGetPathForLocaleItem( LocaleItem* pLocaleItem, const OUString& aNameBase,
+    static OUString implGetFileNameForLocaleItem( LocaleItem const * pLocaleItem, const OUString& aNameBase );
+    static OUString implGetPathForLocaleItem( LocaleItem const * pLocaleItem, const OUString& aNameBase,
         const OUString& aLocation, bool bDefaultFile=false );
 
     bool implReadPropertiesFile( LocaleItem* pLocaleItem,
diff --git a/sdext/source/minimizer/informationdialog.cxx b/sdext/source/minimizer/informationdialog.cxx
index 470512571c75..3710e3dfccca 100644
--- a/sdext/source/minimizer/informationdialog.cxx
+++ b/sdext/source/minimizer/informationdialog.cxx
@@ -303,7 +303,7 @@ void InformationDialog::InitDialog()
 }
 
 
-InformationDialog::InformationDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame >& rxFrame, const OUString& rSaveAsURL, bool& rbOpenNewDocument, sal_Int64 rSourceSize, sal_Int64 rDestSize, sal_Int64 rApproxSize ) :
+InformationDialog::InformationDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame > const & rxFrame, const OUString& rSaveAsURL, bool& rbOpenNewDocument, sal_Int64 rSourceSize, sal_Int64 rDestSize, sal_Int64 rApproxSize ) :
     UnoDialog( rxContext, rxFrame ),
     ConfigurationAccess( rxContext ),
     mxActionListener( new OKActionListener( *this ) ),
diff --git a/sdext/source/minimizer/informationdialog.hxx b/sdext/source/minimizer/informationdialog.hxx
index 60c69bba124c..f70a60c51281 100644
--- a/sdext/source/minimizer/informationdialog.hxx
+++ b/sdext/source/minimizer/informationdialog.hxx
@@ -42,8 +42,8 @@ class InformationDialog : public UnoDialog, public ConfigurationAccess
 public:
 
     InformationDialog( const css::uno::Reference< css::uno::XComponentContext >& rxContext,
-            css::uno::Reference< css::frame::XFrame >& rxFrame, const OUString& rSaveAsURL,
-                bool& bOpenNewDocument, sal_Int64 nSourceSize, sal_Int64 nDestSize, sal_Int64 nApproxDest );
+                       css::uno::Reference< css::frame::XFrame > const & rxFrame, const OUString& rSaveAsURL,
+                       bool& bOpenNewDocument, sal_Int64 nSourceSize, sal_Int64 nDestSize, sal_Int64 nApproxDest );
     ~InformationDialog();
 
     void                execute();
diff --git a/sdext/source/minimizer/optimizerdialog.cxx b/sdext/source/minimizer/optimizerdialog.cxx
index c6f07fd63840..671e0620d6e5 100644
--- a/sdext/source/minimizer/optimizerdialog.cxx
+++ b/sdext/source/minimizer/optimizerdialog.cxx
@@ -192,7 +192,7 @@ void OptimizerDialog::UpdateConfiguration()
 }
 
 
-OptimizerDialog::OptimizerDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame >& rxFrame, Reference< XDispatch > const & rxStatusDispatcher ) :
+OptimizerDialog::OptimizerDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame > const & rxFrame, Reference< XDispatch > const & rxStatusDispatcher ) :
     UnoDialog( rxContext, rxFrame ),
     ConfigurationAccess( rxContext ),
     mnCurrentStep( 0 ),
diff --git a/sdext/source/minimizer/optimizerdialog.hxx b/sdext/source/minimizer/optimizerdialog.hxx
index 7fa161e3b5f4..9d421488046a 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,
+    OptimizerDialog( const css::uno::Reference< css::uno::XComponentContext >& rxContext, css::uno::Reference< css::frame::XFrame > const & rxFrame,
         css::uno::Reference< css::frame::XDispatch > const & rxStatusDispatcher );
     ~OptimizerDialog();
 
diff --git a/sdext/source/minimizer/unodialog.cxx b/sdext/source/minimizer/unodialog.cxx
index fd2a3a00e62d..8dc025ba9c37 100644
--- a/sdext/source/minimizer/unodialog.cxx
+++ b/sdext/source/minimizer/unodialog.cxx
@@ -38,7 +38,7 @@ using namespace ::com::sun::star::view;
 using namespace ::com::sun::star::frame;
 using namespace ::com::sun::star::beans;
 
-UnoDialog::UnoDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame >& rxFrame ) :
+UnoDialog::UnoDialog( const Reference< XComponentContext > &rxContext, Reference< XFrame > const & rxFrame ) :
     mxContext( rxContext ),
     mxController( rxFrame->getController() ),
     mxDialogModel( mxContext->getServiceManager()->createInstanceWithContext(
diff --git a/sdext/source/minimizer/unodialog.hxx b/sdext/source/minimizer/unodialog.hxx
index 76db2f8d8fd6..1903f2da7fdb 100644
--- a/sdext/source/minimizer/unodialog.hxx
+++ b/sdext/source/minimizer/unodialog.hxx
@@ -53,7 +53,7 @@ class UnoDialog
 {
 public:
 
-    UnoDialog( const css::uno::Reference< css::uno::XComponentContext >& rxMSF, css::uno::Reference< css::frame::XFrame >& rxFrame );
+    UnoDialog( const css::uno::Reference< css::uno::XComponentContext >& rxMSF, css::uno::Reference< css::frame::XFrame > const & rxFrame );
     ~UnoDialog();
 
     void execute();


More information about the Libreoffice-commits mailing list