{"thread":{"id":"22194","subject":"[PATCH] Add `format.to` configuration variable.","startedAt":"2010-01-13T07:55:39Z","lastAt":"2010-01-13T08:20:58Z","messageCount":2,"participants":["Steven Drake","Johannes Sixt"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"131439","messageId":"alpine.LNX.2.00.1001132054380.24673@vqena.qenxr.bet.am","threadId":"22194","inReplyTo":null,"subject":"[PATCH] Add `format.to` configuration variable.","fromName":"Steven Drake","fromEmail":"sdrake@xnet.co.nz","sentAt":"2010-01-13T07:55:39Z","receivedAt":"2010-01-13T07:55:39Z","isPatch":true,"sender":{"key":"sdrake@xnet.co.nz","avatar":null},"body":"Signed-off-by: Steven Drake <sdrake@xnet.co.nz>\n---\nThis email was address using this new configuration variable.\n---\n builtin-log.c |    7 +++++++\n 1 files changed, 7 insertions(+), 0 deletions(-)\n\ndiff --git a/builtin-log.c b/builtin-log.c\nindex 41b6df4..a608c3d 100644\n--- a/builtin-log.c\n+++ b/builtin-log.c\n@@ -501,6 +501,13 @@ static int git_format_config(const char *var, const char *value, void *cb)\n \t}\n \tif (!strcmp(var, \"format.suffix\"))\n \t\treturn git_config_string(&fmt_patch_suffix, var, value);\n+\tif (!strcmp(var, \"format.to\")) {\n+\t\tif (!value)\n+\t\t\treturn config_error_nonbool(var);\n+\t\tALLOC_GROW(extra_to, extra_to_nr + 1, extra_to_alloc);\n+\t\textra_to[extra_to_nr++] = xstrdup(value);\n+\t\treturn 0;\n+\t}\n \tif (!strcmp(var, \"format.cc\")) {\n \t\tif (!value)\n \t\t\treturn config_error_nonbool(var);\n-- \n1.6.6.85.ga4c31\n"},{"id":"131440","messageId":"4B4D826A.7090405@viscovery.net","threadId":"22194","inReplyTo":"alpine.LNX.2.00.1001132054380.24673@vqena.qenxr.bet.am","subject":"Re: [PATCH] Add `format.to` configuration variable.","fromName":"Johannes Sixt","fromEmail":"j.sixt@viscovery.net","sentAt":"2010-01-13T08:20:58Z","receivedAt":"2010-01-13T08:20:58Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"You submitted a number of patches in the last hours, but you gave *zero*\nmotivation for them, i.e., the commit messages lack words about *why* the\nchanges are good (and what they are good for).\n\nThis particular patch lacks documentation, in particular, it's worth to\nmention that format.to could be given more than once.\n\n-- Hannes\n"}]}