From: Teemu Likonen Date: Mon, 07 Apr 2008 05:24:10 GMT Subject: Re: [PATCH 1/4] Add history graph API 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.