[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#21057: [PATCH] nnimap.el: add support for IMAP namespaces
From: |
Lars Ingebrigtsen |
Subject: |
bug#21057: [PATCH] nnimap.el: add support for IMAP namespaces |
Date: |
Wed, 27 Dec 2017 22:10:36 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) |
Nikolaus Rath <Nikolaus@rath.org> writes:
>> Looks good to me. It needs to be documented in the manual, though, and
>> perhaps a NEWS entry...
>
> Updated patch is attached.
Looks good, but a couple of niggles:
> +(setq nnimap-namespaces nil)
This should be a defvar, or rather, a defvoo.
[...]
> + (let* ((prefix (cadr (assoc (nnoo-current-server 'nnimap)
> + nnimap-namespaces)))
> + (inbox (substring prefix 0 -1)))
And then you could just set nnimap-namespaces without maintaining a
per-server alist, since that's what the voo machinery does for you
(switching out variables on a per-server basis).
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no