diff options
| author | Gabriel de Dietrich <gabriel.dietrich-de@nokia.com> | 2010-01-22 16:36:05 (GMT) |
|---|---|---|
| committer | Prasanth Ullattil <prasanth.ullattil@nokia.com> | 2010-01-27 15:30:26 (GMT) |
| commit | 9098631b8a3927ebdf945ccdbaf20b9a7616636d (patch) | |
| tree | 30ad52cb826a477fb8afdde0502f50dfc1310af0 /src/gui | |
| parent | 3ec5241a21d233cac879e535719fb546f391add3 (diff) | |
| download | Qt-9098631b8a3927ebdf945ccdbaf20b9a7616636d.zip Qt-9098631b8a3927ebdf945ccdbaf20b9a7616636d.tar.gz Qt-9098631b8a3927ebdf945ccdbaf20b9a7616636d.tar.bz2 | |
Another ASSERT while deleting spans
That rare case when we are deleting the last span was not being taken care
of. Unbelievable but true. Auto-test included.
Reviewed-by: Thierry
Reviewed-by: leo
Task-number: QTBUG-6004
Diffstat (limited to 'src/gui')
| -rw-r--r-- | src/gui/itemviews/qtableview.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/gui/itemviews/qtableview.cpp b/src/gui/itemviews/qtableview.cpp index 7eefb0b..3111896 100644 --- a/src/gui/itemviews/qtableview.cpp +++ b/src/gui/itemviews/qtableview.cpp @@ -114,7 +114,9 @@ void QSpanCollection::updateSpan(QSpanCollection::Span *span, int old_height) } } else if (old_height > span->height()) { //remove the span from all the subspans lists that intersect the columns not covered anymore - Index::iterator it_y = index.lowerBound(qMin(-span->bottom(), 0)); + Index::iterator it_y = index.lowerBound(-span->bottom()); + if (it_y == index.end()) + it_y = index.find(-span->top()); // This is the only span remaining and we are deleting it. Q_ASSERT(it_y != index.end()); //it_y must exist since the span is in the list while (-it_y.key() <= span->top() + old_height -1) { if (-it_y.key() > span->bottom()) { |