From 9baa20e7a74162897816a179e422c77bedb484ee Mon Sep 17 00:00:00 2001 From: Carsten Burghardt Date: Mon, 23 May 2005 18:41:52 +0000 Subject: [PATCH] Ouch. No idea how that crept in but calling KMFolderIndex::take instead of KMFolderMaildir is simply wrong. BUGS:102171 svn path=/trunk/KDE/kdepim/; revision=417469 --- folderstorage.cpp | 3 +-- kmfoldermaildir.cpp | 9 ++++++--- kmfoldermaildir.h | 2 -- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/folderstorage.cpp b/folderstorage.cpp index bbb3e607f..b40c50333 100644 --- a/folderstorage.cpp +++ b/folderstorage.cpp @@ -453,8 +453,7 @@ void FolderStorage::take(QPtrList msgList) if (msg->parent()) { int idx = msg->parent()->find(msg); - assert( idx != -1); - FolderStorage::take(idx); + take(idx); } } } diff --git a/kmfoldermaildir.cpp b/kmfoldermaildir.cpp index ff751e5d6..9fd6261ef 100644 --- a/kmfoldermaildir.cpp +++ b/kmfoldermaildir.cpp @@ -940,12 +940,15 @@ KMMessage* KMFolderMaildir::take(int idx) // first, we do the high-level stuff.. then delete later KMMessage *msg = KMFolderIndex::take(idx); - if (!msg || !msg->fileName()) return 0; + if (!msg || !msg->fileName()) { + return 0; + } - if (removeFile(msg->fileName())) + if ( removeFile(msg->fileName()) ) { return msg; - else + } else { return 0; + } } // static diff --git a/kmfoldermaildir.h b/kmfoldermaildir.h index f062b5d94..afad6167b 100644 --- a/kmfoldermaildir.h +++ b/kmfoldermaildir.h @@ -35,8 +35,6 @@ public: /** Detach message from this folder. Usable to call addMsg() afterwards. Loads the message if it is not loaded up to now. */ virtual KMMessage* take(int idx); - virtual void take(QPtrList msgList) - { return KMFolderIndex::take(msgList); } /** Add the given message to the folder. Usually the message is added at the end of the folder. Returns zero on success and