diff options
authorMichael Stahl <>2018-12-05 16:37:18 +0100
committerMichael Stahl <>2018-12-18 17:55:30 +0100
commitb310378e874bc8fa7005352fcd85fa64eb075f54 (patch)
parent7f42b4de3a5a72e695da0705ddc7db8bce576901 (diff)
sw_redlinehide_4b: temporarily add an assert that is wrong
This assert is wrong, because there may be a legimate need to query a position that is currently hidden. However, oddly enough, we haven't found a legitimate document yet that triggers this, and it has pointed us to several interesting functions that needed adapting to merged frames... so keep it a little while and remove it (and return something other than COMPLETE_STRING from mapping functions) later... Change-Id: Ide2abf0c2ce054b75c917063a5d5aa8d13430426
1 files changed, 1 insertions, 0 deletions
diff --git a/sw/source/core/text/frmcrsr.cxx b/sw/source/core/text/frmcrsr.cxx
index 7b6411163cb3..c756aedf4cd2 100644
--- a/sw/source/core/text/frmcrsr.cxx
+++ b/sw/source/core/text/frmcrsr.cxx
@@ -267,6 +267,7 @@ bool SwTextFrame::GetCharRect( SwRect& rOrig, const SwPosition &rPos,
bool bGoOn = true;
TextFrameIndex const nOffset = MapModelToViewPos(rPos);
+ assert(nOffset != TextFrameIndex(COMPLETE_STRING)); // not going to end well
TextFrameIndex nNextOfst;