[PATCH] Add `format.to` configuration variable.

Subsystems: the rest

DORMANTno replies

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

[PATCH] Add `format.to` configuration variable.

From: Steven Drake <hidden>
Date: 2016-06-15 22:48:00

Signed-off-by: Steven Drake <redacted>
---
This email was address using this new configuration variable.
---
 builtin-log.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/builtin-log.c b/builtin-log.c
index 41b6df4..a608c3d 100644
--- a/builtin-log.c
+++ b/builtin-log.c
@@ -501,6 +501,13 @@ static int git_format_config(const char *var, const char *value, void *cb)
 	}
 	if (!strcmp(var, "format.suffix"))
 		return git_config_string(&fmt_patch_suffix, var, value);
+	if (!strcmp(var, "format.to")) {
+		if (!value)
+			return config_error_nonbool(var);
+		ALLOC_GROW(extra_to, extra_to_nr + 1, extra_to_alloc);
+		extra_to[extra_to_nr++] = xstrdup(value);
+		return 0;
+	}
 	if (!strcmp(var, "format.cc")) {
 		if (!value)
 			return config_error_nonbool(var);
-- 
1.6.6.85.ga4c31

Re: [PATCH] Add `format.to` configuration variable.

From: Johannes Sixt <hidden>
Date: 2016-06-15 22:48:00

You submitted a number of patches in the last hours, but you gave *zero*
motivation for them, i.e., the commit messages lack words about *why* the
changes are good (and what they are good for).

This particular patch lacks documentation, in particular, it's worth to
mention that format.to could be given more than once.

-- Hannes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help