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

[Martin Koller]

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

(Updated April 1, 2014, 7:50 p.m.)


Status
------

This change has been marked as submitted.


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/