git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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

From
Jeff King <peff@peff.net>
Date
Apr 3, 2008, 10:22 UTC
Message-ID
<20080403102214.GA23121@coredump.intra.peff.net>

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 <peff@peff.net>
---
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
Next: Junio C Hamano
Message 1 of 3 in “format-patch: set default stat_width to 72”
  1. format-patch: set default stat_width to 72Jeff King, Apr 3, 2008
  2. Junio C HamanoApr 4, 2008
  3. Jeff KingApr 4, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.