diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-13 15:53:27 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-05-13 18:58:38 +0200 |
commit | 59a5fdd32727447239ee91cea00f943609540507 (patch) | |
tree | 5cd58cab561640695b85c74c015224ffaf5d4d6d /toolkit/source | |
parent | d0780b21cfe235c4446adf649eb690f9c1771dd5 (diff) |
fix ScriptEventContainer::isEmpty
which was relying on the nElementCount field, which was not being
updated. Just delete it and check the map.
Change-Id: If543abcc7c1778a27675a1df69a54325024234b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94133
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'toolkit/source')
-rw-r--r-- | toolkit/source/controls/eventcontainer.cxx | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/toolkit/source/controls/eventcontainer.cxx b/toolkit/source/controls/eventcontainer.cxx index 165c91296cb9..04aeaf2bfc42 100644 --- a/toolkit/source/controls/eventcontainer.cxx +++ b/toolkit/source/controls/eventcontainer.cxx @@ -43,8 +43,7 @@ Type ScriptEventContainer::getElementType() sal_Bool ScriptEventContainer::hasElements() { - bool bRet = (mnElementCount > 0); - return bRet; + return !mHashMap.empty(); } // Methods XNameAccess @@ -171,8 +170,7 @@ void ScriptEventContainer::removeContainerListener( const css::uno::Reference< c ScriptEventContainer::ScriptEventContainer() - : mnElementCount( 0 ), - mType( cppu::UnoType<ScriptEventDescriptor>::get() ), + : mType( cppu::UnoType<ScriptEventDescriptor>::get() ), maContainerListeners( *this ) { } |