diff options
Diffstat (limited to 'svl/source')
| -rw-r--r-- | svl/source/notify/broadcast.cxx | 10 | ||||
| -rw-r--r-- | svl/source/notify/listener.cxx | 10 |
2 files changed, 15 insertions, 5 deletions
diff --git a/svl/source/notify/broadcast.cxx b/svl/source/notify/broadcast.cxx index 237c323e5d0b..62a52dff18c5 100644 --- a/svl/source/notify/broadcast.cxx +++ b/svl/source/notify/broadcast.cxx @@ -60,11 +60,11 @@ void SvtBroadcaster::Remove( SvtListener* p ) } Normalize(); - std::pair<ListenersType::iterator,ListenersType::iterator> r = - std::equal_range(maListeners.begin(), maListeners.end(), p); - if (r.first != r.second) - maListeners.erase(r.first, r.second); + auto it = std::lower_bound(maListeners.begin(), maListeners.end(), p); + if (it != maListeners.end() && *it == p) + maListeners.erase(it); + if (maListeners.empty()) ListenersGone(); } @@ -116,7 +116,7 @@ SvtBroadcaster::~SvtBroadcaster() ++dest; if (dest == maDestructedListeners.end() || *dest != *it) - (*it)->EndListening(*this); + (*it)->BroadcasterDying(*this); } } diff --git a/svl/source/notify/listener.cxx b/svl/source/notify/listener.cxx index 89306a534a0d..506647784451 100644 --- a/svl/source/notify/listener.cxx +++ b/svl/source/notify/listener.cxx @@ -19,6 +19,7 @@ #include <svl/listener.hxx> #include <svl/broadcast.hxx> +#include <cassert> SvtListener::QueryBase::QueryBase( sal_uInt16 nId ) : mnId(nId) {} SvtListener::QueryBase::~QueryBase() {} @@ -65,6 +66,15 @@ bool SvtListener::EndListening( SvtBroadcaster& rBroadcaster ) return true; } +// called from the SvtBroadcaster destructor, used to avoid calling +// back into the broadcaster again +void SvtListener::BroadcasterDying( SvtBroadcaster& rBroadcaster ) +{ + BroadcastersType::iterator it = maBroadcasters.find(&rBroadcaster); + if (it != maBroadcasters.end()) + maBroadcasters.erase(it); +} + void SvtListener::EndListeningAll() { BroadcastersType::iterator it = maBroadcasters.begin(); |