diff options
author | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-06-18 13:52:05 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2005-06-18 13:52:05 -0700 |
commit | 28342a5d328fa11829dbe883837e21e08dd83388 (patch) | |
tree | c58a7f7c4b45af2f4cb4583749522692ecec75c2 | |
parent | 8b3a1e056f2107deedfdada86046971c9ad7bb87 (diff) | |
download | git-28342a5d328fa11829dbe883837e21e08dd83388.tar.gz git-28342a5d328fa11829dbe883837e21e08dd83388.tar.xz |
Make "pretty" commit logs always show merge information
Otherwise the "git log" information doesn't tell enough to make sense of
a merge.
I'll need to add some parent information for regular entries too, I
think, but the merge is more important.
-rw-r--r-- | commit.c | 24 |
1 files changed, 24 insertions, 0 deletions
@@ -222,10 +222,29 @@ static int is_empty_line(const char *line, int len) return !len; } +static int add_parent_info(enum cmit_fmt fmt, char *buf, const char *line, int parents) +{ + int offset = 0; + switch (parents) { + case 1: + break; + case 2: + /* Go back to the previous line: 40 characters of previous parent, and one '\n' */ + offset = sprintf(buf, "Merge: %.40s\n", line-41); + /* Fallthrough */ + default: + /* Replace the previous '\n' with a space */ + buf[offset-1] = ' '; + offset += sprintf(buf + offset, "%.40s\n", line+7); + } + return offset; +} + unsigned long pretty_print_commit(enum cmit_fmt fmt, const char *msg, unsigned long len, char *buf, unsigned long space) { int hdr = 1, body = 0; unsigned long offset = 0; + int parents = 0; for (;;) { const char *line = msg; @@ -257,6 +276,11 @@ unsigned long pretty_print_commit(enum cmit_fmt fmt, const char *msg, unsigned l offset += linelen; continue; } + if (!memcmp(line, "parent ", 7)) { + if (linelen != 48) + die("bad parent line in commit"); + offset += add_parent_info(fmt, buf + offset, line, ++parents); + } if (!memcmp(line, "author ", 7)) offset += add_author_info(fmt, buf + offset, line, linelen); continue; |