[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    [Kde-pim] Re: Review Request: Add XLIST ability to KIMAP::ListJob
From:       "Gregory Schlomoff" <gregory.schlomoff () gmail ! com>
Date:       2010-11-23 1:18:48
Message-ID: 20101123011848.17374.90863 () vidsolbach ! de
[Download RAW message or body]



> On 2010-11-22 16:43:29, Torgny Nyblom wrote:
> > Come to think of it why add optional support for using the XLIST command instead \
> > of integrating this so that if the server supports XLIST always use it instead of \
> > LIST and use LIST as a fall back?

We discussed that on #akonadi, and the position from the lead developers of KIMAP was \
that it's a relatively low-level library, and that it should not do this kind of \
optimization.


- Gregory


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5928/#review8898
-----------------------------------------------------------


On 2010-11-22 10:23:19, Gregory Schlomoff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5928/
> -----------------------------------------------------------
> 
> (Updated 2010-11-22 10:23:19)
> 
> 
> Review request for KDE PIM and Kevin Ottens.
> 
> 
> Summary
> -------
> 
> This patch adds the ability to list mailboxes using the XLIST command instead of \
> LIST. XLIST is an extension to IMAP that provides localized folder names. It's \
> prominently used by Gmail's IMAP servers. 
> As the ListJob currently accepts a boolean flag to specify whether we want to \
> include unsubscribed folders, this patch also replaces this flag by a ListingOption \
> enum. The includeUnsubscribed() / setIncludeUnsubscribed() are now convenience \
> functions to set the appropriate enum value, marked as deprecated. 
> 
> Diffs
> -----
> 
> /trunk/KDE/kdepimlibs/kimap/listjob.h 1199501 
> /trunk/KDE/kdepimlibs/kimap/listjob.cpp 1199501 
> 
> Diff: http://svn.reviewboard.kde.org/r/5928/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gregory
> 
> 

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


[prev in list] [next in list] [prev in thread] [next in thread]