Re: [PATCH 3/3] pretty: support multiline subjects with format:
- From
René Scharfe <rene.scharfe@lsrfire.ath.cx>
- Date
- Dec 29, 2008, 22:15 UTC
- Message-ID
- <49594C16.2010406@lsrfire.ath.cx>
- In-Reply-To
- <4956C47B.4020602@lsrfire.ath.cx>
René Scharfe schrieb:
Show 5 quoted lines
> Markus Heidelberg schrieb: >> Sorry, I haven't tested your patch set, what is the output of >> git-shortlog like now? > > It's unchanged since it has it's own commit message parser.
... which displays the first line of the commit message, unlike --pretty=oneline. Here's a quick fix. I probably won't have time to come up with something prettier this year.
René
builtin-shortlog.c | 9 ++++++--- pretty.c | 4 ++-- 2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/builtin-shortlog.c b/builtin-shortlog.c index d03f14f..e492906 100644 --- a/builtin-shortlog.c +++ b/builtin-shortlog.c @@ -29,6 +29,9 @@ static int compare_by_number(const void *a1, const void *a2) return -1; } +const char *format_subject(struct strbuf *sb, const char *msg, + const char *line_separator); + static void insert_one_record(struct shortlog *log, const char *author, const char *oneline) @@ -41,6 +44,7 @@ static void insert_one_record(struct shortlog *log, size_t len; const char *eol; const char *boemail, *eoemail; + struct strbuf subject = STRBUF_INIT; boemail = strchr(author, '<'); if (!boemail) @@ -89,9 +93,8 @@ static void insert_one_record(struct shortlog *log, while (*oneline && isspace(*oneline) && *oneline != '\n') oneline++; len = eol - oneline; - while (len && isspace(oneline[len-1])) - len--; - buffer = xmemdupz(oneline, len); + format_subject(&subject, oneline, " "); + buffer = strbuf_detach(&subject, NULL); if (dot3) { int dot3len = strlen(dot3); diff --git a/pretty.c b/pretty.c index 343dca5..421d9c5 100644 --- a/pretty.c +++ b/pretty.c @@ -486,8 +486,8 @@ static void parse_commit_header(struct format_commit_context *context) context->commit_header_parsed = 1; } -static const char *format_subject(struct strbuf *sb, const char *msg, - const char *line_separator) +const char *format_subject(struct strbuf *sb, const char *msg, + const char *line_separator) { int first = 1;