Re: [PATCH 1/4] Add history graph API
- From
- Teemu Likonen <tlikonen@iki.fi>
- Date
- Apr 7, 2008, 05:24 UTC
- Message-ID
- <200804070824.10764.tlikonen@iki.fi>
- In-Reply-To
- <20080406224724.GC5822@adamsimpkins.net>
Adam Simpkins kirjoitti:
> Actually, going back and testing this, it looks like I have a bug > when handling --graph together with --pretty=format. There's a > missing newline after the user's format message and the next graph > line. I'll try to fix this and submit a patch later this evening.
Also, the output is not indented for options that display some additional information to commit message. Those include:
--raw --stat --numstat --shortstat --summary --name-only --name-status
I'm not sure if the diff output of -p, -u etc. should be indented--probably not--but for different stat and summary options it would be nice to not have their output displayed over the graph area. Especially --name-status is funny since it displays "M" to column 1 to indicate modified file while "M" also means merge commit in the graph.