[Libreoffice-commits] core.git: include/svx sd/source svx/source

Noel (via logerrit) logerrit at kemper.freedesktop.org
Mon Mar 1 18:31:22 UTC 2021


 include/svx/ShapeTypeHandler.hxx              |    2 +-
 sd/source/ui/accessibility/SdShapeTypes.cxx   |    2 +-
 svx/source/accessibility/ShapeTypeHandler.cxx |    2 +-
 svx/source/accessibility/SvxShapeTypes.cxx    |    2 +-
 svx/source/form/filtnav.cxx                   |    2 +-
 svx/source/form/fmexch.cxx                    |    2 +-
 svx/source/form/fmtextcontrolshell.cxx        |    6 +++---
 svx/source/inc/filtnav.hxx                    |    2 +-
 svx/source/inc/fmexch.hxx                     |    4 ++--
 svx/source/inc/fmtextcontrolshell.hxx         |    2 +-
 10 files changed, 13 insertions(+), 13 deletions(-)

New commits:
commit 3c699b2a480e0795a3a7085c2bcd1a9e8d649ed2
Author:     Noel <noel.grandin at collabora.co.uk>
AuthorDate: Mon Mar 1 14:35:54 2021 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Mon Mar 1 19:30:37 2021 +0100

    loplugin:refcounting in svx
    
    Change-Id: I87b1d0684e6e8921b222f76a13a36706c0eb8c87
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111750
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/include/svx/ShapeTypeHandler.hxx b/include/svx/ShapeTypeHandler.hxx
index ccf61121876f..15992311dfc2 100644
--- a/include/svx/ShapeTypeHandler.hxx
+++ b/include/svx/ShapeTypeHandler.hxx
@@ -44,7 +44,7 @@ typedef int ShapeTypeId;
 /** Define the function type for creating accessible objects for given
     service names.
 */
-typedef AccessibleShape* (*tCreateFunction)
+typedef rtl::Reference<AccessibleShape> (*tCreateFunction)
     (const AccessibleShapeInfo& rShapeInfo,
     const AccessibleShapeTreeInfo& rShapeTreeInfo,
     ShapeTypeId nId);
