Kaydet (Commit) aed4c424 authored tarafından Noel Grandin's avatar Noel Grandin Kaydeden (comit) Noel Grandin

remove some manual ref-counting in framework

Change-Id: I469439abfa4e9dcd29f2d1693b03d37b3d61c81e
Reviewed-on: https://gerrit.libreoffice.org/26047Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarNoel Grandin <noelgrandin@gmail.com>
üst c80d34ad
...@@ -25,6 +25,7 @@ ...@@ -25,6 +25,7 @@
#include <com/sun/star/frame/XUIControllerFactory.hpp> #include <com/sun/star/frame/XUIControllerFactory.hpp>
#include <rtl/ustrbuf.hxx> #include <rtl/ustrbuf.hxx>
#include <rtl/ref.hxx>
#include <cppuhelper/basemutex.hxx> #include <cppuhelper/basemutex.hxx>
#include <cppuhelper/compbase.hxx> #include <cppuhelper/compbase.hxx>
#include <cppuhelper/supportsservice.hxx> #include <cppuhelper/supportsservice.hxx>
...@@ -60,9 +61,9 @@ public: ...@@ -60,9 +61,9 @@ public:
protected: protected:
UIControllerFactory( const css::uno::Reference< css::uno::XComponentContext >& xContext, const rtl::OUString &rUINode ); UIControllerFactory( const css::uno::Reference< css::uno::XComponentContext >& xContext, const rtl::OUString &rUINode );
bool m_bConfigRead; bool m_bConfigRead;
css::uno::Reference< css::uno::XComponentContext > m_xContext; css::uno::Reference< css::uno::XComponentContext > m_xContext;
ConfigurationAccess_ControllerFactory* m_pConfigAccess; rtl::Reference<ConfigurationAccess_ControllerFactory> m_pConfigAccess;
private: private:
virtual void SAL_CALL disposing() override; virtual void SAL_CALL disposing() override;
...@@ -78,7 +79,6 @@ UIControllerFactory::UIControllerFactory( ...@@ -78,7 +79,6 @@ UIControllerFactory::UIControllerFactory(
{ {
m_pConfigAccess = new ConfigurationAccess_ControllerFactory(m_xContext, m_pConfigAccess = new ConfigurationAccess_ControllerFactory(m_xContext,
"/org.openoffice.Office.UI.Controller/Registered/" + rConfigurationNode); "/org.openoffice.Office.UI.Controller/Registered/" + rConfigurationNode);
m_pConfigAccess->acquire();
} }
UIControllerFactory::~UIControllerFactory() UIControllerFactory::~UIControllerFactory()
...@@ -89,12 +89,7 @@ UIControllerFactory::~UIControllerFactory() ...@@ -89,12 +89,7 @@ UIControllerFactory::~UIControllerFactory()
void SAL_CALL UIControllerFactory::disposing() void SAL_CALL UIControllerFactory::disposing()
{ {
osl::MutexGuard g(rBHelper.rMutex); osl::MutexGuard g(rBHelper.rMutex);
if (m_pConfigAccess) m_pConfigAccess.clear();
{
// reduce reference count
m_pConfigAccess->release();
m_pConfigAccess = nullptr;
}
} }
// XMultiComponentFactory // XMultiComponentFactory
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment