Prechádzať zdrojové kódy

XnatTreeModel: Make sure last is at least first when inserting rows

Adding a item to a previously empty resource folder would result in first
being 0 and last being -1. This would trigger a Qt assert.

Reviewed-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>
Caspar Goch 9 rokov pred
rodič
commit
6de4b6276b
1 zmenil súbory, kde vykonal 2 pridanie a 1 odobranie
  1. 2 1
      Libs/XNAT/Core/ctkXnatTreeModel.cpp

+ 2 - 1
Libs/XNAT/Core/ctkXnatTreeModel.cpp

@@ -291,7 +291,8 @@ void ctkXnatTreeModel::refresh(const QModelIndex& parent)
       // -> add it to the treeview
       if (addToTreeView)
       {
-        beginInsertRows(parent, 0, numChildren - 1);
+        int last = qMax(0, numChildren - 1);
+        beginInsertRows(parent, 0, last);
         item->appendChild(new ctkXnatTreeItem(child, item));
         endInsertRows();
         ++numChildren;