[Evolution-hackers] Disabled code in Evolution has a bug

Hi there,

In Evolution this code is not used (imap_parse_namespace_response ->
imap_namespace_decode), nonetheless it contains a bug.

If the delimiter for a namespace is NIL, the imap_next_word will search
for the first space. This ain't correct if there are no parameters to
parse. For example (("MyShared" NIL)).


I'm adding Jeffrey in CC because I think he once intended to use this
code to improve namespace support?

Camel-lite has gotten too specific in this area of the code: it supports
the shared and personal namespace by only asking for LSUB recursively
for those two namespaces, and LIST and LSUB for the personal namespace.

The namespace-line parsing, however, is recovered from things that I
found in existing (but not used) camel's imap code.

ps. I commented out the original line in camel-lite because the += 3
might not be the right fix (perhaps is fixing imap_next_word better).


Philip Van Hoof, freelance software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]