From: Junio C Hamano <junkio@cox.net>
Date: Tue, 27 Jun 2006 18:03:24 GMT
Subject: Re: [PATCH] Print empty line between raw, stat, summary and patch
Message-ID: <7vhd268oib.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20060627150917.7eabde58.tihirvon@gmail.com>

Timo Hirvonen <tihirvon@gmail.com> writes:

> Signed-off-by: Timo Hirvonen <tihirvon@gmail.com>
> ---
>
>   Should we print options->line_termination instead of \n between all
>   fields?

I personally do not think it is a big deal since combination of
stat, summary and patch are primarily for human consumption, but
somebody might want to write a frontend GUI on top of this
output and having an easy way to seperate the parts for machine
consumption might be helpful.

>   The old code didn't support as many combinations of raw,
>   stat, summary and patch so I'm not 100% sure about this.

Think of it as an opportunity to come up with the most sensible
without having to worry about backward compatibility ;-)  I'll
let you know what I think after I stare at its output for some
time.

Thanks.