To: Jameson Graef Rollins <jrollins@finestructure.net>
Subject: Re: [PATCH 2/3] emacs: Don't return the button from
	`notmuch-show-insert-part-header'.
In-Reply-To: <87ipk22439.fsf@servo.finestructure.net>
References: <1327052612-1040-1-git-send-email-dme@dme.org>
	<1327052612-1040-2-git-send-email-dme@dme.org>
	<877h0jmn9a.fsf@servo.finestructure.net>
	<cun8vky6dh8.fsf@hotblack-desiato.hh.sledj.net>
	<87ipk22439.fsf@servo.finestructure.net>
User-Agent: Notmuch/0.11+73~gd51b784 (http://notmuchmail.org) Emacs/24.0.92.1
	(x86_64-pc-linux-gnu)
From: David Edmondson <dme@dme.org>
Date: Mon, 23 Jan 2012 09:12:13 +0000
Message-ID: <cun39b66avm.fsf@hotblack-desiato.hh.sledj.net>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha1; protocol="application/pgp-signature"
Cc: notmuch <notmuch@notmuchmail.org>
Precedence: list

--=-=-=
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

On Mon, 23 Jan 2012 00:52:26 -0800, Jameson Graef Rollins <jrollins@finestr=
ucture.net> wrote:
> On Mon, 23 Jan 2012 08:16:03 +0000, David Edmondson <dme@dme.org> wrote:
> > There was no problem with the logic. The code in the two functions was
> > almost identical, so I'd like to make any future changes in just one
> > place.
> >
> > You didn't actually answer my question - is the logic in the new
> > function correct?
>=20
> Honestly I didn't look too closely yet since I'm not convinced we need
> the change at all.  I would prefer to keep the functions separate.  In
> my opinion, enough special casing would be required that it wouldn't be
> worth it, and it would make the code less clear.

Okay.

--=-=-=
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iEYEARECAAYFAk8dJG0ACgkQaezQq/BJZRbO/QCdHPWe9hYLGNb4XIGGu2JlCrMB
0mIAn3fRdxmFfmYu+OH+atVFM8zJggu3
=2Vi0
-----END PGP SIGNATURE-----
--=-=-=--