MIME-Version: 1.0
Sender: awg@xvx.ca
In-Reply-To: <1329503684-31289-1-git-send-email-daniel@schoepe.org>
References: <1329503684-31289-1-git-send-email-daniel@schoepe.org>
Date: Wed, 18 Apr 2012 09:30:39 -0600
Message-ID:
 <CAMoJFUs_PcPxZD6eRw0S_SNwweKj1+yDCLCcNiFHWA+NiZEwYw@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
Content-Transfer-Encoding: quoted-printable
Cc: notmuch@notmuchmail.org
Precedence: list

Hi Daniel,

On Fri, Feb 17, 2012 at 11:34, Daniel Schoepe <daniel@schoepe.org> wrote:
> This introduces a variable to control after how many characters a line
> is wrapped by notmuch-wash-wrap-long-lines (still wrapping at the
> window width if it is lower).

I really like this idea, and I'm using this patch now. I've been
missing this mechanism in notmuch.

> +(defvar notmuch-wash-wrap-lines-length nil
> + =A0"Wrap line after at most this many characters.
> +
> +If this is nil, lines in messages will be wrapped to fit in the
> +current window. If this is a number, lines will be wrapped after
> +this many characters or at the window width (whichever one is
> +lower).")

This should probably be a defcustom, in the notmuch-show section.
Otherwise, LGTM.

-- Adam