diff options
author | Stephen Kelly <stephen.kelly@kdab.com> | 2012-09-26 16:57:38 +0200 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2012-09-26 19:09:14 +0200 |
commit | ebf0ee34e7a50b8dbe9ed766a9737a8f77805449 (patch) | |
tree | e92c7e2b5c256e74824dc041aafdba35a2180d4e /src/corelib/itemmodels/qitemselectionmodel.cpp | |
parent | 9b439161ac7e85d7c54a403907d7b18843454f83 (diff) |
Don't call parent() so often when processing a range.
Task-number: QTBUG-17732
Change-Id: Iefe5801f5941136f77001c532870e22f7fdd2c69
Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
Diffstat (limited to 'src/corelib/itemmodels/qitemselectionmodel.cpp')
-rw-r--r-- | src/corelib/itemmodels/qitemselectionmodel.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/corelib/itemmodels/qitemselectionmodel.cpp b/src/corelib/itemmodels/qitemselectionmodel.cpp index 1308b07499..98afaa3b48 100644 --- a/src/corelib/itemmodels/qitemselectionmodel.cpp +++ b/src/corelib/itemmodels/qitemselectionmodel.cpp @@ -294,9 +294,10 @@ QItemSelectionRange QItemSelectionRange::intersected(const QItemSelectionRange & static void indexesFromRange(const QItemSelectionRange &range, QModelIndexList &result) { if (range.isValid() && range.model()) { + const QModelIndex parent = range.parent(); for (int column = range.left(); column <= range.right(); ++column) { for (int row = range.top(); row <= range.bottom(); ++row) { - QModelIndex index = range.model()->index(row, column, range.parent()); + QModelIndex index = range.model()->index(row, column, parent); Qt::ItemFlags flags = range.model()->flags(index); if ((flags & Qt::ItemIsSelectable) && (flags & Qt::ItemIsEnabled)) result.append(index); |