diff options
| author | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2022-01-27 09:46:56 +0100 |
|---|---|---|
| committer | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2022-02-01 14:32:57 +0100 |
| commit | 22ac61e62a1a65bf0049229ab6fcaa44e8688951 (patch) | |
| tree | 556d4a5e773a6ec30c5e0bed4500afb9743256b6 /src | |
| parent | 472722382d6aa7560eae716aca76763383ae7984 (diff) | |
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>
Diffstat (limited to 'src')
| -rw-r--r-- | src/widgets/itemviews/qabstractitemview.cpp | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/src/widgets/itemviews/qabstractitemview.cpp b/src/widgets/itemviews/qabstractitemview.cpp index 43919a2efb..9727ddf3d9 100644 --- a/src/widgets/itemviews/qabstractitemview.cpp +++ b/src/widgets/itemviews/qabstractitemview.cpp @@ -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; + } } /*! |