Re: [PATCH] Understand the meaning of commas in git-send-email

From: Junio C Hamano <junkio@cox.net>
Date: Oct 10, 2006, 17:44 UTC
Message-ID: <7vzmc412w7.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20061010145936.GC8993@parisc-linux.org>

Matthew Wilcox <matthew@wil.cx> writes:

Show 5 quoted lines

> I made the mistake of adding one Cc: line with two email addresses on
> it, instead of two Cc: lines with one email address each.  Extending
> git-send-email to parse that case turns out to not be too much work.
>
> Signed-off-by: Matthew Wilcox <matthew@wil.cx>

I am not sure. The input addresses this part of the code deals with are of human readable form

	Matthew Wilcox <matthew@wil.cx>
        matthew@wil.cx (Matthew Wilcox)

and the loop is extracting the part inside <> pair.

I have a mild suspicion that your patch may need to understand quoting to handle somebody else's name safely and correctly.

	junkio@cox.net (Junio C Hamano, aka gitster)