[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