MIME-Version: 1.0
Sender: awg@xvx.ca
In-Reply-To: <87bompjb4n.fsf@schoepe.localhost>
References: <1329503684-31289-1-git-send-email-daniel@schoepe.org>
	<CAMoJFUs_PcPxZD6eRw0S_SNwweKj1+yDCLCcNiFHWA+NiZEwYw@mail.gmail.com>
	<87bompjb4n.fsf@schoepe.localhost>
Date: Thu, 19 Apr 2012 08:33:31 -0600
Message-ID:
 <CAMoJFUvYMrN22DWDqpQXmLOypn82N7P76fgkyoN23ry+8Zn7pg@mail.gmail.com>
Subject: Re: [PATCH] emacs: Add configurable wrapping width for
	notmuch-wash-wrap-long-lines
From: Adam Wolfe Gordon <awg+notmuch@xvx.ca>
To: Daniel Schoepe <daniel@schoepe.org>
Content-Type: text/plain; charset=ISO-8859-1
Cc: notmuch@notmuchmail.org
Precedence: list

On Wed, Apr 18, 2012 at 09:46, Daniel Schoepe <daniel@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