Swiftdiff options
| author | Kevin Smith <git@kismith.co.uk> | 2009-10-11 21:35:54 (GMT) |
|---|---|---|
| committer | Kevin Smith <git@kismith.co.uk> | 2009-10-11 21:35:54 (GMT) |
| commit | c061f286915bdffa93d4dc722764eae1636291cc (patch) | |
| tree | 11b5c28531fc3800df9c32f9fa4dffb174f9eb46 /Swift/QtUI/Roster | |
| parent | cbaa100229b5001f41f13912fba2125213ab3d1f (diff) | |
| download | swift-c061f286915bdffa93d4dc722764eae1636291cc.zip swift-c061f286915bdffa93d4dc722764eae1636291cc.tar.bz2 | |
Fix for segfault on exit with open MUCs, and reinstate roster sorting.
Diffstat (limited to 'Swift/QtUI/Roster')
| -rw-r--r-- | Swift/QtUI/Roster/QtTreeWidgetItem.cpp | 2 | ||||
| -rw-r--r-- | Swift/QtUI/Roster/RosterModel.cpp | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/Swift/QtUI/Roster/QtTreeWidgetItem.cpp b/Swift/QtUI/Roster/QtTreeWidgetItem.cpp index b237929..1f2964e 100644 --- a/Swift/QtUI/Roster/QtTreeWidgetItem.cpp +++ b/Swift/QtUI/Roster/QtTreeWidgetItem.cpp @@ -121,7 +121,7 @@ void QtTreeWidgetItem::removeChild(QtTreeWidgetItem* child) { } void bubbleSort(QList<QtTreeWidgetItem*>& list) { - bool done = true; + bool done = false; for (int i = 0; i < list.size() - 1 && !done; i++) { done = true; for (int j = i + 1; j < list.size(); j++) { diff --git a/Swift/QtUI/Roster/RosterModel.cpp b/Swift/QtUI/Roster/RosterModel.cpp index 8a319ea..568c1f5 100644 --- a/Swift/QtUI/Roster/RosterModel.cpp +++ b/Swift/QtUI/Roster/RosterModel.cpp @@ -58,7 +58,8 @@ QModelIndex RosterModel::parent(const QModelIndex& index) const { Q_ASSERT(item); QtTreeWidgetItem* parentItem = item->getParentItem(); - return parentItem == tree_ ? QModelIndex() : createIndex(parentItem->row(), 0, parentItem); + /* parentItem_ == NULL can happen during destruction.*/ + return parentItem == tree_ || parentItem == NULL ? QModelIndex() : createIndex(parentItem->row(), 0, parentItem); } |