Re: [PATCH] Print empty line between raw, stat, summary and patch
- From
Junio C Hamano <junkio@cox.net>
- Date
- Jun 27, 2006, 18:03 UTC
- Message-ID
- <7vhd268oib.fsf@assigned-by-dhcp.cox.net>
- In-Reply-To
- <20060627150917.7eabde58.tihirvon@gmail.com>
Timo Hirvonen <tihirvon@gmail.com> writes:
Show 5 quoted lines
> 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.