[PATCH] fdo#59090 clean up memory leaks

Joren De Cuyper (via Code Review) gerrit at gerrit.libreoffice.org
Mon Jan 7 14:47:17 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1569

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/69/1569/1

fdo#59090 clean up memory leaks

Change-Id: Ifbade82383694ebac0e82920014cddfff5b12fcb
---
M basic/source/classes/sb.cxx
M cui/source/options/optdict.cxx
M embeddedobj/source/commonembedding/xfactory.cxx
M embeddedobj/source/general/xcreator.cxx
M embeddedobj/source/msole/xdialogcreator.cxx
M extensions/source/propctrlr/defaultforminspection.cxx
M extensions/source/propctrlr/defaulthelpprovider.cxx
M forms/source/component/ComboBox.cxx
M forms/source/runtime/formoperations.cxx
M fpicker/source/office/OfficeFilePicker.cxx
M fpicker/source/office/OfficeFolderPicker.cxx
M package/source/zippackage/ZipPackage.cxx
M package/source/zippackage/zipfileaccess.cxx
M sc/source/ui/unoobj/servuno.cxx
M sfx2/source/appl/xpackcreator.cxx
M sfx2/source/doc/objcont.cxx
M sfx2/source/doc/objstor.cxx
M sfx2/source/doc/ownsubfilterservice.cxx
M sfx2/source/view/frmload.cxx
M starmath/source/smdetect.cxx
M stoc/source/javaloader/javaloader.cxx
M stoc/source/namingservice/namingservice.cxx
M stoc/test/testiadapter.cxx
M svl/source/fsstor/fsfactory.cxx
M svl/source/passwordcontainer/passwordcontainer.cxx
M svtools/source/hatchwindow/hatchwindowfactory.cxx
M svx/source/fmcomp/gridcell.cxx
M svx/source/sdr/contact/viewcontactofunocontrol.cxx
M svx/source/tbxctrls/extrusioncontrols.cxx
M svx/source/tbxctrls/fontworkgallery.cxx
M svx/source/tbxctrls/tbunocontroller.cxx
M svx/source/unodraw/unobtabl.cxx
M svx/source/unodraw/unoctabl.cxx
M svx/source/unodraw/unodtabl.cxx
M svx/source/unodraw/unogtabl.cxx
M svx/source/unodraw/unohtabl.cxx
M svx/source/unodraw/unomtabl.cxx
M svx/source/unodraw/unottabl.cxx
38 files changed, 52 insertions(+), 52 deletions(-)



diff --git a/basic/source/classes/sb.cxx b/basic/source/classes/sb.cxx
index 12cb5da..b75c47f 100644
--- a/basic/source/classes/sb.cxx
+++ b/basic/source/classes/sb.cxx
@@ -1699,7 +1699,7 @@
     // Implementation of the code for the string transport to SFX-Error
     if( !rMsg.isEmpty() )
     {
-        code = (sal_uIntPtr)*new StringErrorInfo( code, rMsg );
+        code = (sal_uIntPtr) StringErrorInfo( code, rMsg );
     }
     SetErrorData( code, l, c1, c2 );
     GetSbData()->bCompiler = true;
@@ -1743,11 +1743,11 @@
         {
             OUString aTmp = "\'" + OUString::valueOf(SbxErrObject::getUnoErrObject()->getNumber()) +
                             "\'\n" + OUString(!GetSbData()->aErrMsg.isEmpty() ? GetSbData()->aErrMsg : rMsg);
-            code = (sal_uIntPtr)*new StringErrorInfo( code, aTmp );
+            code = (sal_uIntPtr) StringErrorInfo( code, aTmp );
         }
         else
         {
-            code = (sal_uIntPtr)*new StringErrorInfo( code, rMsg );
+            code = (sal_uIntPtr) StringErrorInfo( code, rMsg );
         }
     }
 
diff --git a/cui/source/options/optdict.cxx b/cui/source/options/optdict.cxx
index 9855c8a..32788c3 100644
--- a/cui/source/options/optdict.cxx
+++ b/cui/source/options/optdict.cxx
@@ -168,7 +168,7 @@
         // error: couldn't create new dictionary
         SfxErrorContext aContext( ERRCTX_SVX_LINGU_DICTIONARY, String(),
             this, RID_SVXERRCTX, &CUI_MGR() );
