# [PATCH] Add `format.to` configuration variable. 2 messages from 2010-01-13 to 2010-01-13. Participants: Steven Drake, Johannes Sixt. Thread: https://gitlist.dev/t/22194 ## Steven Drake, 2010-01-13 07:55 Subject: [PATCH] Add `format.to` configuration variable. Message-ID: URL: https://gitlist.dev/e/alpine.LNX.2.00.1001132054380.24673%40vqena.qenxr.bet.am ``` Signed-off-by: Steven Drake --- 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 ``` ## Johannes Sixt, 2010-01-13 08:20 Subject: Re: [PATCH] Add `format.to` configuration variable. Message-ID: <4B4D826A.7090405@viscovery.net> URL: https://gitlist.dev/e/4B4D826A.7090405%40viscovery.net In-Reply-To: ``` 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 ```