diff options
| author | Miklos Vajna <vmiklos@collabora.co.uk> | 2016-09-01 09:36:02 +0200 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2016-09-01 12:04:00 +0200 |
| commit | f502cdb7000f8e61e3530370b28f160e6ef499c6 (patch) | |
| tree | 8b7af0d23b7a631f0e8d6a0ddd0f197e6ec984c3 | |
| parent | 2a38d1b186428f0b23722f304addfae43e00677e (diff) | |
CppunitTest_sw_filters_test: fix ASan build
MaybeNotifyModification() should be called only in the POS_INSIDE case,
not when POS_EQUAL gets there via the fallthrough.
Change-Id: I8a05ee508a14f62b12e93799b2e98a33041d6f33
Reviewed-on: https://gerrit.libreoffice.org/28582
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
(cherry picked from commit 3a897f7cbf44f44f2baa750f85d9aecbbbd2b6f9)
| -rw-r--r-- | sw/source/core/doc/DocumentRedlineManager.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx b/sw/source/core/doc/DocumentRedlineManager.cxx index ed98e0c1f799..34a281f0047b 100644 --- a/sw/source/core/doc/DocumentRedlineManager.cxx +++ b/sw/source/core/doc/DocumentRedlineManager.cxx @@ -1215,7 +1215,8 @@ bool DocumentRedlineManager::AppendRedline( SwRangeRedline* pNewRedl, bool bCall } delete pNewRedl; pNewRedl = nullptr; - pRedl->MaybeNotifyModification(); + if (eCmpPos == POS_INSIDE) + pRedl->MaybeNotifyModification(); break; case POS_OUTSIDE: |