-        ErrorHandler::HandleError( *new StringErrorInfo(
+        ErrorHandler::HandleError( StringErrorInfo(
                 ERRCODE_SVX_LINGU_DICT_NOTWRITEABLE, sDict ) );
 
         EndDialog( RET_CANCEL );
diff --git a/embeddedobj/source/commonembedding/xfactory.cxx b/embeddedobj/source/commonembedding/xfactory.cxx
index ec29eda..a8cd472 100644
--- a/embeddedobj/source/commonembedding/xfactory.cxx
+++ b/embeddedobj/source/commonembedding/xfactory.cxx
@@ -54,7 +54,7 @@
 uno::Reference< uno::XInterface > SAL_CALL OOoEmbeddedObjectFactory::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new OOoEmbeddedObjectFactory( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( OOoEmbeddedObjectFactory( xServiceManager ) );
 }
 
 //-------------------------------------------------------------------------
@@ -493,7 +493,7 @@
 uno::Reference< uno::XInterface > SAL_CALL OOoSpecialEmbeddedObjectFactory::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new OOoSpecialEmbeddedObjectFactory( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( OOoSpecialEmbeddedObjectFactory( xServiceManager ) );
 }
 
 //-------------------------------------------------------------------------
diff --git a/embeddedobj/source/general/xcreator.cxx b/embeddedobj/source/general/xcreator.cxx
index 599fc7c..30def27 100644
--- a/embeddedobj/source/general/xcreator.cxx
+++ b/embeddedobj/source/general/xcreator.cxx
@@ -56,7 +56,7 @@
 uno::Reference< uno::XInterface > SAL_CALL UNOEmbeddedObjectCreator::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new UNOEmbeddedObjectCreator( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( UNOEmbeddedObjectCreator( xServiceManager ) );
 }
 
 //-------------------------------------------------------------------------
diff --git a/embeddedobj/source/msole/xdialogcreator.cxx b/embeddedobj/source/msole/xdialogcreator.cxx
index 5cd43ac..6c4da36 100644
--- a/embeddedobj/source/msole/xdialogcreator.cxx
+++ b/embeddedobj/source/msole/xdialogcreator.cxx
@@ -121,7 +121,7 @@
 uno::Reference< uno::XInterface > SAL_CALL MSOLEDialogObjectCreator::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new MSOLEDialogObjectCreator( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( MSOLEDialogObjectCreator( xServiceManager ) );
 }
 
 //-------------------------------------------------------------------------
diff --git a/extensions/source/propctrlr/defaultforminspection.cxx b/extensions/source/propctrlr/defaultforminspection.cxx
index d25824e..56079c8 100644
--- a/extensions/source/propctrlr/defaultforminspection.cxx
+++ b/extensions/source/propctrlr/defaultforminspection.cxx
@@ -103,7 +103,7 @@
     //------------------------------------------------------------------------
     Reference< XInterface > SAL_CALL DefaultFormComponentInspectorModel::Create( const Reference< XComponentContext >& _rxContext )
     {
-        return *new DefaultFormComponentInspectorModel( _rxContext );
+        return DefaultFormComponentInspectorModel( _rxContext );
     }
 
     //--------------------------------------------------------------------
diff --git a/extensions/source/propctrlr/defaulthelpprovider.cxx b/extensions/source/propctrlr/defaulthelpprovider.cxx
index 528316d..05e1603 100644
--- a/extensions/source/propctrlr/defaulthelpprovider.cxx
+++ b/extensions/source/propctrlr/defaulthelpprovider.cxx
@@ -91,7 +91,7 @@
     //------------------------------------------------------------------------
     Reference< XInterface > SAL_CALL DefaultHelpProvider::Create( const Reference< XComponentContext >& _rxContext )
     {
-        return *new DefaultHelpProvider( _rxContext );
+        return DefaultHelpProvider( _rxContext );
     }
 
     //--------------------------------------------------------------------
diff --git a/forms/source/component/ComboBox.cxx b/forms/source/component/ComboBox.cxx
index fb19bcc..2b0a302 100644
--- a/forms/source/component/ComboBox.cxx
+++ b/forms/source/component/ComboBox.cxx
@@ -74,7 +74,7 @@
 //------------------------------------------------------------------
 InterfaceRef SAL_CALL OComboBoxModel_CreateInstance(const Reference<XMultiServiceFactory>& _rxFactory) throw (RuntimeException)
 {
-    return (*new OComboBoxModel(_rxFactory));
+    return ( OComboBoxModel(_rxFactory) );
 }
 
 //------------------------------------------------------------------------------
diff --git a/forms/source/runtime/formoperations.cxx b/forms/source/runtime/formoperations.cxx
index 996368e..14419c8 100644
--- a/forms/source/runtime/formoperations.cxx
+++ b/forms/source/runtime/formoperations.cxx
@@ -157,7 +157,7 @@
     //--------------------------------------------------------------------
     Reference< XInterface > SAL_CALL FormOperations::Create(const Reference< XMultiServiceFactory >& _rxFactory )
     {
-        return *new FormOperations( _rxFactory );
+        return FormOperations( _rxFactory );
     }
 
     //--------------------------------------------------------------------
diff --git a/fpicker/source/office/OfficeFilePicker.cxx b/fpicker/source/office/OfficeFilePicker.cxx
index 81071c2..324ad9f 100644
--- a/fpicker/source/office/OfficeFilePicker.cxx
+++ b/fpicker/source/office/OfficeFilePicker.cxx
@@ -1162,7 +1162,7 @@
     const Reference< XComponentContext >& rxContext) throw( Exception )
 {
     Reference< XMultiServiceFactory > xServiceManager (rxContext->getServiceManager(), UNO_QUERY_THROW);
-    return Reference< XInterface >( *new SvtFilePicker( xServiceManager ) );
+    return Reference< XInterface >( SvtFilePicker( xServiceManager ) );
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/fpicker/source/office/OfficeFolderPicker.cxx b/fpicker/source/office/OfficeFolderPicker.cxx
index 1cf1d05..88d76ca 100644
--- a/fpicker/source/office/OfficeFolderPicker.cxx
+++ b/fpicker/source/office/OfficeFolderPicker.cxx
@@ -198,7 +198,7 @@
     throw( Exception )
 {
     Reference< XMultiServiceFactory > xServiceManager (rxContext->getServiceManager(), UNO_QUERY_THROW);
-    return Reference< XInterface >( *new SvtFolderPicker( xServiceManager ) );
+    return Reference< XInterface >( SvtFolderPicker( xServiceManager ) );
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/package/source/zippackage/ZipPackage.cxx b/package/source/zippackage/ZipPackage.cxx
index 89b3f2f..ea2355b 100644
--- a/package/source/zippackage/ZipPackage.cxx
+++ b/package/source/zippackage/ZipPackage.cxx
@@ -962,9 +962,9 @@
     if ( aArguments.getLength() )
         aArguments[0] >>= bArg;
     if ( bArg )
-        xRef = *new ZipPackageFolder ( m_xFactory, m_nFormat, m_bAllowRemoveOnInsert );
+        xRef = ZipPackageFolder ( m_xFactory, m_nFormat, m_bAllowRemoveOnInsert );
     else
-        xRef = *new ZipPackageStream ( *this, m_xFactory, m_bAllowRemoveOnInsert );
+        xRef = ZipPackageStream ( *this, m_xFactory, m_bAllowRemoveOnInsert );
 
     return xRef;
 }
@@ -1605,7 +1605,7 @@
 uno::Reference < XInterface >SAL_CALL ZipPackage_createInstance(
     const uno::Reference< XMultiServiceFactory > & xMgr )
 {
-    return uno::Reference< XInterface >( *new ZipPackage( xMgr ) );
+    return uno::Reference< XInterface >( ZipPackage( xMgr ) );
 }
 
 //--------------------------------------------------------
diff --git a/package/source/zippackage/zipfileaccess.cxx b/package/source/zippackage/zipfileaccess.cxx
index 09a6e89..b4031da 100644
--- a/package/source/zippackage/zipfileaccess.cxx
+++ b/package/source/zippackage/zipfileaccess.cxx
@@ -452,7 +452,7 @@
 uno::Reference< uno::XInterface > SAL_CALL OZipFileAccess::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& rxMSF )
 {
-    return uno::Reference< uno::XInterface >( *new OZipFileAccess( comphelper::getComponentContext(rxMSF) ) );
+    return uno::Reference< uno::XInterface >( OZipFileAccess( comphelper::getComponentContext(rxMSF) ) );
 }
 
 //-------------------------------------------------------------------------
diff --git a/sc/source/ui/unoobj/servuno.cxx b/sc/source/ui/unoobj/servuno.cxx
index 52f4841..234955c 100644
--- a/sc/source/ui/unoobj/servuno.cxx
+++ b/sc/source/ui/unoobj/servuno.cxx
@@ -570,14 +570,14 @@
             {
                 sal_Bool bListPos = ( nType == SC_SERVICE_LISTCELLBIND );
                 uno::Reference<sheet::XSpreadsheetDocument> xDoc( pDocShell->GetBaseModel(), uno::UNO_QUERY );
-                xRet.set(*new calc::OCellValueBinding( xDoc, bListPos ));
+                xRet.set( calc::OCellValueBinding( xDoc, bListPos ));
             }
             break;
         case SC_SERVICE_LISTSOURCE:
             if (pDocShell)
             {
                 uno::Reference<sheet::XSpreadsheetDocument> xDoc( pDocShell->GetBaseModel(), uno::UNO_QUERY );
-                xRet.set(*new calc::OCellListSource( xDoc ));
+                xRet.set( calc::OCellListSource( xDoc ));
             }
             break;
         case SC_SERVICE_CELLADDRESS:
@@ -585,13 +585,13 @@
             if (pDocShell)
             {
                 sal_Bool bRange = ( nType == SC_SERVICE_RANGEADDRESS );
-                xRet.set(*new ScAddressConversionObj( pDocShell, bRange ));
+                xRet.set( ScAddressConversionObj( pDocShell, bRange ));
             }
             break;
 
         case SC_SERVICE_CHDATAPROV:
             if (pDocShell && pDocShell->GetDocument())
-                xRet = *new ScChart2DataProvider( pDocShell->GetDocument() );
+                xRet = ScChart2DataProvider( pDocShell->GetDocument() );
             break;
 
         case SC_SERVICE_FORMULAPARS:
diff --git a/sfx2/source/appl/xpackcreator.cxx b/sfx2/source/appl/xpackcreator.cxx
index 302833dc..0abf0b6 100644
--- a/sfx2/source/appl/xpackcreator.cxx
+++ b/sfx2/source/appl/xpackcreator.cxx
@@ -63,7 +63,7 @@
 uno::Reference< uno::XInterface > SAL_CALL OPackageStructureCreator::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new OPackageStructureCreator( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( OPackageStructureCreator( xServiceManager ) );
 }
 
 
diff --git a/sfx2/source/doc/objcont.cxx b/sfx2/source/doc/objcont.cxx
index e7bd7f3..2d7b3f0 100644
--- a/sfx2/source/doc/objcont.cxx
+++ b/sfx2/source/doc/objcont.cxx
@@ -496,7 +496,7 @@
                 bUsedOrUserDefined =
                     pExist->IsUsed() || pExist->IsUserDefined();
                 if( ErrorHandler::HandleError(
-                    *new MessageInfo( ERRCODE_SFXMSG_STYLEREPLACE, aOldName ) )
+                    MessageInfo( ERRCODE_SFXMSG_STYLEREPLACE, aOldName ) )
                     != ERRCODE_BUTTON_OK )
                     return sal_False;
                 else
diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx
index 887be25..7b4c0e6 100644
--- a/sfx2/source/doc/objstor.cxx
+++ b/sfx2/source/doc/objstor.cxx
@@ -2229,7 +2229,7 @@
             io::WrongFormatException e;
             if (rWrapped.TargetException >>= e)
             {
-                SetError(*new StringErrorInfo(ERRCODE_SFX_FORMAT_ROWCOL,
+                SetError( StringErrorInfo(ERRCODE_SFX_FORMAT_ROWCOL,
                     e.Message, ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ), "");
             }
         }
diff --git a/sfx2/source/doc/ownsubfilterservice.cxx b/sfx2/source/doc/ownsubfilterservice.cxx
index 1cb5181..a92df39 100644
--- a/sfx2/source/doc/ownsubfilterservice.cxx
+++ b/sfx2/source/doc/ownsubfilterservice.cxx
@@ -58,7 +58,7 @@
 uno::Reference< uno::XInterface > SAL_CALL OwnSubFilterService::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new OwnSubFilterService( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( OwnSubFilterService( xServiceManager ) );
 }
 
 //-------------------------------------------------------------------------
diff --git a/sfx2/source/view/frmload.cxx b/sfx2/source/view/frmload.cxx
index 26b1ec7..a3b8f60 100644
--- a/sfx2/source/view/frmload.cxx
+++ b/sfx2/source/view/frmload.cxx
@@ -695,7 +695,7 @@
 /* Helper for registry */
 UNOREFERENCE< UNOXINTERFACE > SAL_CALL SfxFrameLoader_Impl::impl_createInstance( const UNOREFERENCE< UNOXMULTISERVICEFACTORY >& xServiceManager ) throw( UNOEXCEPTION )
 {
-    return UNOREFERENCE< UNOXINTERFACE >( *new SfxFrameLoader_Impl( xServiceManager ) );
+    return UNOREFERENCE< UNOXINTERFACE >( SfxFrameLoader_Impl( xServiceManager ) );
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/starmath/source/smdetect.cxx b/starmath/source/smdetect.cxx
index da11ce6..3bef594 100644
--- a/starmath/source/smdetect.cxx
+++ b/starmath/source/smdetect.cxx
@@ -461,7 +461,7 @@
 /* Helper for registry */
 UNOREFERENCE< UNOXINTERFACE > SAL_CALL SmFilterDetect::impl_createInstance( const UNOREFERENCE< UNOXMULTISERVICEFACTORY >& xServiceManager ) throw( UNOEXCEPTION )
 {
-    return UNOREFERENCE< UNOXINTERFACE >( *new SmFilterDetect( xServiceManager ) );
+    return UNOREFERENCE< UNOXINTERFACE >( SmFilterDetect( xServiceManager ) );
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/stoc/source/javaloader/javaloader.cxx b/stoc/source/javaloader/javaloader.cxx
index 38029b1..d04bf99 100644
--- a/stoc/source/javaloader/javaloader.cxx
+++ b/stoc/source/javaloader/javaloader.cxx
@@ -401,7 +401,7 @@
         }
         else
         {
-            xRet = *new JavaComponentLoader(xCtx);
+            xRet = JavaComponentLoader(xCtx);
             pStaticRef = new css::uno::Reference< XInterface > ( xRet );
         }
     }
diff --git a/stoc/source/namingservice/namingservice.cxx b/stoc/source/namingservice/namingservice.cxx
index 4f08a9c..1cc5bde 100644
--- a/stoc/source/namingservice/namingservice.cxx
+++ b/stoc/source/namingservice/namingservice.cxx
@@ -114,7 +114,7 @@
 static Reference<XInterface> SAL_CALL NamingService_Impl_create(
     SAL_UNUSED_PARAMETER const Reference<XComponentContext> & )
 {
-    return *new NamingService_Impl();
+    return NamingService_Impl();
 }
 
 //==================================================================================================
diff --git a/stoc/test/testiadapter.cxx b/stoc/test/testiadapter.cxx
index 22230d6..f06e3db 100644
--- a/stoc/test/testiadapter.cxx
+++ b/stoc/test/testiadapter.cxx
@@ -750,7 +750,7 @@
         // this data is never ever granted access to by calls other than equals(), assign()!
         test::TestData aData; // test against this data
 
-        Reference<XInterface > xI( *new OWeakObject() );
+        Reference<XInterface > xI( OWeakObject() );
 
         assign( (test::TestElement &)aData,
                 sal_True, '@', 17, 0x1234, 0xfedc, 0x12345678, 0xfedcba98,
diff --git a/svl/source/fsstor/fsfactory.cxx b/svl/source/fsstor/fsfactory.cxx
index e76c7a6..739ddbf 100644
--- a/svl/source/fsstor/fsfactory.cxx
+++ b/svl/source/fsstor/fsfactory.cxx
@@ -55,7 +55,7 @@
 uno::Reference< uno::XInterface > SAL_CALL FSStorageFactory::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new FSStorageFactory( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( FSStorageFactory( xServiceManager ) );
 }
 
 //-------------------------------------------------------------------------
diff --git a/svl/source/passwordcontainer/passwordcontainer.cxx b/svl/source/passwordcontainer/passwordcontainer.cxx
index ed94353..17a3443 100644
--- a/svl/source/passwordcontainer/passwordcontainer.cxx
+++ b/svl/source/passwordcontainer/passwordcontainer.cxx
@@ -1464,7 +1464,7 @@
 
 Reference< XInterface > SAL_CALL PasswordContainer::impl_createInstance( const Reference< XMultiServiceFactory >& xServiceManager ) throw( RuntimeException )
 {
-    return Reference< XInterface >( *new PasswordContainer( xServiceManager ) );
+    return Reference< XInterface >( PasswordContainer( xServiceManager ) );
 }
 
 //-------------------------------------------------------------------------
diff --git a/svtools/source/hatchwindow/hatchwindowfactory.cxx b/svtools/source/hatchwindow/hatchwindowfactory.cxx
index 02848c2..3065b02 100644
--- a/svtools/source/hatchwindow/hatchwindowfactory.cxx
+++ b/svtools/source/hatchwindow/hatchwindowfactory.cxx
@@ -46,7 +46,7 @@
 uno::Reference< uno::XInterface > SAL_CALL OHatchWindowFactory::impl_staticCreateSelfInstance(
             const uno::Reference< lang::XMultiServiceFactory >& xServiceManager )
 {
-    return uno::Reference< uno::XInterface >( *new OHatchWindowFactory( xServiceManager ) );
+    return uno::Reference< uno::XInterface >( OHatchWindowFactory( xServiceManager ) );
 }
 
 
diff --git a/svx/source/fmcomp/gridcell.cxx b/svx/source/fmcomp/gridcell.cxx
index 2258563..ba428b9 100644
--- a/svx/source/fmcomp/gridcell.cxx
+++ b/svx/source/fmcomp/gridcell.cxx
@@ -4627,7 +4627,7 @@
 //------------------------------------------------------------------------------
 Reference< XInterface >  FmXFilterCell_CreateInstance(const Reference< ::com::sun::star::lang::XMultiServiceFactory >& /*_rxFactory*/)
 {
-    return *new FmXFilterCell();
+    return FmXFilterCell();
 }
 
 DBG_NAME(FmXFilterCell);
diff --git a/svx/source/sdr/contact/viewcontactofunocontrol.cxx b/svx/source/sdr/contact/viewcontactofunocontrol.cxx
index 94cb790..32f9118 100644
--- a/svx/source/sdr/contact/viewcontactofunocontrol.cxx
+++ b/svx/source/sdr/contact/viewcontactofunocontrol.cxx
@@ -114,10 +114,10 @@
         bPrintOrPreview |= ( pPageViewContact != NULL ) && pPageViewContact->GetPageWindow().GetPageView().GetView().IsPrintPreview();
 
         if ( bPrintOrPreview )
-            return *new UnoControlPrintOrPreviewContact( *pPageViewContact, *this );
+            return UnoControlPrintOrPreviewContact( *pPageViewContact, *this );
 
         // all others are nowadays served by the same implementation
-        return *new ViewObjectContactOfUnoControl( _rObjectContact, *this );
+        return ViewObjectContactOfUnoControl( _rObjectContact, *this );
     }
 
     //--------------------------------------------------------------------
diff --git a/svx/source/tbxctrls/extrusioncontrols.cxx b/svx/source/tbxctrls/extrusioncontrols.cxx
index 6ce0665..b0b4722 100644
--- a/svx/source/tbxctrls/extrusioncontrols.cxx
+++ b/svx/source/tbxctrls/extrusioncontrols.cxx
@@ -283,7 +283,7 @@
     const Reference< XMultiServiceFactory >& rSMgr
 )   throw( RuntimeException )
 {
-    return *new ExtrusionDirectionControl( rSMgr );
+    return ExtrusionDirectionControl( rSMgr );
 }
 
 // --------------------------------------------------------------------
