[Libreoffice-commits] core.git: sw/source

[Miklos Vajna]

 sw/source/core/doc/DocumentRedlineManager.cxx |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 3a897f7cbf44f44f2baa750f85d9aecbbbd2b6f9
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Thu Sep 1 09:36:02 2016 +0200

    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 at collabora.co.uk>
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx b/sw/source/core/doc/DocumentRedlineManager.cxx
index 2672437..86c1e82 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: