diff options
author | Michael Stahl <mstahl@redhat.com> | 2017-09-15 13:51:04 +0200 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2017-09-27 11:54:45 +0200 |
commit | 34905a6e9e0c262b61e5969274b661692af44a46 (patch) | |
tree | 02d850c2d7a22b76b25c475f99c5e08c1ac2b566 /cppuhelper | |
parent | 31baba076f6058d675588b2715a3322455c87500 (diff) |
ofz#3301 sw: DeleteAndJoin found yet another way to delete new redline
Not only can that happen in CompressRedlines(), it can also happen
in the SwComparePosition::Outside case when the DeleteRedline()
decides in particular circumstances to split up the inserted
new redline.
Arguably it's wrong to split up the new redline in this case;
not sure if that ever happens in a legitimate use case though.
Avoid this by removing the hack to temporarily insert the new redline
and instead create a temporary SwUnoCursor that will be corrected
on behalf of the new redline, while the new redline is parked on a
safe node.
This not only avoids the crash on this file but also makes the
"corrupted redline table" assertions go away.
Change-Id: I478f4cfc53a19d2cf2f0937f631962f80b1815ff
Reviewed-on: https://gerrit.libreoffice.org/42322
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'cppuhelper')
0 files changed, 0 insertions, 0 deletions