summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEike Rathke <erack@redhat.com>2016-11-09 18:44:01 +0100
committerEike Rathke <erack@redhat.com>2016-11-09 18:45:17 +0100
commit6699afb43bffb9148960cbee2013d1704e462b9e (patch)
tree090832f1ff5022c40317bee070dbf26586d1dda7
parentfa69c29c89608e50d8d41c0008f31c30e60c6d20 (diff)
assert a supposed-to-be-fixed condition, tdf#89957 related
Change-Id: Ie00840e0b8cff747e131b6bc9def0ddaf57edea7
-rw-r--r--sc/source/core/tool/grouparealistener.cxx10
1 files changed, 9 insertions, 1 deletions
diff --git a/sc/source/core/tool/grouparealistener.cxx b/sc/source/core/tool/grouparealistener.cxx
index 5c5b0a310c5c..aca46a4585b0 100644
--- a/sc/source/core/tool/grouparealistener.cxx
+++ b/sc/source/core/tool/grouparealistener.cxx
@@ -219,7 +219,7 @@ void FormulaGroupAreaListener::collectFormulaCells(
const_cast<FormulaGroupAreaListener*>(this)->mnGroupLen = nLen;
}
- /* FIXME: with tdf#89957 it happened that the actual block size in column
+ /* With tdf#89957 it happened that the actual block size in column
* AP (shifted from AO) of sheet 'w' was smaller than the remembered group
* length and correct. This is just a very ugly workaround, the real cause
* is yet unknown, but at least don't crash in such case. The intermediate
@@ -233,6 +233,14 @@ void FormulaGroupAreaListener::collectFormulaCells(
SAL_WARN("sc.core","FormulaGroupAreaListener::collectFormulaCells() nBlockSize " <<
nBlockSize << " < " << mnGroupLen << " mnGroupLen");
const_cast<FormulaGroupAreaListener*>(this)->mnGroupLen = static_cast<SCROW>(nBlockSize);
+
+ // erAck: 2016-11-09T18:30+01:00 XXX This doesn't occur anymore, at
+ // least not in the original bug scenario (insert a column before H on
+ // sheet w) of tdf#89957 with
+ // http://bugs.documentfoundation.org/attachment.cgi?id=114042
+ // Apparently this was fixed in the mean time, let's assume and get the
+ // assert bat out to hit us if it wasn't.
+ assert(!"something is still messing up the formula goup and block size length");
}
ScFormulaCell* const * ppEnd = pp + mnGroupLen;