diff --git a/sd/source/ui/accessibility/SdShapeTypes.cxx b/sd/source/ui/accessibility/SdShapeTypes.cxx
index a6159b05f7a0..7fab0961ee27 100644
--- a/sd/source/ui/accessibility/SdShapeTypes.cxx
+++ b/sd/source/ui/accessibility/SdShapeTypes.cxx
@@ -25,7 +25,7 @@
 
 namespace accessibility {
 
-static AccessibleShape*
+static rtl::Reference<AccessibleShape>
     CreateSdAccessibleShape (
         const AccessibleShapeInfo& rShapeInfo,
         const AccessibleShapeTreeInfo& rShapeTreeInfo,
diff --git a/svx/source/accessibility/ShapeTypeHandler.cxx b/svx/source/accessibility/ShapeTypeHandler.cxx
index 03ef0bf86bb6..368c2f698ee9 100644
--- a/svx/source/accessibility/ShapeTypeHandler.cxx
+++ b/svx/source/accessibility/ShapeTypeHandler.cxx
@@ -40,7 +40,7 @@ ShapeTypeHandler* ShapeTypeHandler::instance = nullptr;
 
 
 // Create an empty reference to an accessible object.
-static AccessibleShape*
+static rtl::Reference<AccessibleShape>
     CreateEmptyShapeReference (
         const AccessibleShapeInfo& /*rShapeInfo*/,
         const AccessibleShapeTreeInfo& /*rShapeTreeInfo*/,
diff --git a/svx/source/accessibility/SvxShapeTypes.cxx b/svx/source/accessibility/SvxShapeTypes.cxx
index 5ca2804b5d7f..b02a153a4cdb 100644
--- a/svx/source/accessibility/SvxShapeTypes.cxx
+++ b/svx/source/accessibility/SvxShapeTypes.cxx
@@ -28,7 +28,7 @@
 
 namespace accessibility {
 
-static AccessibleShape* CreateSvxAccessibleShape (
+static rtl::Reference<AccessibleShape> CreateSvxAccessibleShape (
     const AccessibleShapeInfo& rShapeInfo,
     const AccessibleShapeTreeInfo& rShapeTreeInfo,
     ShapeTypeId nId)
diff --git a/svx/source/form/filtnav.cxx b/svx/source/form/filtnav.cxx
index 7e9e53c62d7f..cee9f5d3c71b 100644
--- a/svx/source/form/filtnav.cxx
+++ b/svx/source/form/filtnav.cxx
@@ -103,7 +103,7 @@ SotClipboardFormatId OFilterItemExchange::getFormatId()
     return s_nFormat;
 }
 
-OLocalExchange* OFilterExchangeHelper::createExchange() const
+rtl::Reference<OLocalExchange> OFilterExchangeHelper::createExchange() const
 {
     return new OFilterItemExchange;
 }
diff --git a/svx/source/form/fmexch.cxx b/svx/source/form/fmexch.cxx
index 38e5fbd947cb..002b3b6175fc 100644
--- a/svx/source/form/fmexch.cxx
+++ b/svx/source/form/fmexch.cxx
@@ -306,7 +306,7 @@ namespace svxform
     }
 
     //= OControlExchangeHelper
-    OLocalExchange* OControlExchangeHelper::createExchange() const
+    rtl::Reference<OLocalExchange> OControlExchangeHelper::createExchange() const
     {
         return new OControlExchange;
     }
diff --git a/svx/source/form/fmtextcontrolshell.cxx b/svx/source/form/fmtextcontrolshell.cxx
index a9ec64108a85..0818e9c7be8a 100644
--- a/svx/source/form/fmtextcontrolshell.cxx
+++ b/svx/source/form/fmtextcontrolshell.cxx
@@ -1226,9 +1226,9 @@ namespace svx
             SfxSlotId* pSlots = _pZeroTerminatedSlots;
             while ( *pSlots )
             {
-                FmTextControlFeature* pDispatcher = implGetFeatureDispatcher( xProvider, pApplication, *pSlots );
+                rtl::Reference<FmTextControlFeature> pDispatcher = implGetFeatureDispatcher( xProvider, pApplication, *pSlots );
                 if ( pDispatcher )
-                    _rDispatchers.emplace( *pSlots, ControlFeature( pDispatcher ) );
+                    _rDispatchers.emplace( *pSlots, pDispatcher );
 
                 ++pSlots;
             }
@@ -1236,7 +1236,7 @@ namespace svx
     }
 
 
-    FmTextControlFeature* FmTextControlShell::implGetFeatureDispatcher( const Reference< XDispatchProvider >& _rxProvider, SfxApplication const * _pApplication, SfxSlotId _nSlot )
+    rtl::Reference<FmTextControlFeature> FmTextControlShell::implGetFeatureDispatcher( const Reference< XDispatchProvider >& _rxProvider, SfxApplication const * _pApplication, SfxSlotId _nSlot )
     {
         OSL_PRECOND( _rxProvider.is() && _pApplication, "FmTextControlShell::implGetFeatureDispatcher: invalid arg(s)!" );
         URL aFeatureURL;
diff --git a/svx/source/inc/filtnav.hxx b/svx/source/inc/filtnav.hxx
index 5e6e91b58f1c..04b4d1991f71 100644
--- a/svx/source/inc/filtnav.hxx
+++ b/svx/source/inc/filtnav.hxx
@@ -209,7 +209,7 @@ public:
     OFilterItemExchange& operator*() const { return *static_cast<OFilterItemExchange*>(m_xTransferable.get()); }
 
 protected:
-    virtual OLocalExchange* createExchange() const override;
+    virtual rtl::Reference<OLocalExchange> createExchange() const override;
 };
 
 class FmFilterNavigator;
diff --git a/svx/source/inc/fmexch.hxx b/svx/source/inc/fmexch.hxx
index d3a7fdc969f5..ad8efc7faafe 100644
--- a/svx/source/inc/fmexch.hxx
+++ b/svx/source/inc/fmexch.hxx
@@ -104,7 +104,7 @@ namespace svxform
         SVX_DLLPRIVATE void     setClipboardListener( const Link<OLocalExchange&,void>& _rListener ) { if ( m_xTransferable.is() ) m_xTransferable->setClipboardListener( _rListener ); }
 
     protected:
-        SVX_DLLPRIVATE virtual OLocalExchange* createExchange() const = 0;
+        SVX_DLLPRIVATE virtual rtl::Reference<OLocalExchange> createExchange() const = 0;
 
     protected:
         SVX_DLLPRIVATE void implReset();
@@ -215,7 +215,7 @@ namespace svxform
         OControlExchange& operator*() const { return *static_cast< OControlExchange* >( m_xTransferable.get() ); }
 
     protected:
-        virtual OLocalExchange* createExchange() const override;
+        virtual rtl::Reference<OLocalExchange> createExchange() const override;
     };
 
 
diff --git a/svx/source/inc/fmtextcontrolshell.hxx b/svx/source/inc/fmtextcontrolshell.hxx
index 42a61dea27fa..4f6714de4d76 100644
--- a/svx/source/inc/fmtextcontrolshell.hxx
+++ b/svx/source/inc/fmtextcontrolshell.hxx
@@ -151,7 +151,7 @@ namespace svx
     private:
         bool    isControllerListening() const { return !m_aControlObservers.empty(); }
 
-        FmTextControlFeature*
+        rtl::Reference<FmTextControlFeature>
                         implGetFeatureDispatcher(
                             const css::uno::Reference< css::frame::XDispatchProvider >& _rxProvider,
                             SfxApplication const * _pApplication,


More information about the Libreoffice-commits mailing list