diff options
| author | Mirko Faina <mroik@delayed.space> | 2026-03-23 17:57:33 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2026-03-23 13:06:59 -0700 |
| commit | 617db87921e835f02ac132e86d0f7edc0d72915e (patch) | |
| tree | 5f30aef836a36c9bfbfff0a0ae395330832570fb /builtin | |
| parent | 24d174f9917cce804c9057061f7da0dbd3b88a24 (diff) | |
| download | git-617db87921e835f02ac132e86d0f7edc0d72915e.tar.xz | |
format-patch: wrap generate_commit_list_cover()
While most conventions should not allow for the text lines in commit
messages to get too long, when they do it could make emails harder to
read.
Teach generate_commit_list_cover() to wrap its commit lines if they are
too long.
Signed-off-by: Mirko Faina <mroik@delayed.space>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/log.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/builtin/log.c b/builtin/log.c index 47126f9064..d1765ce4ad 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -40,6 +40,7 @@ #include "progress.h" #include "commit-slab.h" #include "advice.h" +#include "utf8.h" #include "commit-reach.h" #include "range-diff.h" @@ -1364,6 +1365,7 @@ static void generate_commit_list_cover(FILE *cover_file, const char *format, struct commit **list, int n) { struct strbuf commit_line = STRBUF_INIT; + struct strbuf wrapped_line = STRBUF_INIT; struct pretty_print_context ctx = {0}; struct rev_info rev = REV_INFO_INIT; @@ -1373,12 +1375,16 @@ static void generate_commit_list_cover(FILE *cover_file, const char *format, rev.nr = i; repo_format_commit_message(the_repository, list[n - i], format, &commit_line, &ctx); - fprintf(cover_file, "%s\n", commit_line.buf); + strbuf_add_wrapped_text(&wrapped_line, commit_line.buf, 0, 0, + MAIL_DEFAULT_WRAP); + fprintf(cover_file, "%s\n", wrapped_line.buf); strbuf_reset(&commit_line); + strbuf_reset(&wrapped_line); } fprintf(cover_file, "\n"); strbuf_release(&commit_line); + strbuf_release(&wrapped_line); } static void make_cover_letter(struct rev_info *rev, int use_separate_file, |