From: Michael Haggerty Date: Mon, 14 Jan 2013 09:33:33 GMT Subject: Re: [PATCH 00/14] Remove unused code from imap-send.c Message-ID: <50F3D0ED.5030103@alum.mit.edu> In-Reply-To: <20130114065757.GK3125@elie.Belkin> On 01/14/2013 07:57 AM, Jonathan Nieder wrote: > Michael Haggerty wrote: > >> imap-send.c | 286 +++++++++--------------------------------------------------- >> 1 file changed, 39 insertions(+), 247 deletions(-) > > See my replies for comments on patches 1, 6, 9, 11, and 12. The rest > are > > Reviewed-by: Jonathan Nieder > > The series is tasteful and easy to follow and it's hard to argue with > the resulting code reduction. Thanks for a pleasant read. Thanks for your careful review. I will re-roll the patch series as soon as I get the chance. Michael -- Michael Haggerty mhagger@alum.mit.edu http://softwareswirl.blogspot.com/