Re: [PATCH 8/8] wrap_in_html(): process message in bulk rather than line-by-line

From: Junio C Hamano <gitster@pobox.com>
Date: Dec 2, 2012, 09:25 UTC
Message-ID: <7vpq2slsb4.fsf@alter.siamese.dyndns.org>
In-Reply-To: <50B8B73A.4060801@alum.mit.edu>

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.

Thanks.