Re: Patch: Support for non-ascii characters in imap group names

[Ted Zlatanov <tzz@lifelogs.com>]

On Sun, 21 Mar 2010 18:00:55 +0100 Martin Stjernholm <mast@lysator.liu.se> wrote: 

MS> Ted Zlatanov <tzz@lifelogs.com> wrote:
>> Thank you, it was my fault for ignoring it.

MS> I'm glad it got through. I trust then that my followup patch to improve
MS> the nnimap startup time has reached relevant people even though I have
MS> yet to see it in list archives.

I haven't seen it (which is probably my fault as I was not paying
attention), can you please resend and CC me?

>> I'm not sure why you need
>> 
>> (nnimap-close-group): Call `imap-current-mailbox' instead of using the
>> variable `imap-current-mailbox'.
>> 
>> but left it in, assuming it was sensible.

MS> The variable is utf-7 encoded while the function returns the decoded
MS> name, so using the variable would make the name would look garbled in
MS> that message. In general, I tried to use the functions in imap.el that
MS> deal with decoded names, to keep it clean. There are a few cases where
MS> the variable is still used, when only nil/non-nil matters.

Interesting.  It sounds like we should get rid of the variable in code
completely and always use an accessor function with a "decoding"
parameter (so we can get the encoded or the unencoded name).  I don't
like the current hidden behavior.  Do you agree?

Thanks
Ted