[SCM] ktp-common-internals packaging branch, master, updated. debian/15.12.1-2-1839-gf0635e9

[Maximiliano Curia]

Gitweb-URL: http://git.debian.org/?p=pkg-kde/applications/ktp-common-internals.git;a=commitdiff;h=ee4fe39

The following commit has been merged in the master branch:
commit ee4fe39684c30b0762c60cc7c8fbd4b13cefbff4
Author: Thomas Richard <thomas9999 at gmail.com>
Date:   Sun Jan 16 18:15:00 2011 +0100

    Fix some deletes
    Fix some TODOs that were not needed
---
 KTp/Models/accounts-list-model.cpp | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/KTp/Models/accounts-list-model.cpp b/KTp/Models/accounts-list-model.cpp
index 4db4f5a..b51e4f1 100644
--- a/KTp/Models/accounts-list-model.cpp
+++ b/KTp/Models/accounts-list-model.cpp
@@ -38,8 +38,6 @@ AccountsListModel::AccountsListModel(QObject *parent)
 AccountsListModel::~AccountsListModel()
 {
     kDebug();
-
-    // TODO: Implement me!
 }
 
 int AccountsListModel::rowCount(const QModelIndex &index) const
@@ -187,15 +185,12 @@ void AccountsListModel::onAccountItemRemoved()
         return;
     }
 
-    beginRemoveRows(QModelIndex(), m_accounts.lastIndexOf(item),
+    beginRemoveRows(QModelIndex(), m_accounts.indexOf(item),
                     m_accounts.lastIndexOf(item));
     m_accounts.removeAll(item);
     endRemoveRows();
 
-    Q_ASSERT(!m_accounts.contains(item));
-    if (m_accounts.contains(item)) {
-        kWarning() << "Ready Accounts still contains Accout Item:" << item;
-    }
+    delete item;
 }
 
 void AccountsListModel::onAccountItemUpdated()

-- 
ktp-common-internals packaging