[Kde-pim] Review Request 114341: Fix a bug in kmail that eats all cpu

[Raul Fernandes]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114341/
-----------------------------------------------------------

(Updated Dec. 7, 2013, 6:43 p.m.)


Review request for KDEPIM.


Changes
-------

Log with statements sent to mysql.
The file has 75 MB decompressed.


Bugs: 323929
    http://bugs.kde.org/show_bug.cgi?id=323929


Repository: kdepim


Description
-------

The bug is described in https://bugs.kde.org/show_bug.cgi?id=323929.
The bug triggers several queries to akonadi to get the same information.
I think it is caused by the repeated connect() calls in initFolders() function (even using the Qt::UniqueConnection).
So everytime the function slotDefaultCollectionsChanged() is called, disconnect the signal to avoid accumulate multiple signals.
I think the right way is to rewrite the initFolders to call connect() only once.
Split initFolders() in 2 functions. One is called by kmail and another (with the connect()) called by slotDefaultCollectionsChanged().
For now, the disconnect() should be a good workaround for the (critical) bug.


Diffs
-----

  mailcommon/kernel/mailkernel.cpp 5b92778 

Diff: http://git.reviewboard.kde.org/r/114341/diff/


Testing
-------

Until now, it is ok here.


File Attachments (updated)
----------------

kmail.log.lrz
  http://git.reviewboard.kde.org/media/uploaded/files/2013/12/07/1fe839f7-aea6-462a-8310-57be1afbb14f__kmail.log.lrz


Thanks,

Raul Fernandes

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/