[Kde-pim] Review Request 116985: mbox: make "compact now" working when file is already empty

Martin Koller kollix at aon.at
Sat Mar 22 21:41:06 GMT 2014


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

Review request for KDEPIM and Kevin Krammer.


Repository: kdepim-runtime


Description
-------

It often happens that my mbox file is definitely empty, e.g. because I removed it or emptied it with an external program.
In these cases the mbox resource can still not be compacted as the MBox::purge() returns false to notify that it could
not remove the given deleted items (which does not matter if the file is already empty).

This patch checks that even if purge() returns false but the file is already empty, we still discard our deleted items list
and do not issue an error.


Diffs
-----

  resources/mbox/compactpage.cpp f78782c 

Diff: https://git.reviewboard.kde.org/r/116985/diff/


Testing
-------

yes


Thanks,

Martin Koller

_______________________________________________
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/



More information about the kde-pim mailing list