[Kde-pim] Review Request 111720: Partial implementation of CONDSTORE IMAP extension (RFC4551)

Dan Vrátil dvratil at redhat.com
Fri Jul 26 17:07:03 BST 2013


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

(Updated July 26, 2013, 6:07 p.m.)


Review request for KDEPIM-Libraries.


Changes
-------

 - updated unittest for FETCH
 - removed unrelated diff


Description
-------

This patch adds partial support for CONDSTORE IMAP extension for fast flags change synchronization, specifically parts 3.1.1 (HIGHESTMODSEQ in SELECT reply) and 3.3.1 (CHANGEDSINCE parameter in FETCH command).

Parts 3.1.2 and 3.3.2 (NOMODSEQ) are omitted, because their support is not necessary thanks to current KIMAP design
Part 3.2 (CONDSTORE in STORE command) is omitted because it seems to be too complicated to implement with current KIMAP design and given we don't even need to use it...
Part 3.4 (MODSEQ in SEARCH command) is omitted because we don't actually use or support SEARCH with Akonadi (not yet, at least)
Part 3.7 (CONDSTORE in SELECT command) is omitted because it makes no sense in relation to current KIMAP design


Diffs (updated)
-----

  kimap/tests/fetchjobtest.cpp 0d97ba1 
  kimap/tests/selectjobtest.cpp f0b7185 
  kimap/sessionthread.cpp c7af134 
  kimap/selectjob.h 87157b5 
  kimap/selectjob.cpp 7854758 
  kimap/fetchjob.h 75fe6b6 
  kimap/fetchjob.cpp 3a94e02 

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


Testing
-------


Thanks,

Dan Vrátil

_______________________________________________
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