On 12/02/2012 10:25 AM, Junio C Hamano wrote: > Michael Haggerty <mhagger@xxxxxxxxxxxx> 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@xxxxxxxxxxxx http://softwareswirl.blogspot.com/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html