diff options
author | Michael Stahl <mstahl@redhat.com> | 2016-06-20 23:38:11 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2016-06-24 12:05:09 +0000 |
commit | 136a2fd6c08193793d546e69108765316c96668b (patch) | |
tree | 1e78b7cbb60c0726d4c13ac171b973eff6978466 /scripting/source/provider | |
parent | 7d76bb251e0c88ff17282a33b801a5d17a434af5 (diff) |
compilerplugins: add OWeakObject::release() override check
Change-Id: I767857545d7c91615cf162790c04f0016de9fdf6
Reviewed-on: https://gerrit.libreoffice.org/26555
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'scripting/source/provider')
-rw-r--r-- | scripting/source/provider/BrowseNodeFactoryImpl.cxx | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/scripting/source/provider/BrowseNodeFactoryImpl.cxx b/scripting/source/provider/BrowseNodeFactoryImpl.cxx index 863860e2911e..a312ed4aa35e 100644 --- a/scripting/source/provider/BrowseNodeFactoryImpl.cxx +++ b/scripting/source/provider/BrowseNodeFactoryImpl.cxx @@ -500,15 +500,12 @@ public: throw () override { - osl_atomic_increment( &m_refCount ); + t_BrowseNodeBase::acquire(); } virtual void SAL_CALL release() throw () override { - if ( osl_atomic_decrement( &m_refCount ) == 0 ) - { - delete this; - } + t_BrowseNodeBase::release(); } // XTypeProvider (implemnented by base, but needs to be overridden for // delegating to aggregate) |