@@ -547,7 +547,7 @@
 
 Reference< XInterface > SAL_CALL SAL_CALL ExtrusionDepthController_createInstance( const Reference< XMultiServiceFactory >& rSMgr ) throw( RuntimeException )
 {
-    return *new ExtrusionDepthController( rSMgr );
+    return ExtrusionDepthController( rSMgr );
 }
 
 // --------------------------------------------------------------------
@@ -810,7 +810,7 @@
     const Reference< XMultiServiceFactory >& rSMgr
 )   throw( RuntimeException )
 {
-    return *new ExtrusionLightingControl( rSMgr );
+    return ExtrusionLightingControl( rSMgr );
 }
 
 // --------------------------------------------------------------------
@@ -955,7 +955,7 @@
     const Reference< XMultiServiceFactory >& rSMgr
 )   throw( RuntimeException )
 {
-    return *new ExtrusionSurfaceControl( rSMgr );
+    return ExtrusionSurfaceControl( rSMgr );
 }
 
 // --------------------------------------------------------------------
diff --git a/svx/source/tbxctrls/fontworkgallery.cxx b/svx/source/tbxctrls/fontworkgallery.cxx
index 075e3c8..f2ac095 100644
--- a/svx/source/tbxctrls/fontworkgallery.cxx
+++ b/svx/source/tbxctrls/fontworkgallery.cxx
@@ -453,7 +453,7 @@
 
 Reference< XInterface > SAL_CALL SAL_CALL FontWorkAlignmentControl_createInstance( const Reference< XMultiServiceFactory >& rSMgr ) throw( RuntimeException )
 {
-    return *new FontWorkAlignmentControl( rSMgr );
+    return FontWorkAlignmentControl( rSMgr );
 }
 
 // --------------------------------------------------------------------
