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

Michael Stahl (via logerrit) logerrit at kemper.freedesktop.org
Tue Apr 2 12:10:04 UTC 2019


 sw/source/core/inc/wrong.hxx   |    3 ---
 sw/source/core/text/inftxt.cxx |    2 --
 2 files changed, 5 deletions(-)

New commits:
commit a9f3cee384d9715d289a9a055f085c999b193d3c
Author:     Michael Stahl <Michael.Stahl at cib.de>
AuthorDate: Tue Apr 2 13:07:03 2019 +0200
Commit:     Michael Stahl <Michael.Stahl at cib.de>
CommitDate: Tue Apr 2 14:09:40 2019 +0200

    sw_redlinehide: remove WrongListIterator::MergedOrSame asserts
    
    There is a surprising recursion with 2 SwTextSlot on the stack, with 2
    consecutive fields, and then the m_pOldGrammarCheckList is not the
    SwTextNode's but a SubList of that, so the assert triggers spuriously.
    
     SwExpandPortion::Paint(SwTextPaintInfo const&) const (this=0x7d61cc0, rInf=...) at sw/source/core/text/porexp.cxx:66
     SwFieldPortion::Paint(SwTextPaintInfo const&) const (this=0x7d61cc0, rInf=...) at sw/source/core/text/porfld.cxx:437
     SwLinePortion::PrePaint(SwTextPaintInfo const&, SwLinePortion const*) const (this=0x7d61cc0, rInf=..., pLast=0x6fa49d0) at sw/source/core/text/porlin.cxx:138
     SwExpandPortion::Paint(SwTextPaintInfo const&) const (this=0x6fa49d0, rInf=...) at sw/source/core/text/porexp.cxx:78
    
    Change-Id: I915f31ce6de57b6a4a051598428441ae6c86d264
    Reviewed-on: https://gerrit.libreoffice.org/70131
    Tested-by: Jenkins
    Reviewed-by: Michael Stahl <Michael.Stahl at cib.de>

diff --git a/sw/source/core/inc/wrong.hxx b/sw/source/core/inc/wrong.hxx
index f7e22615578e..3fe7cb7bf75b 100644
--- a/sw/source/core/inc/wrong.hxx
+++ b/sw/source/core/inc/wrong.hxx
@@ -378,9 +378,6 @@ public:
     const SwWrongArea* GetWrongElement(TextFrameIndex nStart);
 
     bool LooksUseful() { return m_pMergedPara || m_pWrongList; }
-    bool MergedOrSame(SwWrongList const*const pList) const {
-        return m_pMergedPara || m_pWrongList == pList;
-    }
 };
 
 class WrongListIteratorCounter
diff --git a/sw/source/core/text/inftxt.cxx b/sw/source/core/text/inftxt.cxx
index 4fe1bf96e9bb..d94fd1cca717 100644
--- a/sw/source/core/text/inftxt.cxx
+++ b/sw/source/core/text/inftxt.cxx
@@ -1781,7 +1781,6 @@ SwTextSlot::SwTextSlot(
             {
                 std::pair<SwTextNode const*, sal_Int32> pos(pNew->GetTextFrame()->MapViewToModel(nIdx));
                 SwWrongList const*const pSmartTags(pos.first->GetSmartTags());
-                assert(m_pOldSmartTagList->MergedOrSame(pSmartTags));
                 if (pSmartTags)
                 {
                     const sal_uInt16 nPos = pSmartTags->GetWrongPos(pos.second);
@@ -1810,7 +1809,6 @@ SwTextSlot::SwTextSlot(
             {
                 std::pair<SwTextNode const*, sal_Int32> pos(pNew->GetTextFrame()->MapViewToModel(nIdx));
                 SwWrongList const*const pGrammar(pos.first->GetGrammarCheck());
-                assert(m_pOldGrammarCheckList->MergedOrSame(pGrammar));
                 if (pGrammar)
                 {
                     const sal_uInt16 nPos = pGrammar->GetWrongPos(pos.second);


More information about the Libreoffice-commits mailing list