whytofly
/
qt6-bb10
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

QAbstractItemView: code tidies 

In preparation for an upcoming fix, refactor an if over an enumerator
to a switch (which is how this code should've been to begin with).

Change-Id: I11a2de6d66f0359b985b587b7fd37022a7bf56e6
Pick-to: 5.15 6.2 6.3
Reviewed-by: David Faure <david.faure@kdab.com>
bb10
Giuseppe D'Angelo 2022-01-27 09:46:56 +01:00
parent 472722382d
commit 22ac61e62a
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/widgets/itemviews/qabstractitemview.cpp
View File

@ -1204,12 +1204,20 @@ QModelIndex QAbstractItemView::rootIndex() const
void QAbstractItemView::selectAll()
{
Q_D(QAbstractItemView);
SelectionMode mode = d->selectionMode;
if (mode == MultiSelection || mode == ExtendedSelection)
const SelectionMode mode = d->selectionMode;
switch (mode) {
case MultiSelection:
case ExtendedSelection:
d->selectAll(QItemSelectionModel::ClearAndSelect
|d->selectionBehaviorFlags());
else if (mode != SingleSelection)
| d->selectionBehaviorFlags());
break;
case NoSelection:
case ContiguousSelection:
d->selectAll(selectionCommand(d->model->index(0, 0, d->root)));
break;
case SingleSelection:
break;
}
}
/*!