aboutsummaryrefslogtreecommitdiff
path: root/diff.c
diff options
context:
space:
mode:
authorKevin Ballard <kevin@sb.org>2008-07-29 22:49:33 -0700
committerJunio C Hamano <gitster@pobox.com>2008-07-29 23:18:15 -0700
commit6b2fbaaffc5ec762eae5d23b58b1dc0a88e2275e (patch)
treec74662decbfde7365c0f7e449458dfa4d5598c0c /diff.c
parent8cb070a4cfea48f88ada02a87320c3ca63b934cb (diff)
downloadgit-6b2fbaaffc5ec762eae5d23b58b1dc0a88e2275e.tar.gz
git-6b2fbaaffc5ec762eae5d23b58b1dc0a88e2275e.tar.xz
format-patch: Produce better output with --inline or --attach
This patch makes two small changes to improve the output of --inline and --attach. The first is to write a newline preceding the boundary. This is needed because MIME defines the encapsulation boundary as including the preceding CRLF (or in this case, just LF), so we should be writing one. Without this, the last newline in the pre-diff content is consumed instead. The second change is to always write the line termination character (default: newline) even when using --inline or --attach. This is simply to improve the aesthetics of the resulting message. When using --inline an email client should render the resulting message identically to the non-inline version. And when using --attach this adds a blank line preceding the attachment in the email, which is visually attractive. Signed-off-by: Kevin Ballard <kevin@sb.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'diff.c')
-rw-r--r--diff.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/diff.c b/diff.c
index a07812c5c..cbf25473c 100644
--- a/diff.c
+++ b/diff.c
@@ -3223,11 +3223,10 @@ void diff_flush(struct diff_options *options)
if (output_format & DIFF_FORMAT_PATCH) {
if (separator) {
+ putc(options->line_termination, options->file);
if (options->stat_sep) {
/* attach patch instead of inline */
fputs(options->stat_sep, options->file);
- } else {
- putc(options->line_termination, options->file);
}
}