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