diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-02 12:25:42 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-02 15:09:34 +0200 |
| commit | d02d57d5c2bbe38722f848460a02511749e50957 (patch) | |
| tree | 3f1183718beff273698f34b1e6d100adae96a4ae /svl | |
| parent | 754f3614837ecd7348b241c9000d5f1d15d16947 (diff) | |
fix bug in SvtListener copy constructor
it was copying maBroadcasters, without actually starting to listen to
those broadcasters, which means if we __did__ call StartListening, that
method would have just returned early.
Since nothing seems to need this functionality, and it's not obvious
what the correct behaviour would be, just don't do the copy.
Change-Id: If5865fa3449839758a43ac537475e0a474f39078
Reviewed-on: https://gerrit.libreoffice.org/61239
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svl')
| -rw-r--r-- | svl/source/notify/listener.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/svl/source/notify/listener.cxx b/svl/source/notify/listener.cxx index c668397baa47..6a54f6465c75 100644 --- a/svl/source/notify/listener.cxx +++ b/svl/source/notify/listener.cxx @@ -31,8 +31,7 @@ sal_uInt16 SvtListener::QueryBase::getId() const SvtListener::SvtListener() {} -SvtListener::SvtListener( const SvtListener &r ) : - maBroadcasters(r.maBroadcasters) {} +SvtListener::SvtListener( const SvtListener & ) {} SvtListener::~SvtListener() COVERITY_NOEXCEPT_FALSE { |