From 306efefe22e02248eff14f8be2cef68d75d26e55 Mon Sep 17 00:00:00 2001 From: Jan Holesovsky Date: Thu, 16 Jan 2014 16:26:55 +0100 Subject: Minimize the constructor functions to a bare minimum. Most of the constructors are supposed to be only a call of new TheInstance(arguments) or an equivalent; so let's just change the constructor caller accordingly, to accept unacquired new instance. If there are exceptions that need to do more heavy lifting, they do not have to use the constructor feature, or there can be a wrapper for the real implementation, doing the additional work in their (C++) constructor. Change-Id: I035c378778aeda60d15af4e56ca3761c586d5ded --- sfx2/source/appl/appbas.cxx | 10 ++++------ sfx2/source/appl/appbaslib.cxx | 10 +++------- sfx2/source/appl/appdispatchprovider.cxx | 4 +--- sfx2/source/appl/macroloader.cxx | 4 +--- sfx2/source/appl/shutdownicon.cxx | 4 +--- sfx2/source/appl/xpackcreator.cxx | 4 +--- sfx2/source/dialog/backingcomp.cxx | 4 +--- sfx2/source/doc/SfxDocumentMetaData.cxx | 8 ++------ sfx2/source/doc/doctemplates.cxx | 4 +--- sfx2/source/doc/iframe.cxx | 4 +--- sfx2/source/doc/ownsubfilterservice.cxx | 4 +--- sfx2/source/doc/plugin.cxx | 4 +--- sfx2/source/inc/appbaslib.hxx | 2 +- sfx2/source/view/frmload.cxx | 4 +--- 14 files changed, 20 insertions(+), 50 deletions(-) (limited to 'sfx2/source') diff --git a/sfx2/source/appl/appbas.cxx b/sfx2/source/appl/appbas.cxx index 5010806635f6..3251a9271cf8 100644 --- a/sfx2/source/appl/appbas.cxx +++ b/sfx2/source/appl/appbas.cxx @@ -127,11 +127,10 @@ BasicManager* SfxApplication::GetBasicManager() //-------------------------------------------------------------------- -Reference< XLibraryContainer > SfxApplication::GetDialogContainer() +XLibraryContainer * SfxApplication::GetDialogContainer() { #ifdef DISABLE_SCRIPTING - Reference< XLibraryContainer > dummy; - return dummy; + return NULL; #else if ( !pAppData_Impl->pBasicManager->isValid() ) GetBasicManager(); @@ -141,11 +140,10 @@ Reference< XLibraryContainer > SfxApplication::GetDialogContainer() //-------------------------------------------------------------------- -Reference< XLibraryContainer > SfxApplication::GetBasicContainer() +XLibraryContainer * SfxApplication::GetBasicContainer() { #ifdef DISABLE_SCRIPTING - Reference< XLibraryContainer > dummy; - return dummy; + return NULL; #else if ( !pAppData_Impl->pBasicManager->isValid() ) GetBasicManager(); diff --git a/sfx2/source/appl/appbaslib.cxx b/sfx2/source/appl/appbaslib.cxx index 4b96cc2e6c0e..df0c237c906d 100644 --- a/sfx2/source/appl/appbaslib.cxx +++ b/sfx2/source/appl/appbaslib.cxx @@ -121,7 +121,7 @@ void SfxBasicManagerHolder::storeLibrariesToStorage( const Reference< XStorage > #endif } -Reference< XLibraryContainer > SfxBasicManagerHolder::getLibraryContainer( ContainerType _eType ) +XLibraryContainer * SfxBasicManagerHolder::getLibraryContainer( ContainerType _eType ) { OSL_PRECOND( isValid(), "SfxBasicManagerHolder::getLibraryContainer: not initialized!" ); @@ -161,9 +161,7 @@ com_sun_star_comp_sfx2_ApplicationDialogLibraryContainer_get_implementation( css::uno::Sequence const &) { SFX_APP()->GetBasicManager(); - Reference< XInterface > xRet( SFX_APP()->GetDialogContainer(), UNO_QUERY ); - xRet->acquire(); - return xRet.get(); + return SFX_APP()->GetDialogContainer(); } //============================================================================ @@ -175,9 +173,7 @@ com_sun_star_comp_sfx2_ApplicationScriptLibraryContainer_get_implementation( css::uno::Sequence const &) { SFX_APP()->GetBasicManager(); - Reference< XInterface > xRet( SFX_APP()->GetBasicContainer(), UNO_QUERY ); - xRet->acquire(); - return xRet.get(); + return SFX_APP()->GetBasicContainer(); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/appl/appdispatchprovider.cxx b/sfx2/source/appl/appdispatchprovider.cxx index 7774582e5d92..c2b88af54dd4 100644 --- a/sfx2/source/appl/appdispatchprovider.cxx +++ b/sfx2/source/appl/appdispatchprovider.cxx @@ -255,9 +255,7 @@ com_sun_star_comp_sfx2_AppDispatchProvider_get_implementation( css::uno::XComponentContext *, css::uno::Sequence const &arguments) { - rtl::Reference x(new SfxAppDispatchProvider(arguments)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new SfxAppDispatchProvider(arguments)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/appl/macroloader.cxx b/sfx2/source/appl/macroloader.cxx index 8d59b8478d90..ff5886148a27 100644 --- a/sfx2/source/appl/macroloader.cxx +++ b/sfx2/source/appl/macroloader.cxx @@ -341,9 +341,7 @@ com_sun_star_comp_sfx2_SfxMacroLoader_get_implementation( css::uno::XComponentContext *, css::uno::Sequence const &arguments) { - rtl::Reference x(new SfxMacroLoader(arguments)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new SfxMacroLoader(arguments)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/appl/shutdownicon.cxx b/sfx2/source/appl/shutdownicon.cxx index bef401945ac3..a7e2e1b67cd8 100644 --- a/sfx2/source/appl/shutdownicon.cxx +++ b/sfx2/source/appl/shutdownicon.cxx @@ -976,9 +976,7 @@ com_sun_star_comp_desktop_QuickstartWrapper_get_implementation( css::uno::XComponentContext *context, css::uno::Sequence const &) { - rtl::Reference x(new ShutdownIcon(context)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new ShutdownIcon(context)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/appl/xpackcreator.cxx b/sfx2/source/appl/xpackcreator.cxx index 5fcc44550d0f..6c654e77a7fe 100644 --- a/sfx2/source/appl/xpackcreator.cxx +++ b/sfx2/source/appl/xpackcreator.cxx @@ -178,9 +178,7 @@ com_sun_star_comp_embed_PackageStructureCreator_get_implementation( css::uno::XComponentContext *, css::uno::Sequence const &) { - rtl::Reference x(new OPackageStructureCreator()); - x->acquire(); - return static_cast(x.get()); + return static_cast(new OPackageStructureCreator()); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/dialog/backingcomp.cxx b/sfx2/source/dialog/backingcomp.cxx index 6052c96a1ba0..fa5cfa81a94e 100644 --- a/sfx2/source/dialog/backingcomp.cxx +++ b/sfx2/source/dialog/backingcomp.cxx @@ -813,9 +813,7 @@ com_sun_star_comp_sfx2_BackingComp_get_implementation( css::uno::XComponentContext *context, css::uno::Sequence const &) { - rtl::Reference x(new BackingComp(context)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new BackingComp(context)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/doc/SfxDocumentMetaData.cxx b/sfx2/source/doc/SfxDocumentMetaData.cxx index 9e1d2f12ede5..90bb305a80d3 100644 --- a/sfx2/source/doc/SfxDocumentMetaData.cxx +++ b/sfx2/source/doc/SfxDocumentMetaData.cxx @@ -2319,9 +2319,7 @@ CompatWriterDocPropsImpl_get_implementation( css::uno::XComponentContext *context, css::uno::Sequence const &) { - rtl::Reference x(new CompatWriterDocPropsImpl(context)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new CompatWriterDocPropsImpl(context)); } extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface * SAL_CALL @@ -2329,9 +2327,7 @@ SfxDocumentMetaData_get_implementation( css::uno::XComponentContext *context, css::uno::Sequence const &) { - rtl::Reference x(new SfxDocumentMetaData(context)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new SfxDocumentMetaData(context)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/doc/doctemplates.cxx b/sfx2/source/doc/doctemplates.cxx index 11461cdeb643..54d2bf95409f 100644 --- a/sfx2/source/doc/doctemplates.cxx +++ b/sfx2/source/doc/doctemplates.cxx @@ -2900,9 +2900,7 @@ com_sun_star_comp_sfx2_DocumentTemplates_get_implementation( css::uno::XComponentContext *context, css::uno::Sequence const &) { - rtl::Reference x(new SfxDocTplService(context)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new SfxDocTplService(context)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/doc/iframe.cxx b/sfx2/source/doc/iframe.cxx index d4d5d5cdcf82..504b2891c795 100644 --- a/sfx2/source/doc/iframe.cxx +++ b/sfx2/source/doc/iframe.cxx @@ -432,9 +432,7 @@ com_sun_star_comp_sfx2_IFrameObject_get_implementation( css::uno::XComponentContext *context, css::uno::Sequence const &arguments) { - rtl::Reference x(new IFrameObject(context, arguments)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new IFrameObject(context, arguments)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/doc/ownsubfilterservice.cxx b/sfx2/source/doc/ownsubfilterservice.cxx index bb817db1e67a..139f7dac587d 100644 --- a/sfx2/source/doc/ownsubfilterservice.cxx +++ b/sfx2/source/doc/ownsubfilterservice.cxx @@ -126,9 +126,7 @@ com_sun_star_comp_document_OwnSubFilter_get_implementation( css::uno::XComponentContext *, css::uno::Sequence const &arguments) { - rtl::Reference x(new OwnSubFilterService(arguments)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new OwnSubFilterService(arguments)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/doc/plugin.cxx b/sfx2/source/doc/plugin.cxx index 86d4945251ff..b79219624646 100644 --- a/sfx2/source/doc/plugin.cxx +++ b/sfx2/source/doc/plugin.cxx @@ -311,9 +311,7 @@ com_sun_star_comp_sfx2_PluginObject_get_implementation( css::uno::XComponentContext *, css::uno::Sequence const &) { - rtl::Reference x(new PluginObject()); - x->acquire(); - return static_cast(x.get()); + return static_cast(new PluginObject()); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/sfx2/source/inc/appbaslib.hxx b/sfx2/source/inc/appbaslib.hxx index 8f804200d20d..2763b75c5a1e 100644 --- a/sfx2/source/inc/appbaslib.hxx +++ b/sfx2/source/inc/appbaslib.hxx @@ -59,7 +59,7 @@ public: */ void reset( BasicManager* _pBasicManager ); - ::com::sun::star::uno::Reference< ::com::sun::star::script::XLibraryContainer > + ::com::sun::star::script::XLibraryContainer * getLibraryContainer( ContainerType _eType ); /** calls the storeLibraries at both our script and basic library container diff --git a/sfx2/source/view/frmload.cxx b/sfx2/source/view/frmload.cxx index c565c3d6c9a8..daab66b976e2 100644 --- a/sfx2/source/view/frmload.cxx +++ b/sfx2/source/view/frmload.cxx @@ -778,9 +778,7 @@ com_sun_star_comp_office_FrameLoader_get_implementation( css::uno::XComponentContext *context, css::uno::Sequence const &) { - rtl::Reference x(new SfxFrameLoader_Impl(context)); - x->acquire(); - return static_cast(x.get()); + return static_cast(new SfxFrameLoader_Impl(context)); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ -- cgit