editeng/source/editeng/impedit2.cxx | 1 -
1 file changed, 1 deletion(-)
New commits:
commit 7a5727ba35ca2cff191208625448994202a94df6
Author: Caolán McNamara <caol...@redhat.com>
Date: Wed Jun 18 13:27:45 2014 +0100
Related: fdo#80140 a11y crash on changing text via spellchecker in impress
i.e. with French dictionary available, open test case of fdo#80140, open
spell checking dialog and choose "correct" on the first example.
Every version of LibreOffice crashes on this, so its an old bug.
As can be seen from the following bt, if a11y is enabled then on calling
ImpEditEngine::ImpInsertText from ImpEditEngine::ImpDeleteSelection then
FormatDoc may be called and throw away all the old portions so the following
ParaPortion* pPortion = FindParaPortion(aStartPaM.GetNode()) will get the
new
ParaPortion, not the old one, and so the MarkInvalid that uses the old
ParaPortion invalid range is not correct anymore.
Seeing as the only caller of ImpRemoveChars is ImpDeleteSelection and
ImpDeleteSelection calls TextModified after ImpRemoveChars and after the
use of
MarkInvalid, it seems safe and logical to remove the TextModified from
ImpRemoveChars and rely on the ImpDeleteSelection TextModified.
#0 TextPortion::TextPortion (this=0x30bd020, nL=5) at
/home/caolan/LibreOffice/core/editeng/source/editeng/editdoc.hxx:401
#1 0x00007fffd93f4d5a in ImpEditEngine::CreateTextPortions
(this=0x3f42700, pParaPortion=0x3584930, rStart=@0x7fffffffa234: 0)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit3.cxx:2372
#2 0x00007fffd93ed087 in ImpEditEngine::CreateLines (this=0x3f42700,
nPara=0, nStartPosY=0)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit3.cxx:687
#3 0x00007fffd93ebb3b in ImpEditEngine::FormatDoc (this=0x3f42700)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit3.cxx:394
#4 0x00007fffd93fd239 in ImpEditEngine::FormatAndUpdate (this=0x3f42700,
pCurView=0x0)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit3.cxx:4156
#5 0x00007fffd937c3f4 in EditEngine::GetDocPosTopLeft (this=0x3f42690,
nParagraph=0)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/editeng.cxx:1920
#6 0x00007fffd94d298a in Outliner::GetDocPosTopLeft (this=0x3f42390,
nParagraph=0)
at
/home/caolan/LibreOffice/core/editeng/source/outliner/outlin2.cxx:429
#7 0x00007fffd951c2d3 in SvxOutlinerForwarder::GetParaBounds
(this=0x36015a0, nPara=0)
at /home/caolan/LibreOffice/core/editeng/source/uno/unoforou.cxx:344
#8 0x00007fffd950efe5 in SvxAccessibleTextAdapter::GetParaBounds
(this=0x3394e88, nPara=0)
at /home/caolan/LibreOffice/core/editeng/source/uno/unoedprx.cxx:798
#9 0x00007fffd9aa25b5 in
accessibility::AccessibleTextHelper_Impl::UpdateVisibleChildren
(this=0x3394e00, bBroadcastEvents=true)
at
/home/caolan/LibreOffice/core/svx/source/accessibility/AccessibleTextHelper.cxx:851
#10 0x00007fffd9aa38b3 in
accessibility::AccessibleTextHelper_Impl::ProcessQueue (this=0x3394e00)
at
/home/caolan/LibreOffice/core/svx/source/accessibility/AccessibleTextHelper.cxx:1332
#11 0x00007fffd9aa400c in accessibility::AccessibleTextHelper_Impl::Notify
(this=0x3394e00, rHint=...)
at
/home/caolan/LibreOffice/core/svx/source/accessibility/AccessibleTextHelper.cxx:1482
#12 0x00007ffff548c6ba in SfxBroadcaster::Broadcast (this=0x3394ac0,
rHint=...)
at /home/caolan/LibreOffice/core/svl/source/notify/brdcst.cxx:43
#13 0x00007fffd895ce27 in SvxTextEditSourceImpl::NotifyHdl
(this=0x3394a50, aNotify=0x7fffffffb4c0)
at /home/caolan/LibreOffice/core/svx/source/unodraw/unoshtxt.cxx:1030
#14 0x00007fffd895cd61 in SvxTextEditSourceImpl::LinkStubNotifyHdl
(pThis=0x3394a50, pCaller=0x7fffffffb4c0)
at /home/caolan/LibreOffice/core/svx/source/unodraw/unoshtxt.cxx:1023
#15 0x00007fffd936252a in Link::Call (this=0x3f426a8,
pCaller=0x7fffffffb4c0)
at /home/caolan/LibreOffice/core/include/tools/link.hxx:123
#16 0x00007fffd94db186 in Outliner::EditEngineNotifyHdl (this=0x3f42390,
pNotify=0x7fffffffb4c0)
at
/home/caolan/LibreOffice/core/editeng/source/outliner/outliner.cxx:2041
#17 0x00007fffd94db149 in Outliner::LinkStubEditEngineNotifyHdl
(pThis=0x3f42390, pCaller=0x7fffffffb4c0)
at
/home/caolan/LibreOffice/core/editeng/source/outliner/outliner.cxx:2038
#18 0x00007fffd936252a in Link::Call (this=0x7fffffffb490,
pCaller=0x7fffffffb4c0)
at /home/caolan/LibreOffice/core/include/tools/link.hxx:123
#19 0x00007fffd93e0155 in ImpEditEngine::CallNotify (this=0x3f42700,
rNotify=...)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit2.cxx:4339
#20 0x00007fffd93cf462 in ImpEditEngine::TextModified (this=0x3f42700)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit2.cxx:707
#21 0x00007fffd93d5a73 in ImpEditEngine::ImpRemoveChars (this=0x3f42700,
rPaM=..., nChars=8, pCurUndo=0x0)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit2.cxx:2064
#22 0x00007fffd93d77bd in ImpEditEngine::ImpDeleteSelection
(this=0x3f42700, rCurSel=...)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit2.cxx:2424
#23 0x00007fffd93d89f6 in ImpEditEngine::ImpInsertText (this=0x3f42700,
aCurSel=..., rStr="Smoothie")
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit2.cxx:2645
#24 0x00007fffd94139f7 in ImpEditEngine::ApplyChangedSentence
(this=0x3f42700, rEditView=...,
rNewPortions=std::__debug::vector of length 2, capacity 2 = {...},
bRecheck=false)
at
/home/caolan/LibreOffice/core/editeng/source/editeng/impedit4.cxx:2172
#25 0x00007fffd937d0f1 in EditEngine::ApplyChangedSentence
(this=0x3f42690, rEditView=...,
rNewPortions=std::__debug::vector of length 2, capacity 2 = {...},
bRecheck=false)
(cherry picked from commit ce61e7f9debfc7572ce4bb30d13e13aca79f010a)
(cherry picked from commit d5798014060b913c57aa7c602d01b0c1b8225029)
Conflicts:
editeng/source/editeng/impedit2.cxx
Change-Id: I6d0a96c8875858d8a25455f0dd1408f07e1f6e14
Reviewed-on: https://gerrit.libreoffice.org/9820
Tested-by: Markus Mohrhard <markus.mohrh...@googlemail.com>
Reviewed-by: Markus Mohrhard <markus.mohrh...@googlemail.com>
diff --git a/editeng/source/editeng/impedit2.cxx
b/editeng/source/editeng/impedit2.cxx
index 1c094c3..e5d91f6 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -2080,7 +2080,6 @@ void ImpEditEngine::ImpRemoveChars( const EditPaM& rPaM,
sal_uInt16 nChars, Edit
}
aEditDoc.RemoveChars( rPaM, nChars );
- TextModified();
}
EditSelection ImpEditEngine::ImpMoveParagraphs( Range aOldPositions, sal_Int32
nNewPos )
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
