[SCM] ktp-text-ui packaging branch, master, updated. debian/15.12.1-1-1918-gdf4b0ec

Maximiliano Curia maxy at moszumanska.debian.org
Sat May 28 00:20:53 UTC 2016


Gitweb-URL: http://git.debian.org/?p=pkg-kde/applications/ktp-text-ui.git;a=commitdiff;h=31811e1

The following commit has been merged in the master branch:
commit 31811e13dc0c9b8899d6b53414d1be526d789293
Author: Lasath Fernando <kde at lasath.org>
Date:   Tue Jan 10 19:49:52 2012 +1100

    Changed implementations of QAbstractItemModel::data() to use QModelIndex::isValid().
---
 lib/conversations-model.cpp | 2 +-
 lib/messages-model.cpp      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/conversations-model.cpp b/lib/conversations-model.cpp
index 6de4ff8..514e1da 100644
--- a/lib/conversations-model.cpp
+++ b/lib/conversations-model.cpp
@@ -34,7 +34,7 @@ public:
 QVariant ConversationsModel::data(const QModelIndex& index, int role) const
 {
     QVariant result;
-    if (index.row() >= 0 && index.row() < d->conversations.count()) {
+    if (index.isValid()) {
         Q_ASSERT(role == ConversationRole);
         result = QVariant::fromValue<Conversation*>(d->conversations[index.row()]);
         kDebug() << "returning value " << result;
diff --git a/lib/messages-model.cpp b/lib/messages-model.cpp
index bbc4da9..bb1f025 100644
--- a/lib/messages-model.cpp
+++ b/lib/messages-model.cpp
@@ -180,7 +180,7 @@ QVariant MessagesModel::data(const QModelIndex& index, int role) const
 {
     QVariant result;
 
-    if (index.row() < d->messages.size()) {
+    if (index.isValid()) {
         MessageItem* requestedData = &d->messages[index.row()];
 
         switch (role) {

-- 
ktp-text-ui packaging



More information about the pkg-kde-commits mailing list