[Kde-pim] Review Request 111726: Make use of the CONDSTORE IMAP extension

Volker Krause vkrause at kde.org
Sat Jul 27 09:37:12 BST 2013


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


Awesome, thanks for looking into supporting CONDSTORE!

How does this behave if the server doesn't support CONDSTORE? Fall back to listing everything automatically because the highest modified sequence number will be 0?


resources/imap/retrieveitemstask.cpp
<http://git.reviewboard.kde.org/r/111726/#comment26987>

    I might be missing something, but isn't this the code path that handled server-side deletions?
    
    All the new itemsRetrievedIncremental() calls have empty deletion lists it seems.


- Volker Krause


On July 26, 2013, 4:08 p.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111726/
> -----------------------------------------------------------
> 
> (Updated July 26, 2013, 4:08 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> IMAP resource can now sync flag changes at the speed of light ;)
> 
> 
> Diffs
> -----
> 
>   resources/imap/retrieveitemstask.cpp 1df3f6e 
>   resources/imap/retrieveitemstask.h b90cac1 
> 
> Diff: http://git.reviewboard.kde.org/r/111726/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