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

List:       kmail-devel
Subject:    [Bug 84215] check new mail on IMAP and sending email hangs
From:       Till Adam <adam () kde ! org>
Date:       2004-07-26 21:35:39
Message-ID: 20040726213539.12799.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
      
http://bugs.kde.org/show_bug.cgi?id=84215      
adam kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From adam kde org  2004-07-26 23:35 -------
CVS commit by tilladam: 

Make sure an imap sent mail folder is properly kept open during move of
the message from the local sent mail (where it is placed for fallback) to
the imap folder by using a KMMoveCommand, which nicely encapsulates all
that. This avoids the folder constantly opening and closing during
processing of incoming data which makes KMail appear hung and consumes
massive amounts of cpu.

Ferdinand, can you confirm this fixes it?

CCMAIL: 84215-done bugs kde org


  M +6 -1      kmsender.cpp   1.218


--- kdepim/kmail/kmsender.cpp  #1.217:1.218
 @ -36,4 +36,5  @ using namespace KMime::Types;
 #include "kmmsgpart.h"
 #include "protocols.h"
+#include "kmcommands.h"
 #include <mimelib/mediatyp.h>
 
 @ -350,5 +351,9  @ void KMSender::doSendMsg()
         return;
       }
-      if (imapSentFolder) imapSentFolder->moveMsg(mCurrentMsg);
+      if (imapSentFolder) {
+        // Does proper folder refcounting and message locking
+        KMCommand *command = new KMMoveCommand( imapSentFolder, mCurrentMsg );
+        command->start();
+      }
     default:
       break;
_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic