{"thread":{"id":"44028","subject":"[PATCH] send-email: use sanitized address for cc","startedAt":"2016-09-07T07:00:05Z","lastAt":"2016-09-07T07:00:05Z","messageCount":1,"participants":["changbin.du@gmail.com"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"301241","messageId":"20160907065733.28467-1-changbin.du@gmail.com","threadId":"44028","inReplyTo":null,"subject":"[PATCH] send-email: use sanitized address for cc","fromName":"","fromEmail":"changbin.du@gmail.com","sentAt":"2016-09-07T06:57:33Z","receivedAt":"2016-09-07T07:00:05Z","isPatch":true,"sender":{"key":"changbin.du@gmail.com","avatar":null},"body":"From: \"Du, Changbin\" <changbin.du@gmail.com>\n\nSome username in the email address may include a ','. In this case,\nwe need quoting the username field so it will not be parsed as two\nsingle addresses by Mail::Address->parse().\n\nFor example, my eamil address \"Du, Changbin <changbin.du@gmail.com>\"\ncan be parsed as two addresses \"Du\" and \"Changbin <changbin.du@gmail.com>\"\nif username is not quoted. (\"Du, Changbin\" is a legal format of signature\nin Chinese)\n\nThe sanitized address can be used because quote is added automactically.\n\nSigned-off-by: Du, Changbin <changbin.du@gmail.com>\n---\n git-send-email.perl | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/git-send-email.perl b/git-send-email.perl\nindex 6958785..6ec189e 100755\n--- a/git-send-email.perl\n+++ b/git-send-email.perl\n@@ -1554,9 +1554,9 @@ foreach my $t (@files) {\n \t\t\t\tnext if $suppress_cc{'sob'} and $what =~ /Signed-off-by/i;\n \t\t\t\tnext if $suppress_cc{'bodycc'} and $what =~ /Cc/i;\n \t\t\t}\n-\t\t\tpush @cc, $c;\n+\t\t\tpush @cc, $sc;\n \t\t\tprintf(\"(body) Adding cc: %s from line '%s'\\n\",\n-\t\t\t\t$c, $_) unless $quiet;\n+\t\t\t\t$sc, $_) unless $quiet;\n \t\t}\n \t}\n \tclose $fh;\n-- \n2.7.4\n\n"}]}