@@ -689,7 +689,7 @@
 
 Reference< XInterface > SAL_CALL SAL_CALL FontWorkCharacterSpacingControl_createInstance( const Reference< XMultiServiceFactory >& rSMgr ) throw( RuntimeException )
 {
-    return *new FontWorkCharacterSpacingControl( rSMgr );
+    return FontWorkCharacterSpacingControl( rSMgr );
 }
 
 // --------------------------------------------------------------------
diff --git a/svx/source/tbxctrls/tbunocontroller.cxx b/svx/source/tbxctrls/tbunocontroller.cxx
index d2c7b9d..16c5635 100644
--- a/svx/source/tbxctrls/tbunocontroller.cxx
+++ b/svx/source/tbxctrls/tbunocontroller.cxx
@@ -413,7 +413,7 @@
 uno::Reference< uno::XInterface > SAL_CALL FontHeightToolBoxControl_createInstance(
     const uno::Reference< lang::XMultiServiceFactory >& rSMgr )
 {
-    return *new FontHeightToolBoxControl( rSMgr );
+    return FontHeightToolBoxControl( rSMgr );
 }
 
 }
diff --git a/svx/source/unodraw/unobtabl.cxx b/svx/source/unodraw/unobtabl.cxx
index 7051d1b..f412cab 100644
--- a/svx/source/unodraw/unobtabl.cxx
+++ b/svx/source/unodraw/unobtabl.cxx
@@ -107,7 +107,7 @@
  */
 uno::Reference< uno::XInterface > SAL_CALL SvxUnoBitmapTable_createInstance( SdrModel* pModel )
 {
-    return *new SvxUnoBitmapTable(pModel);
+    return SvxUnoBitmapTable(pModel);
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/svx/source/unodraw/unoctabl.cxx b/svx/source/unodraw/unoctabl.cxx
index 72bb162..0d81154 100644
--- a/svx/source/unodraw/unoctabl.cxx
+++ b/svx/source/unodraw/unoctabl.cxx
@@ -212,11 +212,11 @@
  */
 uno::Reference< uno::XInterface > SAL_CALL SvxUnoColorTable_createInstance(const uno::Reference< lang::XMultiServiceFactory > & ) throw(uno::Exception)
 {
-    return *new SvxUnoColorTable();
+    return SvxUnoColorTable();
 }
 uno::Reference< uno::XInterface > SAL_CALL create_EnhancedCustomShapeEngine( const uno::Reference< lang::XMultiServiceFactory >& rxFact ) throw(uno::Exception)
 {
-    return *new EnhancedCustomShapeEngine( rxFact );
+    return EnhancedCustomShapeEngine( rxFact );
 }
 
 //
diff --git a/svx/source/unodraw/unodtabl.cxx b/svx/source/unodraw/unodtabl.cxx
index 1db5395..a0d8ca3 100644
--- a/svx/source/unodraw/unodtabl.cxx
+++ b/svx/source/unodraw/unodtabl.cxx
@@ -91,7 +91,7 @@
  */
 uno::Reference< uno::XInterface > SAL_CALL SvxUnoDashTable_createInstance( SdrModel* pModel )
 {
-    return *new SvxUnoDashTable(pModel);
+    return SvxUnoDashTable(pModel);
 }
 
 
diff --git a/svx/source/unodraw/unogtabl.cxx b/svx/source/unodraw/unogtabl.cxx
index deb2e1d..bbe46fa 100644
--- a/svx/source/unodraw/unogtabl.cxx
+++ b/svx/source/unodraw/unogtabl.cxx
@@ -90,7 +90,7 @@
  */
 uno::Reference< uno::XInterface > SAL_CALL SvxUnoGradientTable_createInstance( SdrModel* pModel )
 {
-    return *new SvxUnoGradientTable(pModel);
+    return SvxUnoGradientTable(pModel);
 }
 
 
diff --git a/svx/source/unodraw/unohtabl.cxx b/svx/source/unodraw/unohtabl.cxx
index 8f3f677..11f3090 100644
--- a/svx/source/unodraw/unohtabl.cxx
+++ b/svx/source/unodraw/unohtabl.cxx
@@ -88,7 +88,7 @@
  */
 uno::Reference< uno::XInterface > SAL_CALL SvxUnoHatchTable_createInstance( SdrModel* pModel )
 {
-    return *new SvxUnoHatchTable(pModel);
+    return SvxUnoHatchTable(pModel);
 }
 
 
diff --git a/svx/source/unodraw/unomtabl.cxx b/svx/source/unodraw/unomtabl.cxx
index ce70254..1983750 100644
--- a/svx/source/unodraw/unomtabl.cxx
+++ b/svx/source/unodraw/unomtabl.cxx
@@ -455,7 +455,7 @@
  */
 uno::Reference< uno::XInterface > SAL_CALL SvxUnoMarkerTable_createInstance( SdrModel* pModel )
 {
-    return *new SvxUnoMarkerTable(pModel);
+    return SvxUnoMarkerTable(pModel);
 }
 
 
diff --git a/svx/source/unodraw/unottabl.cxx b/svx/source/unodraw/unottabl.cxx
index d76c637..b4be04b 100644
--- a/svx/source/unodraw/unottabl.cxx
+++ b/svx/source/unodraw/unottabl.cxx
@@ -90,7 +90,7 @@
  */
 uno::Reference< uno::XInterface > SAL_CALL SvxUnoTransGradientTable_createInstance( SdrModel* pModel )
 {
-    return *new SvxUnoTransGradientTable(pModel);
+    return SvxUnoTransGradientTable(pModel);
 }
 
 

-- 
To view, visit https://gerrit.libreoffice.org/1569
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifbade82383694ebac0e82920014cddfff5b12fcb
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Joren De Cuyper <joren.libreoffice at telenet.be>



More information about the LibreOffice mailing list