DORMANTno replies

[PATCH] format-patch: set default stat_width to 72

From: Jeff King <hidden>
Date: 2016-06-15 22:44:27
Subsystem: the rest · Maintainer: Linus Torvalds

The usual default is 80, which should fit in the terminals
of most users. However, it is usually considered polite to
keep email lines to 72 characters to accomodate a few layers
of quoting.

Signed-off-by: Jeff King <redacted>
---
On Thu, Apr 03, 2008 at 06:14:18AM -0400, Jeff King wrote:
format-patch already defaults to "--stat --summary -p" if you give it
no format options. The default stat is 80, I believe; for format-patch
it should probably be bumped to 72 anyway.
This should be totally harmless and maint-worthy, I would think.

 builtin-log.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/builtin-log.c b/builtin-log.c
index 5c00725..8d7b975 100644
--- a/builtin-log.c
+++ b/builtin-log.c
@@ -771,6 +771,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)
 	rev.ignore_merges = 1;
 	rev.diffopt.msg_sep = "";
 	DIFF_OPT_SET(&rev.diffopt, RECURSIVE);
+	rev.diffopt.stat_width = 72;
 
 	rev.subject_prefix = fmt_patch_subject_prefix;
 
-- 
1.5.5.rc2.171.g7962.dirty
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help