diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2021-01-05 10:49:45 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2021-01-05 12:23:16 +0100 |
commit | d988b0cdeb54d84a0347a036a277c21c12544ab9 (patch) | |
tree | 7c24b2034fb4169cd0e924dff91a4c6e3032c0ba /sw/source/core/txtnode/txtedt.cxx | |
parent | 2407f8aca45a3fde5a0c6808d62a0435ff1de565 (diff) |
Revert "NotifyClients no more"
This reverts commit 96bafa464ebdbce3ef04bec9beae5e745bb37794. It started to
cause an ASan heap-use-after-free during CppunitTest_sw_uiwriter, see the
comments starting at <https://gerrit.libreoffice.org/c/core/+/108604/
5#message-681a6100cff11cb51d20e5a2b2fb38da1f92ab32> "NotifyClients no more".
Change-Id: Id0eef2e75ea047d4b337c51df929eea81fb69f5a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108717
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'sw/source/core/txtnode/txtedt.cxx')
-rw-r--r-- | sw/source/core/txtnode/txtedt.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sw/source/core/txtnode/txtedt.cxx b/sw/source/core/txtnode/txtedt.cxx index ba25c5fccba1..edc0ac296d00 100644 --- a/sw/source/core/txtnode/txtedt.cxx +++ b/sw/source/core/txtnode/txtedt.cxx @@ -634,9 +634,9 @@ void SwTextNode::RstTextAttr( nMax, 0); - CallSwClientNotify(sw::LegacyModifyHint(nullptr, &aHint)); + NotifyClients( nullptr, &aHint ); SwFormatChg aNew( GetFormatColl() ); - CallSwClientNotify(sw::LegacyModifyHint(nullptr, &aNew)); + NotifyClients( nullptr, &aNew ); } static sal_Int32 clipIndexBounds(const OUString &rStr, sal_Int32 nPos) @@ -1983,10 +1983,10 @@ void SwTextNode::ReplaceTextOnly( sal_Int32 nPos, sal_Int32 nLen, // notify the layout! SwDelText aDelHint( nPos, nTLen ); - CallSwClientNotify(sw::LegacyModifyHint(nullptr, &aDelHint)); + NotifyClients( nullptr, &aDelHint ); SwInsText aHint( nPos, nTLen ); - CallSwClientNotify(sw::LegacyModifyHint(nullptr, &aHint)); + NotifyClients( nullptr, &aHint ); } // the return values allows us to see if we did the heavy- |