From: Michael Haggerty <mhagger@alum.mit.edu>
Date: Sun, 02 Dec 2012 10:35:29 GMT
Subject: Re: [PATCH 8/8] wrap_in_html(): process message in bulk rather than line-by-line
Message-ID: <50BB2EF1.5020003@alum.mit.edu>
In-Reply-To: <7vpq2slsb4.fsf@alter.siamese.dyndns.org>

On 12/02/2012 10:25 AM, Junio C Hamano wrote:
> Michael Haggerty <mhagger@alum.mit.edu> writes:
> 
>> Nevertheless, the old code was even *more* broken because it added a
>> "</pre>" regardless of whether the separator line had been seen,...
> 
> OK. I'll rewrite the tail-end of the original log message to read:
> 
>     The old code would have created invalid output when there was no
>     body, emitting a closing </pre> without a blank line nor an opening
>     <pre> after the header.  The new code simply returns in this
>     situation without doing harm (even though either would not make much
>     sense in the context of imap-send that is meant to send out patches).
> 
> and squash this in.

ACK.  Thanks.

Michael

-- 
Michael Haggerty
mhagger@alum.mit.edu
http://softwareswirl.blogspot.com/