syncing custom IMAP flags

Sebastian Spaeth Sebastian at SSpaeth.de
Thu Dec 9 16:39:01 GMT 2010


On Thu, 09 Dec 2010 10:12:02 -0500, Dan Christensen <jdc at uwo.ca> wrote:
> The patch is simply logical code clean-up:  instead of repeating a
> mapping table twice, once for each direction, it is only stored once.
> This is safer and faster.  So I think you should merge it.

I had a look at the patch and it seems to be indeed this: a cleanup
without functional changes.

One flagmap rather than two. So this would get a 

Signed-off-by: Sebastian Spaeth <Sebastian at SSpaeth.de>

From me. I would feel even more comfortable if those functions had
documentation, especially what type of input parameter they expect, and
what they return. But as I said, looks fine to me.

Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/offlineimap-project/attachments/20101209/02fa56a5/attachment-0001.sig>


More information about the OfflineIMAP-project mailing list