Details
-
Suggestion
-
Resolution: Unresolved
-
Not Evaluated
-
None
-
None
-
None
Description
When fixing a bug in QAIV, I've spotted a strange asymmetry: QAbstractItemView::rowsAboutToBeRemoved has
// Ensure one selected item in single selection mode.
QModelIndex current = currentIndex();
if (d->selectionMode == SingleSelection
&& current.isValid()
&& current.row() >= start
&& current.row() <= end
&& current.parent() == parent) { // <-
while QAbstractItemViewPrivate::_q_columnsAboutToBeRemoved has
// Ensure one selected item in single selection mode.
QModelIndex current = q->currentIndex();
if (current.isValid()
&& selectionMode == QAbstractItemView::SingleSelection
&& current.column() >= start
&& current.column() <= end) {
Why isn't this checking that the current index's parent is the same under which columns are being removed? Is there some subtlety going on that likely deserves some comments in the code?