On Wed, Apr 18, 2012 at 09:46, Daniel Schoepe <daniel at schoepe.org> wrote: > the reason for not making this a defcustom was that all of the other > functions in notmuch-wash.el use defvar for customization too, so > perhaps they should all be defcustom'd in a separate patch?
Ah, yes, I see now. Most of those variables (the regexps and button-formats) probably don't need to be customizable - it seems unlikely that anyone will want to change them except for internationalization, which should probably be done differently anyway. I think it makes sense for the others to be customs, so doing that in a separate patch is a good idea. Thanks again for this patch - it makes my mail read reading much more pleasant. -- Adam
