[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/KDE/3.5/kdewebdev/kommander/widgets
From:       Eric Laffoon <sequitur () kde ! org>
Date:       2009-05-08 11:12:26
Message-ID: 1241781146.247630.1224.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 965171 by sequitur:

fix current item crash

 M  +10 -5     treewidget.cpp  


--- branches/KDE/3.5/kdewebdev/kommander/widgets/treewidget.cpp #965170:965171
@@ -143,8 +143,8 @@
 
 int TreeWidget::itemToIndex(QListViewItem* item)
 {
-  if (!item->isSelected())
-    return -1;
+//  if (!item->isSelected())
+//    return -1;
   QListViewItemIterator it(this);
   int index = 0;
   while (it.current()) {
@@ -377,11 +377,16 @@
       m_lastPath.clear();
       break;
     case DCOP::removeItem:
-      delete indexToItem(args[0].toInt());
-      m_lastPath.clear();
+    {
+      if (args[0].toInt() >= 0 )
+      {
+        delete indexToItem(args[0].toInt());
+        m_lastPath.clear();
+      }
       break;
+    }
     case DCOP::currentItem:
-      return QString::number(itemToIndex(currentItem()));
+      return QString::number(itemToIndexSafe(currentItem()));
       break;
     case DCOP::setCurrentItem:
       setCurrentItem(indexToItem(args[0].toInt()));
[prev in list] [next in list] [prev in thread] [next in thread]