[Libreoffice-commits] core.git: Branch 'libreoffice-6-2' - sw/source

Michael Stahl (via logerrit) logerrit at kemper.freedesktop.org
Tue Apr 2 14:05:53 UTC 2019


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

New commits:
commit 32fb8d6a4be8187580bf228b74401f08b1209f92
Author:     Michael Stahl <Michael.Stahl at cib.de>
AuthorDate: Tue Apr 2 13:07:03 2019 +0200
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Tue Apr 2 16:05:32 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>
    (cherry picked from commit a9f3cee384d9715d289a9a055f085c999b193d3c)
    Reviewed-on: https://gerrit.libreoffice.org/70137
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/source/core/inc/wrong.hxx b/sw/source/core/inc/wrong.hxx
index 8340d3337ca4..ec8106ea889d 100644
--- a/sw/source/core/inc/wrong.hxx
+++ b/sw/source/core/inc/wrong.hxx
@@ -379,9 +379,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 3bee295c6bd9..29ef2035bee7 100644
--- a/sw/source/core/text/inftxt.cxx
+++ b/sw/source/core/text/inftxt.cxx
@@ -1773,7 +1773,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);
@@ -1802,7 +1801,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