diff options
author | ChunLin Wang <wangchunlin@uniontech.com> | 2021-08-26 21:15:31 +0800 |
---|---|---|
committer | ChunLin Wang <wangchunlin@uniontech.com> | 2021-09-01 16:03:15 +0000 |
commit | ca69e5aeef2fef540e687475ac00a4f332fdc5f3 (patch) | |
tree | 31ee0d390d7b5862400880d89bc93b4d246f9bc3 /doc | |
parent | 38ec2c830b849ad44ca7e16bd9c4722e0bcdb61f (diff) |
Fix QListView assert when the last row is moved in IconMode
After the last row is moved, 0 will be returned when obtaining
row and column data. At this time, QListView::doitemslayout will
not call d->doitemslayout, so the QBspTree data structure will
not be cleaned up, leaving a stale tree structure behind. This
will trigger an assert during paintEvent handling if QListView is
set to IconMode
In QListView::ListMode the test for a valid model index doesn't
use an assert.
Call QListViewPrivate::clear explicitly if the column count is 0
so that the QBspTree and other data structures are cleared.
Add a test case that simulates this scenario by implementing a
model that returns a 0 column count for an index after the model
structure was changed through a move of rows.
Done-with: Volker Hilsheimer
Fixes: QTBUG-95463
Pick-to: 5.15 6.1 6.2
Change-Id: I36419be5459b8ced930c619f538482ea1db4ad03
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Diffstat (limited to 'doc')
0 files changed, 0 insertions, 0 deletions