diff options
| author | Kirill Smelkov <kirr@landau.phys.spbu.ru> | 2009-01-12 15:22:11 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2009-01-12 15:22:11 -0800 |
| commit | ddfb3696b91e2a2261b79d0828a7a715e3d71c1e (patch) | |
| tree | 269ef06570087226f3c9d939ba8349a74a29b82e | |
| parent | mailinfo: correctly handle multiline 'Subject:' header (diff) | |
| download | git-ddfb3696b91e2a2261b79d0828a7a715e3d71c1e.tar.gz git-ddfb3696b91e2a2261b79d0828a7a715e3d71c1e.zip | |
mailinfo: 'From:' header should be unfold as well
At present we do headers unfolding (see RFC822 3.1.1. LONG HEADER FIELDS) for
all fields except 'From' (always) and 'Subject' (when keep_subject is set)
Not unfolding 'From' is a bug -- see above-mentioned RFC link.
Signed-off-by: Kirill Smelkov <kirr@landau.phys.spbu.ru>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin-mailinfo.c | 1 | ||||
| -rw-r--r-- | t/t5100/sample.mbox | 5 |
2 files changed, 5 insertions, 1 deletions
diff --git a/builtin-mailinfo.c b/builtin-mailinfo.c index fcb32c9818..dacc8ac2d0 100644 --- a/builtin-mailinfo.c +++ b/builtin-mailinfo.c @@ -871,6 +871,7 @@ static void handle_info(void) } output_header_lines(fout, "Subject", hdr); } else if (!memcmp(header[i], "From", 4)) { + cleanup_space(hdr); handle_from(hdr); fprintf(fout, "Author: %s\n", name.buf); fprintf(fout, "Email: %s\n", email.buf); diff --git a/t/t5100/sample.mbox b/t/t5100/sample.mbox index 94da4daa1a..38725f38d2 100644 --- a/t/t5100/sample.mbox +++ b/t/t5100/sample.mbox @@ -2,7 +2,10 @@ From nobody Mon Sep 17 00:00:00 2001 -From: A U Thor <a.u.thor@example.com> +From: A + U + Thor + <a.u.thor@example.com> Date: Fri, 9 Jun 2006 00:44:16 -0700 Subject: [PATCH] a commit. |